Closed Bug 558610 Opened 14 years ago Closed 14 years ago

mochitests-2: test_Prototype.html fails testElementReplace, testGetValue and testSetValue. And these failures are not _counted_

Categories

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

defect
Not set
major

Tracking

()

RESOLVED FIXED
mozilla1.9.3a5

People

(Reporter: sgautherie, Assigned: MatsPalmgren_bugz)

References

()

Details

(Keywords: regression, Whiteboard: [fixed by bug 558726])

Attachments

(1 obsolete file)

This could be intermittent, but it (started to) fail on the 3 platforms at the same time...

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1270941321.1270945345.22362.gz
Linux comm-central-trunk debug test mochitests-2/5 on 2010/04/10 16:15:21
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1270947405.1270950187.31638.gz
OS X 10.5 comm-central-trunk debug test mochitests-2/5 on 2010/04/10 17:56:45
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1270944164.1270946304.24412.gz
WINNT 5.2 comm-central-trunk debug test mochitests-2/5 on 2010/04/10 17:02:44
{
3041 ERROR TEST-UNEXPECTED-FAIL | /tests/dom/tests/mochitest/ajax/prototype/test_Prototype.html | testElementReplace - 6 assertions, 1 failures, 0 errors
3164 ERROR TEST-UNEXPECTED-FAIL | /tests/dom/tests/mochitest/ajax/prototype/test_Prototype.html | testGetValue - 3 assertions, 1 failures, 0 errors
3165 ERROR TEST-UNEXPECTED-FAIL | /tests/dom/tests/mochitest/ajax/prototype/test_Prototype.html | testSetValue - 12 assertions, 1 failures, 0 errors
}

The other odd part is that these errors (only) are not counted as failures:
{
8280 INFO Passed: 3567
8281 INFO Failed: 0
8282 INFO Todo:   435
}

Regression timeframe:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=aa360768a319&tochange=1a6aac2558e4
Optimized Firefox seems to have this bug too:

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1271031669.1271032176.17793.gz&fulltext=1
WINNT 5.2 mozilla-central opt test mochitests-2/5 on 2010/04/11 17:21:09
s: win32-slave25
No longer blocks: SmTestFail
blocking2.0: --- → ?
Summary: [(Debug) SeaMonkey] test_Prototype.html fails testElementReplace, testGetValue and testSetValue. And these failures are not _counted_ → test_Prototype.html fails testElementReplace, testGetValue and testSetValue. And these failures are not _counted_
Summary: test_Prototype.html fails testElementReplace, testGetValue and testSetValue. And these failures are not _counted_ → mochitests-2: test_Prototype.html fails testElementReplace, testGetValue and testSetValue. And these failures are not _counted_
Attached patch Count them [See comment 3] (obsolete) — Splinter Review
It looks like these failures aren't counted because the test uses its own copy of the test files.
Attachment #438495 - Flags: review?
Attachment #438495 - Flags: review? → review?(sayrer)
Comment on attachment 438495 [details] [diff] [review]
Count them
[See comment 3]


That's indeed the counting problem, which depends on bug 427500 and blocks bug 451287.
I may have not filed it yet as I'm desperately waiting for review of the former.

This is also not the only affected file: see at least
http://mxr.mozilla.org/mozilla-central/search?string=lib%2FSimpleTest.js&case=1&find=%2Fdom%2Ftests%2Fmochitest%2Fajax%2F

Could you file a separate bug on that issue?
Thanks.
Bisected the range from comment 0:

The first bad revision is:
changeset:   40657:2b560e956063
user:        Geoff Lankow <geoff@darktrojan.net>
date:        Sat Apr 10 16:05:58 2010 -0400
summary:     Bug 166235: moz-user-select = 'none' should prevent copying to the clipboard. r=mats
Blocks: 166235
The regression from bug 166235 has been fixed now:
http://hg.mozilla.org/mozilla-central/rev/81c83a98798d
Assignee: nobody → matspal
No longer blocks: 451287
Status: NEW → RESOLVED
blocking2.0: ? → ---
Closed: 14 years ago
Depends on: 451287, 558726
Flags: in-testsuite-
Resolution: --- → FIXED
Whiteboard: [fixed by bug 558726]
Target Milestone: --- → mozilla1.9.3a5
Attachment #438495 - Attachment description: Count them → Count them [See comment 3]
Attachment #438495 - Attachment is obsolete: true
Attachment #438495 - Flags: review?(sayrer)
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: