974358 - Add TBPL support for web-platform-tests

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect)

Description

[Ed Morley [:emorley]]

https://tbpl.mozilla.org/?tree=Cedar&jobname=web-platform-tests

eg:
Rev4 MacOSX Snow Leopard 10.6 cedar debug test web-platform-tests
Rev4 MacOSX Snow Leopard 10.6 cedar opt test web-platform-tests
Rev5 MacOSX Mavericks 10.9 cedar debug test web-platform-tests
Rev5 MacOSX Mavericks 10.9 cedar opt test web-platform-tests
Rev5 MacOSX Mountain Lion 10.8 cedar debug test web-platform-tests
Rev5 MacOSX Mountain Lion 10.8 cedar opt test web-platform-tests
Ubuntu ASAN VM 12.04 x64 cedar opt test web-platform-tests
Ubuntu VM 12.04 cedar debug test web-platform-tests
Ubuntu VM 12.04 cedar opt test web-platform-tests
Ubuntu VM 12.04 x64 cedar debug test web-platform-tests
Ubuntu VM 12.04 x64 cedar opt test web-platform-tests

James, do you have any preference as to the TBPL symbol used for this job? (see the help menu on TBPL for the current symbol list)

Comment 1

[James Graham [:jgraham]]

Would wpt work?

Comment 2

[Ed Morley [:emorley]]

Yeah that sounds great :-)

Comment 3

[Ed Morley [:emorley]]

Attachment: Add support for web-platform-tests

Comment 4

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8378272 [details] [diff] [review]
Add support for web-platform-tests

Review of attachment 8378272 [details] [diff] [review]:
-----------------------------------------------------------------

r+ with comments

::: js/Config.js
@@ +421,5 @@
>      "Gaia Integration Test" : "Gi",
>      "Gaia UI Test" : "Gu",
>      "Gaia Unit Test" : "G",
>      "XPCShellTest" : "X",
> +    "W3C Web Platform Tests" : "wpt",

At a minimum, we should probably be Wpt for consistency's sake with other top-level tests. But honestly, W isn't in use, so can we just go with that?

@@ +426,1 @@
>      "Android x86 Test Combos" : "Sets", 

Kill the trailing whitespace while you're here?

Comment 5

[Ed Morley [:emorley]]

Updated & pushed :-)

https://hg.mozilla.org/webtools/tbpl/rev/39431408001e

Comment 6

[Ryan VanderMeulen [:RyanVM]]

In production :)