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

Status: NEW

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

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

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.

Comment 1

[Masatoshi Kimura [:emk]]

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.

Comment 2

[Honza Bambas (:mayhemer)]

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.

Comment 3

[Phil Ringnalda (:philor)]

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

Comment 4

[Phil Ringnalda (:philor)]

Skipped on android in https://hg.mozilla.org/mozilla-central/rev/9ab72e2ec5e4

Comment 5

[Ryan VanderMeulen [:RyanVM]]

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

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/42be2d33f6ed