Closed Bug 1549894 Opened 5 years ago Closed 5 years ago

mach addtest does the wrong thing for .js web platform tests

Categories

(Testing :: web-platform-tests, defect, P3)

Version 3
defect

Tracking

(firefox70 fixed)

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: bzbarsky, Assigned: jgraham)

Details

Attachments

(1 file)

I just ran:

  mach addtest testing/web-platform/tests/html/webappapis/dynamic-markup-insertion/opening-the-input-stream/parser-no-script-nesting.window.js

and it created the file, but filled it in with the HTML template for web platform tests, which isn't quite right in this case.

I probably shouldn't have used addtest for this at all, since .js wpts have no particular template to them, but it would be nice if addtest noticed and said something.

The priority flag is not set for this bug.
:jgraham, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(james)

This by default creates an empty js file, but does support the
--long-timeout flag.

Flags: needinfo?(james)
Priority: -- → P3
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/autoland/rev/adfc9e87dbcd
Support wpt .js tests in addtest, r=bgrins
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Assignee: nobody → james
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: