Closed
Bug 830523
Opened 13 years ago
Closed 13 years ago
Excludetest list of all dom test failures for b2g.json
Categories
(Testing :: Mochitest, defect)
Tracking
(firefox20 wontfix, firefox21 fixed, b2g18 affected, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 wontfix)
RESOLVED
FIXED
mozilla21
People
(Reporter: martijn.martijn, Assigned: martijn.martijn)
References
Details
Attachments
(3 files, 1 obsolete file)
I've attached a b2g.json file that made a succesful mochitest run inside the dom/ directory on the b2g emulator.vc
I added some complete directories, because there were too many failures in there.
Comment 1•13 years ago
|
||
pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=f93a2a5a073f
Martijn, can you clarify on the title of the bug, layout is a different bug right?
Comment 3•13 years ago
|
||
Martijn, there were three failures reported in this try run. Can you exclude those tests also in b2g.json, and then we'll push to try again?
| Assignee | ||
Comment 4•13 years ago
|
||
Sorry for the delay, I added those tests to the exclude list. I need to retest locally, still.
Attachment #701997 -
Attachment is obsolete: true
| Assignee | ||
Comment 5•13 years ago
|
||
Ok, there were a bunch of extra failures in tests (they might have been added). This should provide a clean run on tryserver.
| Assignee | ||
Comment 6•13 years ago
|
||
Ugh, sorry, only noticed now that I messed up the summary. This has nothing to do with layout.
Summary: Excludetest list of all dom test failures in layout for b2g.json → Excludetest list of all dom test failures for b2g.json
Comment 7•13 years ago
|
||
Pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=b36156ecc9ed
| Assignee | ||
Comment 8•13 years ago
|
||
Ok, the try-run is all green!
| Assignee | ||
Comment 9•13 years ago
|
||
| Assignee | ||
Comment 10•13 years ago
|
||
I've run all the layout and dom tests again with that patch and it is all green on my machine.
Comment 11•13 years ago
|
||
Final version pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=b7a3febe428a
| Assignee | ||
Comment 12•13 years ago
|
||
All green!
Comment 13•13 years ago
|
||
Assignee: nobody → martijn.martijn
Target Milestone: --- → mozilla21
Comment 14•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Comment 17•13 years ago
|
||
Comment on attachment 710138 [details] [diff] [review]
patch
[Approval Request Comment]
Bug caused by (feature/regressing bug #):
User impact if declined:
Testing completed:
Risk to taking this patch (and alternatives if risky):
This patch enables a larger set of mochitests to be run on mozilla-b2g18. These are already running on mozilla-central. No user or partner impact.
String or UUID changes made by this patch:
Attachment #710138 -
Flags: review+
Attachment #710138 -
Flags: approval-mozilla-b2g18?
Comment 18•13 years ago
|
||
Comment on attachment 710138 [details] [diff] [review]
patch
Approving for v1-train uplift - fwiw you can uplift tests with a=test-only if they truly are test-only patches.
Attachment #710138 -
Flags: approval-mozilla-b2g18? → approval-mozilla-b2g18+
Updated•13 years ago
|
Comment 19•13 years ago
|
||
Comment 20•13 years ago
|
||
Updated•13 years ago
|
Attachment #710138 -
Flags: approval-mozilla-b2g18+
You need to log in
before you can comment on or make changes to this bug.
Description
•