Skip to content

assert: Add support for Map and Set in deepEqual #12142

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

Closed
josephg wants to merge 13 commits into from
Closed

assert: Add support for Map and Set in deepEqual #12142

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
561561a
assert: Add support for Map and Set in deepEqual
josephg Mar 31, 2017
f051840
assert: Use isSet and isMap from process.binding
josephg Mar 31, 2017
1d6cda6
assert: Added deeper equality checking for Map,Set
josephg Mar 31, 2017
800ae46
assert: Fixed nit in code review
josephg Mar 31, 2017
031f6f3
assert: Fixed typo in comment
josephg Mar 31, 2017
d6baaee
assert: var -> const and added tests
josephg Mar 31, 2017
8fb6ebf
assert: restrict O(n^2) search
josephg Mar 31, 2017
acef701
assert: added null check in set/map short circuit
josephg Apr 1, 2017
ee131e8
assert: update documentation to reflect changes
josephg Apr 1, 2017
7bc29b0
assert: Updated based on PR
josephg Apr 1, 2017
6bdfcaf
assert: updated docs based on PR
josephg Apr 1, 2017
fc5196a
assert: fixed docs # link
josephg Apr 1, 2017
7f9d4d8
assert: refactored based on PR
josephg Apr 1, 2017
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
assert: fixed docs # link
  • Loading branch information
@josephg
josephg committed Apr 1, 2017
commit fc5196a20d1dc90d15f07ca17456256643497f50
3 changes: 2 additions & 1 deletion doc/api/assert.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ Generally identical to `assert.deepEqual()` with three exceptions:

1. Primitive values are compared using the [Strict Equality Comparison][]
( `===` ). Set values and Map keys are compared using the [SameValueZero][]
comparison. (Which means they are free of the [Caveats][]).
comparison. (Which means they are free of the [caveats][]).
2. [`[[Prototype]]`][prototype-spec] of objects are compared using
the [Strict Equality Comparison][] too.
3. [Type tags][Object.prototype.toString()] of objects should be the same.
Expand Down Expand Up @@ -586,6 +586,7 @@ For more information, see
[`assert.ok()`]: #assert_assert_ok_value_message
[`assert.throws()`]: #assert_assert_throws_block_error_message
[`Error`]: errors.html#errors_class_error
[caveats]: #assert_caveats
[`RegExp`]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions
[`TypeError`]: errors.html#errors_class_typeerror
[Abstract Equality Comparison]: https://tc39.github.io/ecma262/#sec-abstract-equality-comparison
Expand Down