Ensures ARIA attributes are allowed for an element's role violation on global loading indicator

[bhavyarm]

Kibana version: main

Browser version: chrome latest

Browser OS version: OS X

Describe the bug: Dashboard full screen a11y test is failing because of the way we are using globalLoadingIndicator is violating the rule "Ensures ARIA attributes are allowed for an element's role"

Accessibility Tests
test/accessibility/apps/dashboard.ts

Buildkite Job
https://buildkite.com/elastic/kibana-pull-request/builds/115148#01870fc1-8e11-4537-b8c8-2456c4c14caf

Error: a11y report:

VIOLATION
[aria-allowed-attr]: Ensures ARIA attributes are allowed for an element's role
Impact: serious
Help: https://dequeuniversity.com/rules/axe/4.6/aria-allowed-attr?application=axeAPI
Elements:
-

at AccessibilityService.assertValidAxeReport (a11y.ts:75:13)
at AccessibilityService.testAppSnapshot (a11y.ts:48:10)
at runMicrotasks ()
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at Context. (dashboard.ts:112:7)
at Object.apply (wrap_function.js:73:16)

\

[elasticmachine]

Pinging @elastic/kibana-presentation (Team:Presentation)

[elasticmachine]

Pinging @elastic/kibana-accessibility (Project:Accessibility)

[Heenawter]

Tagging @elastic/appex-sharedux, since they own the LoadingIndicator component - looking into it, and it seems like the EuiProgress bar doesn't expect to have an aria-label attribute. Tested this on the EUI docs by adding an aria-label to the top example here and ran an Axe scan - before adding it, there were zero Axe failures; after, there was one:

[1Copenut]

@Heenawter Thank you for testing this and following up with a screenshot. Adding an aria-label to the indeterminate progress bar throws because of the DIV generic role.

For the moment, you could remove the aria-label and aria-hidden="false" declarations to pass the validation, or if you have a max and value data you could pass in as props, treat it as a progress with values.

I think there's an opportunity to improve our default experience for assistive technology, but will take some research into the history and UX thinking.

[bhavyarm]

Thanks for the investigation Hannah <3 @Heenawter