Skip to content

util: add skipPrototype option to deep equality checks #59762

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
nodejs-github-bot merged 17 commits into from
Sep 14, 2025
Merged

util: add skipPrototype option to deep equality checks #59762

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
3cd8765
lib: update isDeepStrictEqual to support options
miguelmarcondesf Sep 4, 2025
0a88d1b
doc: add options parameter to util.isDeepStrictEqual
miguelmarcondesf Sep 4, 2025
cf0d1eb
lib: add skipPrototypeComparison option to deepStrictEqual for Assert…
miguelmarcondesf Sep 4, 2025
5d5a6f8
test: add skipPrototypeComparison tests for Assert class
miguelmarcondesf Sep 4, 2025
68d6d79
doc: add skipPrototypeComparison option to Assert class
miguelmarcondesf Sep 4, 2025
c9cef45
lib: update notDeepStrictEqual to support skipPrototypeComparison option
miguelmarcondesf Sep 4, 2025
540097a
test: enhance skipPrototypeComparison tests for Assert class
miguelmarcondesf Sep 4, 2025
e8ca02a
test: add skipPrototypeComparison tests for util.isDeepStrictEqual
miguelmarcondesf Sep 4, 2025
642911b
lib: add missing proto
miguelmarcondesf Sep 5, 2025
1532570
lib: add missing proto
miguelmarcondesf Sep 5, 2025
dc0c5bc
lib: refine keyCheck mode condition for strict comparisons
miguelmarcondesf Sep 5, 2025
d14f030
doc: update examples to use class syntax for Foo and Bar
miguelmarcondesf Sep 5, 2025
3c6bf3c
lib: remove unecessary option check
miguelmarcondesf Sep 5, 2025
01daebd
doc: update missing pr number
miguelmarcondesf Sep 8, 2025
b3a4f56
doc: update missing pr number
miguelmarcondesf Sep 8, 2025
87e11ab
lib: simplify isDeepStrictEqual options handling
miguelmarcondesf Sep 8, 2025
0ae1910
lib: update skipPrototypeComparison to skipPrototype
miguelmarcondesf Sep 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test: add skipPrototypeComparison tests for Assert class
  • Loading branch information
@miguelmarcondesf
miguelmarcondesf committed Sep 4, 2025
commit 5d5a6f83b966eb02d1d11c0e4d447fb03af84608
43 changes: 43 additions & 0 deletions test/parallel/test-assert-class.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -478,3 +478,46 @@ test('Assert class non strict with simple diff', () => {
);
}
});

// Shared setup for skipPrototypeComparison tests
{
const message = 'Expected values to be strictly deep-equal:\n' +
'+ actual - expected\n' +
'\n' +
' [\n' +
' 1,\n' +
' 2,\n' +
' 3,\n' +
' 4,\n' +
' 5,\n' +
'+ 6,\n' +
'- 9,\n' +
' 7\n' +
' ]\n';

function CoolClass(name) { this.name = name; }

function AwesomeClass(name) { this.name = name; }
const person = new CoolClass('Assert is inspiring');
const user = new AwesomeClass('Assert is inspiring');

test('Assert class strict with skipPrototypeComparison', () => {
const assertInstance = new Assert({ skipPrototypeComparison: true });

assert.throws(
() => assertInstance.deepEqual([1, 2, 3, 4, 5, 6, 7], [1, 2, 3, 4, 5, 9, 7]),
{ message }
);
assertInstance.deepEqual(person, user);
});

test('Assert class non strict with skipPrototypeComparison', () => {
const assertInstance = new Assert({ strict: false, skipPrototypeComparison: true });

assert.throws(
() => assertInstance.deepStrictEqual([1, 2, 3, 4, 5, 6, 7], [1, 2, 3, 4, 5, 9, 7]),
{ message }
);
assertInstance.deepStrictEqual(person, user);
});
}