Closed Bug 865451 Opened 9 years ago Closed 9 years ago

Story - Add support for the new mochitest metro chrome test suite

Categories

(Tree Management Graveyard :: TBPL, defect, P2)

x86_64
Windows 8
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jimm, Assigned: emorley)

References

Details

(Whiteboard: feature=story c=testing u=developer p=0)

Attachments

(1 file)

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
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: nobody → emorley
Status: NEW → ASSIGNED
Attachment #741800 - Flags: review?(jmathies)
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+
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
Depends on: 865667
In production :-)
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Thanks for the quick turn around!
Blocks: metrov1it6
No longer blocks: metrov1backlog
Priority: -- → P2
Depends on: 881260
Depends on: 881104
Depends on: 880704
Depends on: 880703
Depends on: 880693
Depends on: 891892
Depends on: 884247
Depends on: 880745
Depends on: 880739
Depends on: 881463
Depends on: 895873
Depends on: 886280
Depends on: 896082
Depends on: 897175
Depends on: 899623
Depends on: 900641
wrong bug, this is for tinderbox push log.
No longer depends on: 880739, 880745, 884247, 886280, 891892, 895873, 896082, 897175, 900641
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.