Skip to content

add axe-playwright-python to testing #577

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

Open
kjaymiller opened this issue Nov 15, 2024 · 7 comments
Open

add axe-playwright-python to testing #577

kjaymiller opened this issue Nov 15, 2024 · 7 comments
Assignees

Comments

@kjaymiller
Copy link
Contributor

add accessibility testing to our GitHub actions by using axe-playwright-python.

https://pamelafox.github.io/axe-playwright-python/

@TimOsahenru TimOsahenru self-assigned this Dec 13, 2024
@TimOsahenru
Copy link
Contributor

I encountered the following error when trying to import Axe as shown in the documentation:
image

I used from axe_core_python.sync_playwright import Axe, but the error makes me wonder if I need to install axe-core-python, even though the documentation doesn't mention it.

Interestingly, everything works fine when I use from axe_playwright_python.async_playwright import Axe, but I can't find any reference to this import path in the documentation.

Hey @kjaymiller, do you have any insights or suggestions about this?

@kjaymiller
Copy link
Contributor Author

Looking at how I use it with my personal site... I also use from axe_playwright_python

I would file an issue with with the package, it may be a documentation issue.

@TimOsahenru
Copy link
Contributor

@kjaymiller any updates on this ? Just checking

@kjaymiller
Copy link
Contributor Author

I thought the last comment was the resolution... (Use from axe_playwright_python) Were you able to continue on from there?

@TimOsahenru
Copy link
Contributor

Hello @kjaymiller I have been able to run the axe runner, detected some accessibility violations, currently fixing them.

@kjaymiller
Copy link
Contributor Author

Submit the PR and then we can address the violations in a separate set of PRs

@TimOsahenru
Copy link
Contributor

Submit the PR and then we can address the violations in a separate set of PRs

alright cool cool. fyi I had to also install this package pip install axe-core-python(which isn't stated in the documentation) to be able to use the Axe runner

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

No branches or pull requests

2 participants