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

Status: RESOLVED FIXED

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

Description

[Jim Mathies [:jimm]]

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

Comment 1

[Ed Morley [:emorley]]

Attachment: Add support for mochitest metro chrome;

Comment 2

[Jim Mathies [:jimm]]

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. :)

Comment 3

[Ed Morley [:emorley]]

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

Comment 4

[Ed Morley [:emorley]]

In production :-)

Comment 5

[Jim Mathies [:jimm]]

Thanks for the quick turn around!

Comment 6

[Jim Mathies [:jimm]]

wrong bug, this is for tinderbox push log.