Closed Bug 628890 Opened 10 years ago Closed 10 years ago

mochitests-2: test_bug509055.html should not call SimpleTest.finish() before it actually has

Categories

(Core Graveyard :: History: Global, defect)

defect
Not set
normal

Tracking

(status1.9.2 .14-fixed)

VERIFIED FIXED
mozilla2.0b11
Tracking Status
status1.9.2 --- .14-fixed

People

(Reporter: sgautherie, Assigned: sgautherie)

References

()

Details

Attachments

(1 file)

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1296007795.1296008703.26408.gz&fulltext=1
OS X 10.6 comm-central-trunk debug test mochitests-2/5 on 2011/01/25 18:09:55
{
...
460 INFO TEST-PASS | /tests/docshell/test/test_bug509055.html | SHEntry's title should change when we change. - "Changed" should equal "Changed"
461 INFO TEST-END | /tests/docshell/test/test_bug509055.html | finished in 2976ms
462 INFO TEST-START | /tests/docshell/test/test_bug511449.html
Final yield.
...
}

I noticed this while checking bug 556687:
I assume they are unrelated but it would be preferable to be sure(r).

NB: This would be fixed by bug 615546, if it was allowed to stay in the tree :-<
The code in question is:

  SimpleTest.finish();
  dump('Final yield.\n');
  yield;
} // end of test

It would probably be reasonable to call SimpleTest.finish() after the yield.  I dunno if that would fix any problems you're observing, though.
Hm.  Yes, I think that patch is better than moving the SimpleTest.finish().  If we move the finish(), it never gets called.  But if we take out the yield, we get a StopIteration exception.
["Mid-air collision detected!" ...]

(In reply to comment #1)

> I dunno if that would fix any problems you're observing, though.

Probably not, but it will sort the log out at least.
Attachment #507019 - Flags: review?(Olli.Pettay) → review+
Comment on attachment 507019 [details] [diff] [review]
(Av1) Wrap the call in SimpleTest.executeSoon() (ftb)
[Checked in: Comment 5]

http://hg.mozilla.org/mozilla-central/rev/cb707fbc99d4

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/bd974e9f527a
Attachment #507019 - Attachment description: (Av1) Wrap the call in SimpleTest.executeSoon() (ftb) → (Av1) Wrap the call in SimpleTest.executeSoon() (ftb) [Checked in: Comment 5]
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1296067541.1296068149.30123.gz&fulltext=1
Rev3 Fedora 12x64 mozilla-central opt test mochitests-2/5 on 2011/01/26 10:45:41
{
554 INFO TEST-START | /tests/docshell/test/test_bug509055.html
Waiting for initial load.
Got initial load.  Spinning event loop.
Waiting for hashchange.
onChildHashchange() called.
Waiting for second hashchange.
onChildHashchange() called.
Got second hashchange.  Spinning event loop.
555 INFO TEST-PASS | /tests/docshell/test/test_bug509055.html | SHEntry's title should change when we change. - "Changed" should equal "Changed"
Final yield.
556 INFO TEST-END | /tests/docshell/test/test_bug509055.html | finished in 357ms
}

V.Fixed
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.