Skip to content

Add tests to improve internal/crypto/diffiehellman.js coverage

Rodrigo Muino Tomonari requested to merge github/fork/Leko/test_crypto_diffiehellman into master

I added these tests

  • Call crypto.ECDH without new keyword
  • Call crypto.DiffieHellmanGroup without new keyword
  • Call crypto.DiffieHellman without new keyword

Current coverage is here: https://coverage.nodejs.org/coverage-1fa59b4c7e575ca7/root/internal/crypto/diffiehellman.js.html

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

test

Merge request reports