Closed Bug 1041535 Opened 5 years ago Closed 5 years ago

Add web-platform-tests to trychooser

Categories

(Release Engineering :: General, defect)

x86_64
Linux
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jgraham, Assigned: jgriffin)

References

Details

(Keywords: trychooser)

Attachments

(1 file, 2 obsolete files)

Once web-platform-tests are enabled on try, they should be added as an option in trychooser.
Keywords: trychooser
Can't land until web-platform-tests is enabled on Try, obviously. Fixesa bug where it was assumed that there was only one group of unittests, so checking any one would enable all groups.
Attachment #8475968 - Flags: review?(emorley)
Assignee: nobody → james
Status: NEW → ASSIGNED
Comment on attachment 8475968 [details] [diff] [review]
Add web-platform-tests to trychooser

Thank you :-)
Attachment #8475968 - Flags: review?(emorley) → review+
Oops, you're right, the first patch has the effect of disabling wpt-chunked on cedar except for linux64, which is enabled globally.  This adds it back.
Attachment #8478574 - Flags: review?(nthomas)
Assignee: james → jgriffin
Fix typo...
Attachment #8478575 - Flags: review?(nthomas)
Attachment #8478574 - Attachment is obsolete: true
Attachment #8478574 - Flags: review?(nthomas)
Comment on attachment 8478575 [details] [diff] [review]
Re-enable wpt-chunked on cedar for non-linux64 platforms,

Please put a comment on bug 1056973 to say the followup ended up here.
Attachment #8478575 - Flags: review?(nthomas) → review+
Comment on attachment 8478575 [details] [diff] [review]
Re-enable wpt-chunked on cedar for non-linux64 platforms,

Whoops, attached this to wrong bug, I'll put in on the right one.  :)
Attachment #8478575 - Attachment is obsolete: true
I'm not sure what's needed to update the website?
(In reply to James Graham [:jgraham] from comment #8)
> I'm not sure what's needed to update the website?

Another bug to deploy it, eg bug 1059236
Blocks: 1063613
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Component: Tools → General
You need to log in before you can comment on or make changes to this bug.