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

[#12081] Modify accessibility tests workflow to rerun failed tests once #12272

Merged
merged 2 commits into from
Mar 28, 2023

Conversation

zhaojj2209
Copy link
Contributor

@zhaojj2209 zhaojj2209 commented Mar 27, 2023

Part of #12081

Accessibility tests occasionally fail due to Selenium instability, similar to E2E tests. From observing E2E tests, most tests pass within 2 tries, therefore the accessibility tests workflow has been modified to rerun failed tests once. This reduces work on the maintainers' end to rerun failed tests.

@zhaojj2209 zhaojj2209 added the s.ToReview The PR is waiting for review(s) label Mar 27, 2023
@zhaojj2209 zhaojj2209 added s.ToMerge The PR is approved by all reviewers including final reviewer; ready for merging and removed s.ToReview The PR is waiting for review(s) labels Mar 28, 2023
@zhaojj2209 zhaojj2209 merged commit 70b2ca9 into TEAMMATES:master Mar 28, 2023
@zhaojj2209 zhaojj2209 added this to the V8.26.0 milestone Apr 2, 2023
@zhaojj2209 zhaojj2209 added the c.DevOps Process-related or build-related improvement and addition label Apr 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c.DevOps Process-related or build-related improvement and addition s.ToMerge The PR is approved by all reviewers including final reviewer; ready for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants