Skip to content

test: add missing assert.deepEqual() test case

Rodrigo Muino Tomonari requested to merge github/fork/Trott/missing into master
Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

test assert

Description of change

None of the existing tests checked for the situation where assert.deepEqual() receives two objects that have the same number of keys but different key names. Therefore, line 242 of lib/assert.js was not being exercised by any tests.

This change adds the missing test case.

Refs: https://node-core-coverage.addaleax.net/coverage-05b566a5ef7c41f6/root/assert.js.html

Merge request reports