Closed
Bug 865451
Opened 12 years ago
Closed 12 years ago
Story - Add support for the new mochitest metro chrome test suite
Categories
(Tree Management Graveyard :: TBPL, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jimm, Assigned: emorley)
References
Details
(Whiteboard: feature=story c=testing u=developer p=0)
Attachments
(1 file)
(deleted),
patch
|
jimm
:
review+
|
Details | Diff | Splinter Review |
Metro browser mochitests should be running on inbound, try and mc fairly soon. We'll need to add a new moniker for these tests ('mc' for metro chrome?), which should initially be hidden so we can sort out random orange issues. The tests that run are browser chrome tests that run in the metro browser.
We're currently working out some config issues on Cedar in bug 864418. Initial log results can be viewed here -
https://tbpl.mozilla.org/?tree=Cedar&showall=1&jobname=WINNT%206.2%20cedar%20opt%20test%20metro-immersive
Reporter | ||
Updated•12 years ago
|
Blocks: metro-testing
Updated•12 years ago
|
Blocks: metrov1backlog
Summary: Add support for the new mochitest metro chrome test suite → Story - Add support for the new mochitest metro chrome test suite
Whiteboard: feature=story c=testing u=developer p=0
Assignee | ||
Comment 1•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → emorley
Status: NEW → ASSIGNED
Assignee | ||
Updated•12 years ago
|
Attachment #741800 -
Flags: review?(jmathies)
Reporter | ||
Comment 2•12 years ago
|
||
Comment on attachment 741800 [details] [diff] [review]
Add support for mochitest metro chrome;
Review of attachment 741800 [details] [diff] [review]:
-----------------------------------------------------------------
::: js/Data.js
@@ +563,5 @@
> /dxr/i.test(name) ? "DXR Index Build" :
> /build$/i.test(name) ? "Build" :
> // ** Unit tests **
> /mochitest-other/i.test(name) ? "Mochitest Other" :
> + /metro-immersive/i.test(name) ? "Mochitest Metro Browser Chrome" :
The naming you've used seems fine to me. I can't really say if the above line is correct or not as I have no clue what it does. :)
Attachment #741800 -
Flags: review?(jmathies) → review+
Assignee | ||
Comment 3•12 years ago
|
||
Sorry I should have said the r? was more for the naming (rest is just copy pasta of the horrid horrid nested ternaries).
https://hg.mozilla.org/webtools/tbpl/rev/ba7fe899ad55
Assignee | ||
Comment 4•12 years ago
|
||
In production :-)
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 5•12 years ago
|
||
Thanks for the quick turn around!
Updated•12 years ago
|
Updated•12 years ago
|
Priority: -- → P2
Reporter | ||
Comment 6•11 years ago
|
||
wrong bug, this is for tinderbox push log.
Updated•10 years ago
|
Product: Webtools → Tree Management
Updated•10 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•