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

feat(tests): run axe-core tests #21037

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

feat(tests): run axe-core tests #21037

wants to merge 22 commits into from

Conversation

revolunet
Copy link
Member

Test de axe-core pour les PRs

Il faut specifier les urls à la main dans le workflow sinon on a trop de pages je pense

@revolunet revolunet requested a review from astranchet February 10, 2025 13:00
@astranchet
Copy link
Contributor

Les axes d'améliorations que je vois :

  • Tester uniquement les pages modifiées
  • Ne pas tester les pages avec des erreurs connues
  • Ne même pas afficher " 0 violations found!" et n'afficher un message que s'il y a un problème

@revolunet revolunet linked an issue Feb 10, 2025 that may be closed by this pull request
@revolunet
Copy link
Member Author

Tester uniquement les pages modifiées

Ça ne me parait pas possible en l'état de prévoir quelles pages HTML vont être impactées :/

Ne pas tester les pages avec des erreurs connues

Il faut les définir à la main dans le workflow, j'ai pas mieux :/

Ne même pas afficher " 0 violations found!" et n'afficher un message que s'il y a un problème

Ca doit être mieux pour ça

Copy link

♿️ Axe report

Audited website :

Running axe-core 4.10.2 in chrome-headless

Testing http://127.0.0.1:8080/stats/index.html ... please wait, this may take a minute.

  Violation of "aria-allowed-attr" with 9 occurrences!
    Ensure an element's role supports its ARIA attributes. Correct invalid elements at:
     - .h300:nth-child(6) #mantine-3alkv79w3-target
     - .h300:nth-child(11) #mantine-t2vcnqmcd-target
     - .fr-mb-8w:nth-child(15) > .h300 #mantine-lq34a0kzb-target
     - .fr-mb-8w:nth-child(16) > .h300 #mantine-v0cvrxf03-target
     - .fr-mb-8w:nth-child(17) > .h300 #mantine-w6cdenran-target
     - .fr-mb-4w > .h300 #mantine-v9xkeaf1x-target
     - .h800 #mantine-7zxsti8pg-target
     - .fr-col-md-6.fr-col-sm-6.fr-col-lg-6:nth-child(1) > .h300 #mantine-kqpg6nfag-target
     - .fr-col-md-6.fr-col-sm-6.fr-col-lg-6:nth-child(2) > .h300 #mantine-i4eypxsmz-target
    For details, see: https://dequeuniversity.com/rules/axe/4.10/aria-allowed-attr

  Violation of "color-contrast" with 11 occurrences!
    Ensure the contrast between foreground and background colors meets WCAG 2 AA minimum contrast ratio thresholds. Correct invalid elements at:
     - .h300:nth-child(6) .emotion-12uxj9b
     - .h300:nth-child(11) .emotion-12uxj9b
     - .fr-mb-8w:nth-child(15) > .h300 .emotion-12uxj9b
     - .fr-mb-8w:nth-child(16) > .h300 .emotion-12uxj9b
     - .fr-mb-8w:nth-child(17) > .h300 .emotion-12uxj9b
     - .fr-mb-4w > .h300 .emotion-12uxj9b
     - .h300:nth-child(23) h2
     - .h300:nth-child(23) .emotion-12uxj9b
     - .h800 .emotion-12uxj9b
     - .fr-col-md-6.fr-col-sm-6.fr-col-lg-6:nth-child(1) > .h300 .emotion-12uxj9b
     - .fr-col-md-6.fr-col-sm-6.fr-col-lg-6:nth-child(2) > .h300 .emotion-12uxj9b
    For details, see: https://dequeuniversity.com/rules/axe/4.10/color-contrast

  Violation of "landmark-unique" with 2 occurrences!
    Ensure landmarks are unique. Correct invalid elements at:
     - #contenu
     - .h300:nth-child(6) footer
    For details, see: https://dequeuniversity.com/rules/axe/4.10/landmark-unique

22 Accessibility issues detected.

Testing complete of 13 pages

Please note that only 20% to 50% of all accessibility issues can automatically be detected. 
Manual testing is always required. For more information see:
https://dequeuniversity.com/class/testing

end

(node:1699) [DEP0066] DeprecationWarning: OutgoingMessage.prototype._headers is deprecated
(Use  to show where the warning was created)
Killed


This report has been generated using @axe-core/cli.

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.

Ajouter des tests automatiques d'accessibilité
2 participants