Closed Bug 1275233 Opened 3 years ago Closed 3 years ago

Web-platform-test form-validation-reportValidity.html use skip-if which is never true


(Core :: DOM: Core & HTML, defect)

Not set



Tracking Status
firefox49 --- fixed


(Reporter: jdai, Assigned: jdai)



(Whiteboard: btpp-active)


(1 file, 1 obsolete file)

Web-platform-test shouldn't use skip-if. It cause form-validation-reportValidity web-platform-test has been skipped in all platform.
I guess we need to fix wpt annotations so that they don't let one to use things which it can't recognize. 
The current setup feels really error prone, and not something a review can easily catch.
Flags: needinfo?(james)
aha, I see, we use different syntax for wpt.
skip-if = (os == 'mac') and debug:
should be something like
if os == "mac":

thought, I don't know how to express 'debug' there.
if os == "mac" and debug:

maybe? At least 'debug' is used elsewhere.
Yes. I don't think this is too weird; it's consistent with the rest of web-platform-tests and it would be super-weird to have to put all the expectation values on the left hand side of if expressions like error-if: and crash-if:.

We could maybe lint for use of skip-if on the rhs, but it isn't something I have seen before so it's at least not obviously a common mistake.
Flags: needinfo?(james)
Whiteboard: btpp-active
Comment on attachment 8756671 [details] [diff] [review]
Bug 1275233 - Web-platform-test shouldn't use skip-if.

Drive-by review to help smaug's queue.
Attachment #8756671 - Flags: review+
Attachment #8756671 - Flags: review?(bugs)
Rename reviewer name.
Attachment #8756671 - Attachment is obsolete: true
Attachment #8756763 - Flags: review+
Keywords: checkin-needed
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.