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

Axe #74

Closed
wants to merge 7 commits into from
Closed

Axe #74

wants to merge 7 commits into from

Conversation

gabalafou
Copy link
Collaborator

No description provided.

@gabalafou
Copy link
Collaborator Author

gabalafou commented Feb 7, 2022

@tonyfast when you get a chance could you try checking out and running this PR on your local machine?

I believe that the only prerequisites (beyond basic OS / POSIX stuff) are:

  • mamba
  • git
  • nox
  • certain system-level Playwright dependencies. I don't know exactly what these are but I believe you should be okay here if you have Chrome installed on your computer.

(Neither Python nor NodeJS are in that list because I think you get Python when you create a conda environment or when you install nox into an environment, and I guess NodeJS comes along with yarn or playwright? I'm not quite sure. All I know is that when testing this in an isolated Docker environment, I did not have to explicitly install them and I don't think the image I tested with came bundled with Python or Node.)

So if you have the above on your system, I believe you should be able to:

cd tests/retrolab
nox -s test

and then it should launch your browser and you should be able to see the failing tests and when you click into a test, you should be able to download the JSON containing the axe failures for that test.

@gabalafou
Copy link
Collaborator Author

Superseded by #83
(probably. can always reopen this PR if needed)

@gabalafou gabalafou closed this May 11, 2022
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