Closed Bug 784873 Opened 7 years ago Closed 7 years ago

Improve ParallelArray testsuite code

Categories

(Core :: JavaScript Engine, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: shu, Assigned: shu)

Details

Attachments

(1 file)

Currently I'm testing equality on ParallelArray objects by calling .toString(), which is unfortunate.
Assignee: general → shu
Flags: in-testsuite+
Attached patch patchSplinter Review
Add and use |assertEqParallelArray| helper to compare PAs directly instead of using |toString|.
Attachment #654490 - Flags: review?(dvander)
Attachment #654490 - Flags: review?(dvander) → review-
Comment on attachment 654490 [details] [diff] [review]
patch

Review of attachment 654490 [details] [diff] [review]:
-----------------------------------------------------------------

Whoops, I'm sorry! That was a total misclick on my part.
Attachment #654490 - Flags: review- → review+
https://hg.mozilla.org/mozilla-central/rev/0eece65be2df
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.