Skip to content

test: Add test case for missing branch

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

Enhance crypto/util coverage. I added:

  • The case of id is string, flags is number
  • The case of flags is not 0

Current coverage: screen shot 2017-12-02 at 3 17 39 https://coverage.nodejs.org/coverage-06e1b0386196f8f8/root/internal/crypto/util.js.html

After this PR: screen shot 2017-12-02 at 3 24 45

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