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

Figure out how to get a better link URL for all accessibility tests in the WPT dashboard #44

Closed
cookiecrook opened this issue May 18, 2023 · 9 comments
Assignees

Comments

@cookiecrook
Copy link
Collaborator

Figure out how to get a better link URL for all accessibility tests than this:

https://wpt.fyi/results/?label=experimental&label=master&aligned&q=path%3A%2Faccessibility%20or%20path%3A%2Faccname%20or%20path%3A%2Fhtml-aam%20or%20path%3A%2Fwai-aria%20or%20path%3A%2Fsvg-aam%20or%20path%3A%2Fgraphics-aria%20or%20path%3A%2Fgraphics-aam%20or%20path%3A%2Fdpub-aam%20or%20path%3A%2Fhtml%2Fdom%2Faria-element-reflection.html

I looked into WPT.fyi label searches, but as I understand it, we'd need to add the label manually to every test file after it populated to the dashboard. I was hoping to do something by top-level directory.

@zcorpan
Copy link
Member

zcorpan commented May 29, 2023

@zcorpan
Copy link
Member

zcorpan commented May 29, 2023

Note that PRs to that repo are merged by a bot unless the PR is "draft" or has a "do not merge yet" label

@cookiecrook cookiecrook self-assigned this May 30, 2023
@cookiecrook
Copy link
Collaborator Author

thanks

@cookiecrook
Copy link
Collaborator Author

May be solved by #43

@foolip
Copy link
Member

foolip commented Jun 7, 2023

@chrishtr poked me about this and I filed web-platform-tests/wpt.fyi#3351. But I didn't see web-platform-tests/wpt-metadata#4297, which is doing exactly what I suggested as option 1 👍

@cookiecrook
Copy link
Collaborator Author

Currently blocked by:

@cookiecrook
Copy link
Collaborator Author

cookiecrook commented Aug 14, 2023

Unblocked soon I think. See update in #43. This issue #44 might be superseded by #43.

@cookiecrook
Copy link
Collaborator Author

cookiecrook commented Aug 21, 2023

Once the patch for #43 lands, these should be available as:

General label:
Accessibility: https://wpt.fyi/results/?label=master&label=experimental&aligned&q=label%3Aaccessibility

Or more specifically:
ARIA: https://wpt.fyi/results/?label=master&label=experimental&aligned&q=label%3Aaria
AccName: https://wpt.fyi/results/?label=master&label=experimental&aligned&q=label%3Aaccname
HTML-AAM: https://wpt.fyi/results/?label=master&label=experimental&aligned&q=label%3Ahtml-aam

We also should be able to get pretty granular about the per-file subsections of spec testing, for example... We could associate a specific label with the invalid roles related tests, or some subsection of the AccName name computation algorithm.

@cookiecrook
Copy link
Collaborator Author

cookiecrook commented Aug 21, 2023

Links above are live. Leaving this open until we discuss with ARIA WG (about what other labels they want) and the Accessibility Interop group, as an FYI to the other members, and an acknowledgement and appreciation of Kyle Ju's work.

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

3 participants