Open
Bug 1020875
Opened 11 years ago
Updated 9 months ago
Define, consolidate and fix Assert.jsm's deepEqual() method
Categories
(Testing :: General, defect)
Testing
General
Tracking
(Not tracked)
NEW
People
(Reporter: mikedeboer, Unassigned)
References
Details
(Whiteboard: p=3)
In a relevant dev-platform thread[1] jgraham noted that the `deepEqual` method underspecified and its behavior underdefined.
To quote jgraham: "They encourage the use of deepEqual which has underdefined semantics, particularly in the case of objects that contain cycles (it looks like
Assert.jsm goes into an infinite loop in this case [...])."
Additionally, the property inspection is probably incorrect when comparing dictionary objects (Iterables).
I suggest spec'ing this method to the extent we can at this point before continuing with an implementation.
[1] https://groups.google.com/forum/#!topic/mozilla.dev.platform/vdyLJBJTdYY
Flags: firefox-backlog+
|
Reporter | |
Updated•9 years ago
|
Assignee: mdeboer → nobody
Status: ASSIGNED → NEW
|
||
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•