Closed Bug 986653 Opened 10 years ago Closed 10 years ago

Add --e10s option to reftests

Categories

(Testing :: Reftest, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla31

People

(Reporter: billm, Assigned: billm)

References

Details

Attachments

(1 file)

Attached patch reftest-e10sSplinter Review
I realize that we already have the reftest-ipc test suite, but it works a little differently. On tinderbox, it only runs a much reduced set of tests. Also, I think having the --e10s option is nice for uniformity since we already have it for mochitests.
Attachment #8395020 - Flags: review?(ted)
Comment on attachment 8395020 [details] [diff] [review]
reftest-e10s

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

This is fine, but like I mentioned in person we should either make reftest-ipc use this mechanism, or get rid of reftest-ipc in favor of your reftest-e10s (or whatever you want to call it).
https://hg.mozilla.org/build/mozharness/file/52636b92cbfd/configs/unittests/linux_unittest.py#l75
Attachment #8395020 - Flags: review?(ted) → review+
https://hg.mozilla.org/mozilla-central/rev/52bc15db9553
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
You need to log in before you can comment on or make changes to this bug.