-
Notifications
You must be signed in to change notification settings - Fork 769
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: Update ACT test cases to run from W3C site #3637
Conversation
@@ -0,0 +1,5 @@ | |||
{ | |||
"id": "5c01ea", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: how do we want to name these files? We don't appear to have a naming convention for axe-core, and I'm not sure where the aria-props-allowed
name comes from. Is that what it's called in ACT (but not listed in the rule itself)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was sort of just winging it. ACT rules do have a file name. I could copy those I suppose. Here's the list of them: https://github.com/act-rules/act-rules.github.io/tree/develop/_rules
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See question above
const shouldRun = testcase.relativePath.match(/\.(xhtml|html?)$/); | ||
(shouldRun ? it : xit)(testcase.testcaseTitle, async () => { | ||
await driver.get(`${addr}/${testcase.relativePath}`); | ||
const builder = new AxeBuilder(driver, axeSource); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, I like this way of doing the tests
Co-authored-by: Steven Lambert <[email protected]>
Closes issue: #3499