Closed
Bug 483992
Opened 16 years ago
Closed 13 years ago
9 dom-level1-core and 4 dom-level2-html Mochitests run fine(?) but need to check/report success(/failure?)
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
VERIFIED
FIXED
mozilla14
People
(Reporter: sgautherie, Assigned: sgautherie)
References
(Blocks 1 open bug, )
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
{
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1237366670.1237376580.14560.gz&fulltext=1
Linux mozilla-central unit test on 2009/03/18 01:57:50
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1237366174.1237371300.5105.gz&fulltext=1
OS X 10.5.2 mozilla-central unit test on 2009/03/18 01:49:34
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1237364751.1237377089.15377.gz&fulltext=1
WINNT 5.2 mozilla-central unit test on 2009/03/18 01:25:51
*** 34178 INFO Running /tests/dom/tests/mochitest/dom-level1-core/test_elementreplaceattributewithself.html...
*** 34180 INFO Running /tests/dom/tests/mochitest/dom-level1-core/test_elementreplaceexistingattribute.html...
}
(I noticed this on my Windows SeaMonkey/1.9.2 build...)
The 12 others are:
{
34676 INFO Running /tests/dom/tests/mochitest/dom-level1-core/test_hc_elementreplaceattributewithself.html...
34703 INFO Running /tests/dom/tests/mochitest/dom-level1-core/test_hc_entitiesremovenameditem1.html...
34706 INFO Running /tests/dom/tests/mochitest/dom-level1-core/test_hc_entitiessetnameditem1.html...
35139 INFO Running /tests/dom/tests/mochitest/dom-level1-core/test_hc_nodevalue07.html...
35142 INFO Running /tests/dom/tests/mochitest/dom-level1-core/test_hc_nodevalue08.html...
35145 INFO Running /tests/dom/tests/mochitest/dom-level1-core/test_hc_notationsremovenameditem1.html...
35148 INFO Running /tests/dom/tests/mochitest/dom-level1-core/test_hc_notationssetnameditem1.html...
35507 INFO Running /tests/dom/tests/mochitest/dom-level1-core/test_nodelistindexequalzero.html...
37381 INFO Running /tests/dom/tests/mochitest/dom-level2-html/test_HTMLDocument18.html...
37384 INFO Running /tests/dom/tests/mochitest/dom-level2-html/test_HTMLDocument19.html...
37387 INFO Running /tests/dom/tests/mochitest/dom-level2-html/test_HTMLDocument20.html...
37390 INFO Running /tests/dom/tests/mochitest/dom-level2-html/test_HTMLDocument21.html...
}
Assignee | ||
Updated•15 years ago
|
Summary: 8 dom-level1-core and 4 dom-level2-html Mochitests run fine(?) but need to check/report success(/failure?) → 9 dom-level1-core and 4 dom-level2-html Mochitests run fine(?) but need to check/report success(/failure?)
Assignee | ||
Comment 1•15 years ago
|
||
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1242092605.1242096280.17513.gz&fulltext=1
OS X 10.5.2 mozilla-central unit test on 2009/05/11 18:43:25
Assignee | ||
Comment 2•15 years ago
|
||
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.5/1242134886.1242139285.14399.gz&fulltext=1
WINNT 5.2 mozilla-1.9.1 unit test on 2009/05/12 06:28:06
Assignee | ||
Updated•15 years ago
|
Assignee | ||
Comment 4•13 years ago
|
||
https://tbpl.mozilla.org/php/getParsedLog.php?id=10199321&tree=Firefox&full=1
Rev3 Fedora 12 mozilla-central debug test mochitests-3/5 on 2012-03-19 19:40:48 PDT for push b972b89518c3
{
1777 INFO TEST-START | /tests/dom/tests/mochitest/dom-level1-core/test_elementreplaceattributewithself.html
1778 INFO TEST-END | /tests/dom/tests/mochitest/dom-level1-core/test_elementreplaceattributewithself.html | finished in 640ms
}
*****
Ftr, when running this test alone:
[Mozilla/5.0 (Windows NT 5.0; rv:13.0a1) Gecko/20120201 Firefox/13.0a1 SeaMonkey/2.10a1] (nightly, 2012-02-01-00-30-09-comm-central-trunk)
{
Error: parentRunner is null
Source File: http://mochi.test:8888/tests/dom/tests/mochitest/dom-level1-core/DOMTestCase.js
Line: 648
}
That is a separate bug, but noting it here ftb.
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → sgautherie.bz
Target Milestone: --- → mozilla14
Assignee | ||
Comment 5•13 years ago
|
||
> Error: parentRunner is null
>
> That is a separate bug, but noting it here ftb.
Actually, that is not as separate as I thought initially...
I don't know why it was added at the time,
but it seems unneeded (to say the least) nowadays.
Attachment #607478 -
Flags: review?(sayrer)
Assignee | ||
Comment 6•13 years ago
|
||
I tried to be both minimalist and consistent.
Attachment #607514 -
Flags: review?(sayrer)
Assignee | ||
Comment 7•13 years ago
|
||
Comment on attachment 607514 [details] [diff] [review]
(Bv1) dom-level*-*/*: Work around tests that report no SimpleTest checks
[Checked in: Comment 12]
Whoever reviews this/these first.
Attachment #607514 -
Flags: review?(bzbarsky)
Comment 8•13 years ago
|
||
Try run for e67e54ee6782 is complete.
Detailed breakdown of the results available here:
https://tbpl.mozilla.org/?tree=Try&rev=e67e54ee6782
Results (out of 98 total builds):
success: 90
warnings: 8
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/sgautherie.bz@free.fr-e67e54ee6782
Assignee | ||
Comment 9•13 years ago
|
||
Comment on attachment 607478 [details] [diff] [review]
(Av1) dom-level*-*/DOMTestCase.js: Remove sayrer's override of SimpleTest._logResult()
[Backed out: Comment 21]
(In reply to Mozilla RelEng Bot from comment #8)
> https://tbpl.mozilla.org/?tree=Try&rev=e67e54ee6782
Succeeded.
Comment 10•13 years ago
|
||
Comment on attachment 607514 [details] [diff] [review]
(Bv1) dom-level*-*/*: Work around tests that report no SimpleTest checks
[Checked in: Comment 12]
r=me
Attachment #607514 -
Flags: review?(bzbarsky) → review+
Comment 11•13 years ago
|
||
Try run for b5edde2abb07 is complete.
Detailed breakdown of the results available here:
https://tbpl.mozilla.org/?tree=Try&rev=b5edde2abb07
Results (out of 108 total builds):
exception: 1
success: 95
warnings: 12
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/sgautherie.bz@free.fr-b5edde2abb07
Assignee | ||
Comment 12•13 years ago
|
||
Comment on attachment 607514 [details] [diff] [review]
(Bv1) dom-level*-*/*: Work around tests that report no SimpleTest checks
[Checked in: Comment 12]
https://hg.mozilla.org/mozilla-central/rev/866ef7425e08
Attachment #607514 -
Attachment description: (Bv1) dom-level*-*/*: Work around tests that report no SimpleTest checks → (Bv1) dom-level*-*/*: Work around tests that report no SimpleTest checks
[Checked in: Comment 12]
Attachment #607514 -
Flags: review?(sayrer)
Assignee | ||
Updated•13 years ago
|
Status: NEW → ASSIGNED
Flags: in-testsuite+
Comment 13•13 years ago
|
||
Try run for e3c5c83f2afb is complete.
Detailed breakdown of the results available here:
https://tbpl.mozilla.org/?tree=Try&rev=e3c5c83f2afb
Results (out of 119 total builds):
exception: 3
success: 90
warnings: 14
failure: 1
other: 11
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/sgautherie.bz@free.fr-e3c5c83f2afb
Timed out after 12 hours without completing.
Assignee | ||
Comment 14•13 years ago
|
||
Comment on attachment 607478 [details] [diff] [review]
(Av1) dom-level*-*/DOMTestCase.js: Remove sayrer's override of SimpleTest._logResult()
[Backed out: Comment 21]
Whoever reviews this first.
Attachment #607478 -
Flags: review?(bzbarsky)
Comment 15•13 years ago
|
||
Comment on attachment 607478 [details] [diff] [review]
(Av1) dom-level*-*/DOMTestCase.js: Remove sayrer's override of SimpleTest._logResult()
[Backed out: Comment 21]
I have no idea what this code is doing; someone else should review this.
Attachment #607478 -
Flags: review?(bzbarsky)
Assignee | ||
Updated•13 years ago
|
Attachment #607478 -
Flags: review?(rcampbell)
Assignee | ||
Updated•13 years ago
|
Assignee | ||
Comment 16•13 years ago
|
||
With
(In reply to Alex Keybl [:akeybl] from bug 735805 comment #77)
> (the above includes blanket approval for the other test fixes for Aurora 13)
status-firefox-esr10:
--- → affected
status-firefox11:
--- → wontfix
status-firefox12:
--- → affected
status-firefox13:
--- → affected
Keywords: checkin-needed
Whiteboard: [c-n: 866ef7425e08 to m-a]
Updated•13 years ago
|
Attachment #607478 -
Flags: review?(sayrer)
Attachment #607478 -
Flags: review?(rcampbell)
Attachment #607478 -
Flags: review+
Assignee | ||
Comment 17•13 years ago
|
||
Comment on attachment 607478 [details] [diff] [review]
(Av1) dom-level*-*/DOMTestCase.js: Remove sayrer's override of SimpleTest._logResult()
[Backed out: Comment 21]
https://hg.mozilla.org/mozilla-central/rev/273173a592dc
Attachment #607478 -
Attachment description: (Av1) dom-level*-*/DOMTestCase.js: Remove sayrer's override of SimpleTest._logResult() → (Av1) dom-level*-*/DOMTestCase.js: Remove sayrer's override of SimpleTest._logResult()
[Checked in: Comment 17]
Assignee | ||
Updated•13 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 18•13 years ago
|
||
if this is going to backport bug 739927 to Aurora, we should really not push these to Aurora. It's painful enough to star central now to bring that pain to another tree.
Comment 19•13 years ago
|
||
clearing until clarified the benefit vs the obvious downside.
Keywords: checkin-needed
Whiteboard: [c-n: 866ef7425e08 to m-a]
Comment 20•13 years ago
|
||
Comment on attachment 607478 [details] [diff] [review]
(Av1) dom-level*-*/DOMTestCase.js: Remove sayrer's override of SimpleTest._logResult()
[Backed out: Comment 21]
The overrides were using todoTests objects that define special test exclusions. SimpleTest doesn't know about these, and we ended up causing Bug 739927. We should backout this imo, until those specific instances are fixed. Before making impossible to star m3 failures, not after.
Attachment #607478 -
Flags: review-
Comment 21•13 years ago
|
||
Backed out for comment 20:
https://hg.mozilla.org/mozilla-central/rev/f0a104d6cc36
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 22•13 years ago
|
||
Comment on attachment 607478 [details] [diff] [review]
(Av1) dom-level*-*/DOMTestCase.js: Remove sayrer's override of SimpleTest._logResult()
[Backed out: Comment 21]
Let's move this part to bug 739927.
Attachment #607478 -
Attachment description: (Av1) dom-level*-*/DOMTestCase.js: Remove sayrer's override of SimpleTest._logResult()
[Checked in: Comment 17] → (Av1) dom-level*-*/DOMTestCase.js: Remove sayrer's override of SimpleTest._logResult()
[Backed out: Comment 21]
Attachment #607478 -
Attachment is obsolete: true
Assignee | ||
Comment 23•13 years ago
|
||
(In reply to Marco Bonardo [:mak] from comment #18)
> if this is going to backport bug 739927 to Aurora
(In reply to Marco Bonardo [:mak] from comment #19)
> clearing until clarified the benefit vs the obvious downside.
No, that is the other part.
(In reply to Marco Bonardo [:mak] from comment #20)
> The overrides were using todoTests objects that define special test
> exclusions. SimpleTest doesn't know about these, and we ended up causing Bug
> 739927.
I understood that but missed to notice bug 739927, because these "failures" are actually morphed into "todos" wrt result counts (in markTodos()) :-|
> We should backout this imo, until those specific instances are
> fixed. Before making impossible to star m3 failures, not after.
Agreed.
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [c-n: 866ef7425e08 to m-a]
Assignee | ||
Comment 24•13 years ago
|
||
V.Fixed, as still "succeeding" after bug 735805 landing.
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 25•13 years ago
|
||
Missed Aurora-13 :-| Giving up on older branches.
Keywords: checkin-needed
Whiteboard: [c-n: 866ef7425e08 to m-a]
You need to log in
before you can comment on or make changes to this bug.
Description
•