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

Setup visual and a11y tests #21

Merged
merged 1 commit into from Aug 1, 2022

Conversation

stefanteixeira
Copy link
Owner

@stefanteixeira stefanteixeira commented Jul 25, 2022

Description

Setting up a sample a11y test with axe-playwright and a couple of visual regression tests using happo-playwright integration. Also, configured a CI job for visual regression tests and enabled Happo app in the repository.

How to test

  • yarn install
  • Check a11y test in tests/e2e/create-user.e2e.test.js (it's skipped as it will always fail, since the component has a11y issues)
  • To run visual regression tests: yarn test:visual (you will need an account on Happo to run it locally)
  • Check Happo report with added screenshots

Checklist

  • Branch name follows the pattern: concise-feature-description
  • PR has a descriptive name
  • PR branch is up-to-date with main (if not, rebase it)
  • Double-check the quality of commit messages

@stefanteixeira stefanteixeira self-assigned this Jul 25, 2022
@stefanteixeira stefanteixeira force-pushed the setup-visual-regression-and-a11y-tests branch 2 times, most recently from 3dd71f0 to 9218239 Compare July 25, 2022 23:54
@stefanteixeira stefanteixeira marked this pull request as ready for review July 25, 2022 23:55
@stefanteixeira stefanteixeira force-pushed the setup-visual-regression-and-a11y-tests branch 2 times, most recently from 4f44699 to de9cd04 Compare July 28, 2022 01:04
@uLucasFraga
Copy link
Collaborator

Severest isn't ready for Happo haha

@uLucasFraga uLucasFraga added the ready to review This PR is ready to be reviewed label Jul 28, 2022
@stefanteixeira
Copy link
Owner Author

@uLucasFraga indeed, Safari is a pain 😅
The idea is to serve mostly as an example, to demonstrate the usage of Happo-Playwright integration.

@stefanteixeira stefanteixeira force-pushed the setup-visual-regression-and-a11y-tests branch from de9cd04 to 6e55cc9 Compare July 29, 2022 11:48
@stefanteixeira stefanteixeira temporarily deployed to github-pages July 29, 2022 11:49 Inactive
@stefanteixeira stefanteixeira force-pushed the setup-visual-regression-and-a11y-tests branch from 6e55cc9 to e96a03d Compare July 30, 2022 15:12
@stefanteixeira stefanteixeira temporarily deployed to github-pages July 30, 2022 15:13 Inactive
@stefanteixeira stefanteixeira force-pushed the setup-visual-regression-and-a11y-tests branch from e96a03d to 910642e Compare July 30, 2022 15:15
@stefanteixeira stefanteixeira temporarily deployed to github-pages July 30, 2022 15:16 Inactive
@uLucasFraga
Copy link
Collaborator

LGTM! <3

@stefanteixeira stefanteixeira force-pushed the setup-visual-regression-and-a11y-tests branch from 910642e to 0ccd98a Compare August 1, 2022 23:25
@stefanteixeira stefanteixeira merged commit b58c30e into main Aug 1, 2022
@stefanteixeira stefanteixeira deleted the setup-visual-regression-and-a11y-tests branch August 1, 2022 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to review This PR is ready to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setup axe-playwright to run a11y tests Add Visual Regression Tests for Login and Navigation
2 participants