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

RESOLVED FIXED

Status

defect
P2
normal
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: jimm, Assigned: emorley)

Tracking

Trunk
x86_64
Windows 8
Dependency tree / graph

Details

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

Attachments

(1 attachment)

Reporter

Description

6 years ago
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

6 years ago
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

Updated

6 years ago
Assignee: nobody → emorley
Status: NEW → ASSIGNED
Assignee

Updated

6 years ago
Attachment #741800 - Flags: review?(jmathies)
Reporter

Comment 2

6 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

6 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

Updated

6 years ago
Depends on: 865667
Assignee

Comment 4

6 years ago
In production :-)
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Reporter

Comment 5

6 years ago
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
Reporter

Comment 6

6 years ago
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.