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

Moving report only smoke test to a11y namespace. #6372

Merged
merged 1 commit into from
Nov 17, 2022
Merged

Moving report only smoke test to a11y namespace. #6372

merged 1 commit into from
Nov 17, 2022

Conversation

1Copenut
Copy link
Contributor

@1Copenut 1Copenut commented Nov 16, 2022

Summary

While bumping Node in Docker, I realized I had this a11y smoke test still running as part of our functional test batch. Renaming it and moving it to run with a11y tests when they're scheduled.

QA

  • Ran Cypress a11y tests headlessly on local machine

Screen Shot 2022-11-16 at 3 38 26 PM

General checklist

  • Checked in Chrome, Safari, Edge, and Firefox
  • Added or updated jest and cypress tests
  • Checked for breaking changes and labeled appropriately
  • Checked for accessibility including keyboard-only and screenreader modes

@kibanamachine
Copy link

Preview documentation changes for this PR: https://eui.elastic.co/pr_6372/

@1Copenut 1Copenut merged commit 1a4b156 into elastic:main Nov 17, 2022
@1Copenut 1Copenut deleted the feature/move-a11y-reporter branch November 17, 2022 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants