Open Bug 962029 Opened 6 years ago Updated 6 months ago

test_bug746272-2.html must not assume that it runs immediately after test_bug746272-1.html

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

People

(Reporter: masayuki, Unassigned)

References

Details

(Whiteboard: [test disabled on Android and B2G][leave open])

I'm adding a new test in bug 935876. That causes test_bug746272-1.html moved to mochitest-6 from mochitest-7 on Android.

However, test_bug746272-2.html assumes that it runs immediately after test_bug746272-1.html. In other words, -2 depends on -1.

Therefore, this causes new orange.
https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=cddad7e6f3c0
https://tbpl.mozilla.org/php/getParsedLog.php?id=33322043&tree=Try&full=1#error0

I'll disable the -2 test in bug 935876 if this isn't fixed before that.

I guess that when 2 tests must be run sequentially, you should use <iframe> or mochitest-browser-chrome.
Is it impossible to combine the two tests? I don't think it's a good idea for a test to depend on another test.
I think we may do something similar as has been done for the cookie test recently, just run the two parts in iframe.  The goal was to make the localStorage (old implementation) reload from disk.  Also, there is a testing API that may force it w/o using a separate window/iframe for the two test parts.
Looks like someone else beat you to the punch and added some other test which moved the boundaries, so https://tbpl.mozilla.org/php/getParsedLog.php?id=33647179&tree=Mozilla-Central etc., it's now permaorange
This is biting B2G debug on Aurora now too. Disabled.

https://hg.mozilla.org/integration/b2g-inbound/rev/42be2d33f6ed
https://hg.mozilla.org/releases/mozilla-aurora/rev/16f8f82f2582
Whiteboard: [test disabled on Android and B2G][leave open]
Assignee: honzab.moz → nobody
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.