Skip to content
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 tests #4233

Merged
merged 1 commit into from
Nov 3, 2023
Merged

chore: update ACT tests #4233

merged 1 commit into from
Nov 3, 2023

Conversation

WilcoFiers
Copy link
Contributor

@WilcoFiers WilcoFiers commented Nov 2, 2023

Updated to the latest ACT tests (with some notes on things that are no longer consistent). This should prevent nightly from failing.

@WilcoFiers WilcoFiers requested a review from a team as a code owner November 2, 2023 17:57
axeRules: ['avoid-inline-spacing']
axeRules: ['avoid-inline-spacing'],
// See: https://github.com/dequelabs/axe-core/issues/4232
skipTests: [
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heads up, there's a problem in the testcases.json file. Due to a mistake all but these three test cases are missing for this rule. I'm working to have that fixed. I don't know if the fix of that is going to reveal new issues here. We'll see.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there an issue open for the problem? Would hate for it to slip through the cracks.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

axeRules: ['avoid-inline-spacing']
axeRules: ['avoid-inline-spacing'],
// See: https://github.com/dequelabs/axe-core/issues/4232
skipTests: [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there an issue open for the problem? Would hate for it to slip through the cracks.

@WilcoFiers WilcoFiers merged commit 853a8ea into develop Nov 3, 2023
18 checks passed
@WilcoFiers WilcoFiers deleted the act-failings branch November 3, 2023 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants