[SPIKE] Check all pages' accessibility

[domoscargin]

Popped this together to check how long it'd take to run accessibility tests on all our pages. Just a hacky function to get all directories with an index.html file, then running an axe validation test on each of those.

Takes around 80 seconds on my machine, but open for ways to speed it up.

There's a few failures flagged, mostly to do with our heading order.

If taken forward, the code would need to be cleaned up and optimised.

Edit: looks like the tests took about 2m 45s.

[netlify]

✅ You can preview this change here:

Name	Link
🔨 Latest commit	9199c96
🔍 Latest deploy log	https://app.netlify.com/sites/govuk-design-system-preview/deploys/65e5ad0d8f33cf00083334ec
😎 Deploy Preview	https://deploy-preview-2957--govuk-design-system-preview.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[domoscargin]

Had a brief look at it.concurrent.each, but at first glance this doesn't play nicely with Puppeteer. Maybe something with promise.all?

[colinrotherham]

Hey @domoscargin I've rebased this one, seems a shame to leave it

I've brought over the Axe rules from alphagov/govuk-frontend@ebb9e4e added in:

• Switch Accessibility tests to @axe-core/puppeteer govuk-frontend#3522

Locally I'm seeing Puppeteer (with Axe) disconnect after a while but let's hope it's resolved by:

• axe-core-npm v4.8.0 Release dequelabs/axe-core-npm#827
• puppeteer: Investigate if Puppeteer headless: new and unloaded iframes are still an issue dequelabs/axe-core-npm#828

[colinrotherham]

@domoscargin Just rebased this to see if the @axe-core/puppeteer update fixed the disconnect issue

Looking good locally

[domoscargin]

Darn, failed here.

[colinrotherham]

Darn, failed here.

How frustrating 😭

Worth trying a separate await browser.newPage() per test (just pushed)

Otherwise, let's check again when more next major updates come out 🤦‍♂️

[colinrotherham]

Another Puppeteer update so rebasing this again

[colinrotherham]

Had some more Axe and Puppeteer updates in #3468 so giving this one a rebase

[colinrotherham]

@domoscargin Looks like it's all working 🙌

Status check failures are now accessibility report issues, as intended

[domoscargin]

I've reduced failures on this PR down to zero BUT only by disabling rules across the board.

Just wanted to get this down to zero and allow us to think about how to address each of the special cases (which are all documented in __tests__/accessibility-audit.test.js).

The rules that cause failures unless disabled:

[
  'region',
  'color-contrast-enhanced',
  'aria-allowed-attr',
  'target-size',
  'aria-allowed-role'
]