interop-accessibility icon indicating copy to clipboard operation
interop-accessibility copied to clipboard

Accessibility test standardization, improvements and documentation

Open rahimabdi opened this issue 2 years ago • 8 comments

Tracker:

  • [x] Share draft documentation with @boazsender @cookiecrook (done Apr 28 2024)
  • [ ] Determine publication platform (likely our GitHub Wiki page)
  • [ ] Determine/publish updates to related ARIA testing documentation
  • [ ] Determine if we need to merge this existing ARIA WPT documentation into the new document: https://github.com/w3c/aria/blob/main/documentation/wpt.md

@cookiecrook Consolidating general thoughts here on improving the quality/consistency of our WPT accessibility test files and providing supporting documentation for test authors (also to make it less daunting for WPT contributors). All feedback welcome! Happy to take this forward after you and other folks have provided input.

I know that detailed guidance is already captured here and this may be moved to web-platform-tests/interop-accessibility: Running the WPT Tests for ARIA and Related Specs.

  • Writing tests, guidance and implementation practices
    • Boilerplate comment header for all test files (e.g., "This file tests X...")
    • Providing documentation links to relevant parts of a spec being tested
    • Comment links to where relevant testing is located (and documentation scheme for referencing files in other locations)
    • Naming .html test files (usually after a spec's section)
    • Readability improvement guidance: e.g., alphabetization where possible, prescriptive order of element attributes (e.g., <div id="id" data-expectedtestname="someTest" data-expectedlabel="label">, spacing and indentation, etc.
    • Subtest names must be globally unique (the more descriptive the better)
    • id naming scheme for elements/containers (e.g., many elements with unique ids referenced via aria-labelledby)
    • Performing HTML validation
    • Using PR checks (primarily ensuring that lint is clean, removing whitespace)
    • PR commit message (description and co-author attribution)
  • Supporting documentation for test authors/maintainers
    • Overview of the WPT project
    • Understanding/viewing CI results
    • More on testharness.js
    • Index page with all a11y related tests (e.g., accname, wai-aria, core-arm, html-aam, svg-aam, etc.)
    • WPT Debugging Q&A and common issues (e.g., troubleshooting bad data* attribute on test, non-unique test name, improper use of aria-utils utility functions)
    • Explainer for aria-utils.js and what it does

rahimabdi avatar Nov 16 '23 21:11 rahimabdi

Generally all sounds good. I think some of the non-accessibility-specific stuff you mentioned above may already be documented in the main WPT project... Any accessibility-specific docs could go here, along with pointers to the main WPT docs where relevant.

cookiecrook avatar Nov 20 '23 06:11 cookiecrook

@rahimabdi @lolaodelola and I met and discussed a few additional ideas:

  • We talked about creating a guide for where to put tests
  • Create an index page of all the tests using the tags
  • Going back and tagging existing tests
  • Linking to these docs from web-platform-tests.org

We also talked about our multiple audiences

  • People talking about different accessibility tests (e.g. aria test writers)
  • people in other groups who want to test accessibility

boazsender avatar Jan 31 '24 17:01 boazsender

Adding Agenda+ for docs discussion in the May meeting.

cookiecrook avatar Apr 02 '24 21:04 cookiecrook

@boazsender @cookiecrook I've emailed you (and Lola) a link to the draft documentation I'm proposing. Your thoughts and feedback are welcome!

I've also added a tracker to the issue description with outstanding tasks/questions.

rahimabdi avatar Apr 29 '24 05:04 rahimabdi

Documentation draft: https://docs.google.com/document/d/1Q3A8Y-xcqcKL2BUYyakroWDdHSiYQjkxtxzx_twR0GQ/edit?usp=sharing.

rahimabdi avatar Jul 02 '24 16:07 rahimabdi

@chrishtr you took an action to review. Thank you!

cookiecrook avatar Jul 02 '24 16:07 cookiecrook

Determine publication platform (likely our GitHub Wiki page)

@chrishtr asked me, and I think most of that document would make sense in https://github.com/web-platform-tests/wpt/tree/master/docs/writing-tests, as a new accessibility.md file most likely. It would then be published to https://web-platform-tests.org/.

Parts of the document are about the Accessibility Interop Group, and that I'd suggest putting right here in this repo's README where it's easy to find or stumble upon.

Some of the FAQ questions could probably be put into the main WPT docs and some others are about the Accessibility Interop Group. Splitting that and the overall doc would be shame in a way since it's useful as a rather self-contained resource. An option could to publish it as a whole in this repo as well, but just note which parts are also available on https://web-platform-tests.org/ and which is the source of truth.

Hope that helps.

foolip avatar Jul 02 '24 21:07 foolip

Thanks for chiming in @foolip.

Splitting that and the overall doc would be shame in a way since it's useful as a rather self-contained resource. An option could to publish it as a whole in this repo as well, but just note which parts are also available on https://web-platform-tests.org/ and which is the source of truth.

Agreed, I would also vote for keeping it self-contained in this repo but could be split up otherwise.

rahimabdi avatar Jul 14 '24 16:07 rahimabdi

Thanks for all the work that has gone into this. Is feedback still welcome on the Google Docs draft? If so, could I have comment access? Or is there somewhere else I should provide feedback? I only had some minor clarifications in mind, nothing major. Thanks.

jcsteh avatar Sep 02 '24 05:09 jcsteh

@jcsteh I've just added you: Accessibility Interop Project Documentation DRAFT.

Thank you for the review/feedback!

rahimabdi avatar Sep 03 '24 16:09 rahimabdi

I've published the documentation to the Accessibility Interop's Wiki: https://github.com/web-platform-tests/interop-accessibility/wiki/Accessibility-Interop-Project-Overview-and-Contribution-Guidelines.

Many thanks @gr3g-apple-com @jcsteh for taking the time to review.

@zcorpan @cookiecrook I'll be unavailable for the Oct 1 2024 meeting but could we please add this to the agenda. Unless there is any remaining feedback, this issue is nearing completion.

rahimabdi avatar Sep 20 '24 04:09 rahimabdi

Oct 1 meeting discussion: This documentation page should be linked from:

cookiecrook avatar Oct 01 '24 16:10 cookiecrook

Okay to close now that the follow-ons are tracked separately. Thanks @rahimabdi!

cookiecrook avatar Oct 01 '24 16:10 cookiecrook

Completed, closing.

Final documentation link: https://github.com/web-platform-tests/interop-accessibility/wiki/Accessibility-Interop-Project-Overview-and-Contribution-Guidelines.

rahimabdi avatar Oct 06 '24 05:10 rahimabdi