Open Bug 1020875 Opened 6 years ago Updated 2 years ago

Define, consolidate and fix Assert.jsm's deepEqual() method

Categories

(Testing :: General, defect)

defect
Not set

Tracking

(Not tracked)

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+
See Also: → 1074049
Assignee: mdeboer → nobody
Status: ASSIGNED → NEW
You need to log in before you can comment on or make changes to this bug.