Closed
Bug 486256
Opened 15 years ago
Closed 14 years ago
Mochitest test_Scriptaculous.html (all but 2) results fail to be actually logged
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
VERIFIED
FIXED
mozilla1.9.3a5
People
(Reporter: sgautherie, Assigned: Ms2ger)
References
Details
Attachments
(2 files)
820 bytes,
patch
|
sayrer
:
review+
|
Details | Diff | Splinter Review |
901 bytes,
patch
|
Details | Diff | Splinter Review |
Noticed while looking into bug 451287: (Reproduced on my local Win2K) { http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1237154733.1237161001.15457.gz&fulltext=1 Linux mozilla-central unit test on 2009/03/15 15:05:33 *** 32176 INFO Running /tests/dom/tests/mochitest/ajax/scriptaculous/test_Scriptaculous.html... *** 32177 INFO TEST-PASS | /tests/dom/tests/mochitest/ajax/scriptaculous/test_Scriptaculous.html | testDummy *** 32178 INFO TEST-PASS | /tests/dom/tests/mochitest/ajax/scriptaculous/test_Scriptaculous.html | testMultipleTestRunner *** 32180 INFO Running /tests/dom/tests/mochitest/bugs/test_bug132255.html... } With my local: *run alone: 818 checks. *run in the whole suite: 694 checks ("only") "seen" be the harness; but 2 only are logged. (Yet, these 692 "missing" 'TEST-PASS' are added to the "Passed" count.)
Flags: wanted1.9.2?
Assignee | ||
Comment 1•14 years ago
|
||
We don't support multiple test runners in one file. The second test causes us to call SimpleTest.finish while the final test is still being run.
Updated•14 years ago
|
Attachment #439536 -
Flags: review?(sayrer) → review+
Reporter | ||
Comment 2•14 years ago
|
||
Comment on attachment 439536 [details] [diff] [review] Remove the extra test Nit: I'd suggest to comment out this code and add a comment, rather than remove it. (To ease diff, ...)
Assignee | ||
Comment 3•14 years ago
|
||
(In reply to comment #2) > Nit: I'd suggest to comment out this code and add a comment, rather than remove > it. (To ease diff, ...) Sure.
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Comment 4•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/fe051655d85e
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
Reporter | ||
Comment 5•13 years ago
|
||
https://tbpl.mozilla.org/php/getParsedLog.php?id=8350686&tree=Firefox&full=1 Rev3 Fedora 12 mozilla-central opt test mochitests-2/5 on 2012-01-05 09:35:37 PST { 12491 INFO TEST-START | /tests/dom/tests/mochitest/ajax/scriptaculous/test_Scriptaculous.html 12492 INFO TEST-PASS | /tests/dom/tests/mochitest/ajax/scriptaculous/test_Scriptaculous.html | testDisposesProperly - 10 assertions, 0 failures, 0 errors ... 13192 INFO TEST-PASS | /tests/dom/tests/mochitest/ajax/scriptaculous/test_Scriptaculous.html | testAssertVisible - 26 assertions, 0 failures, 0 errors 13193 INFO TEST-END | /tests/dom/tests/mochitest/ajax/scriptaculous/test_Scriptaculous.html | finished in 14114ms } 702 checks logged :-) V.Fixed
Status: RESOLVED → VERIFIED
Flags: wanted1.9.2? → in-testsuite+
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•