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

[Task]: Add a pa11y ci step to spin up the API so it shows search results #2147

Open
acouch opened this issue Sep 17, 2024 · 1 comment
Open
Assignees

Comments

@acouch
Copy link
Collaborator

acouch commented Sep 17, 2024


Migrated from navapbc#74
Originally created by @rylew1 on Fri, 07 Jun 2024 16:46:34 GMT


blocked by #40

Summary

When pa11y tests the search page, it is doing so against just the frontend, and therefore the search page will not have any search results loaded (it will be just the red error alert) - but we want to a11y test against the actual search result list content.

ci-frontend-a11y.yml should add a step to load the API (see the ci-frontend-e2e.yml file for how to do this - it should be the same exact process)

Acceptance criteria

  • pa11y runs against loaded search results
  • pa11y tests pass
@acouch
Copy link
Collaborator Author

acouch commented Sep 17, 2024


Migrated from navapbc#74 (comment)
Originally created by @rylew1 on Fri, 07 Jun 2024 16:49:31 GMT


I think we actually want to test against both the red error state, and valid search results state.

acouch pushed a commit to navapbc/simpler-grants-gov that referenced this issue Sep 18, 2024
Fixes #{74}

> Added API setup to PA11y
> Updated Pa11y runtime to enable FF for search

> FF requirement was discovered with Ryan while exploring feature. Will
enable the search page for proper testing
> • svg elements with an img role have an alternative text
(https://dequeuniversity.com/rules/axe/4.2/svg-img-alt?application=axeAPI)

   (html)

   <html lang="en"><head><meta charset="utf-8"><me...</html>

• svg elements with an img role have an alternative text
(https://dequeuniversity.com/rules/axe/4.2/svg-img-alt?application=axeAPI)

   (html)

   <html lang="en"><head><meta charset="utf-8"><me...</html>
>Two new errors found now that API results are loading.

![desktop-main-view](https://github.com/navapbc/simpler-grants-gov/assets/29316916/f13859bd-87a7-466d-a20b-e86a9dbe71e5)
acouch pushed a commit to navapbc/simpler-grants-gov that referenced this issue Sep 18, 2024
Fixes #{74}

> Added API setup to PA11y
> Updated Pa11y runtime to enable FF for search

> FF requirement was discovered with Ryan while exploring feature. Will
enable the search page for proper testing
> • svg elements with an img role have an alternative text
(https://dequeuniversity.com/rules/axe/4.2/svg-img-alt?application=axeAPI)

   (html)

   <html lang="en"><head><meta charset="utf-8"><me...</html>

• svg elements with an img role have an alternative text
(https://dequeuniversity.com/rules/axe/4.2/svg-img-alt?application=axeAPI)

   (html)

   <html lang="en"><head><meta charset="utf-8"><me...</html>
>Two new errors found now that API results are loading.

![desktop-main-view](https://github.com/navapbc/simpler-grants-gov/assets/29316916/f13859bd-87a7-466d-a20b-e86a9dbe71e5)
acouch pushed a commit that referenced this issue Sep 18, 2024
Fixes #{74}

> Added API setup to PA11y
> Updated Pa11y runtime to enable FF for search

> FF requirement was discovered with Ryan while exploring feature. Will
enable the search page for proper testing
> • svg elements with an img role have an alternative text
(https://dequeuniversity.com/rules/axe/4.2/svg-img-alt?application=axeAPI)

   (html)

   <html lang="en"><head><meta charset="utf-8"><me...</html>

• svg elements with an img role have an alternative text
(https://dequeuniversity.com/rules/axe/4.2/svg-img-alt?application=axeAPI)

   (html)

   <html lang="en"><head><meta charset="utf-8"><me...</html>
>Two new errors found now that API results are loading.

![desktop-main-view](https://github.com/navapbc/simpler-grants-gov/assets/29316916/f13859bd-87a7-466d-a20b-e86a9dbe71e5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Icebox
Development

No branches or pull requests

2 participants