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: catch a11y errors and ignore them for now. #30505

Merged
merged 4 commits into from
Nov 6, 2024

Conversation

jennifer-shehane
Copy link
Member

@jennifer-shehane jennifer-shehane commented Oct 31, 2024

Additional details

We frequently run into a situation where there are multiple runs attributed when we 'rerun from failures' and the accessibility results errors - we want to just ignore errors thrown from this since there's no action we can take without this being supported in Cypress

Steps to test

I tried to rerun this a few times and can't get it to fail / rerun properly to trigger this error, so hopefully this doesn't introduce a problem.

How has the user experience changed?

PR Tasks

@jennifer-shehane jennifer-shehane self-assigned this Oct 31, 2024
Copy link

cypress bot commented Oct 31, 2024

cypress    Run #58077

Run Properties:  status check passed Passed #58077  •  git commit 1f052676e1: Revert "make a test fail"
Project cypress
Branch Review ignore-a11y-results-errors
Run status status check passed Passed #58077
Run duration 22m 29s
Commit git commit 1f052676e1: Revert "make a test fail"
Committer Jennifer Shehane
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 4
Tests that did not run due to a developer annotating a test with .skip  Pending 1326
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 29395
View all changes introduced in this branch ↗︎
UI Coverage  46.24%
  Untested elements 188  
  Tested elements 166  
Accessibility  92.55%
  Failed rules  3 critical   8 serious   2 moderate   2 minor
  Failed elements 905  

@jennifer-shehane jennifer-shehane merged commit 74b9fa3 into develop Nov 6, 2024
86 of 87 checks passed
@jennifer-shehane jennifer-shehane deleted the ignore-a11y-results-errors branch November 6, 2024 17:02
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