Skip to content

Support for testing additional accessibility properties beyond name and role. #55784

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Support for testing additional accessibility properties beyond name and role. #55784

wants to merge 1 commit into from
+248 −1

Conversation

@jcsteh
Copy link
Contributor

@jcsteh jcsteh commented Oct 31, 2025

  1. Implement get_element_accessible_node and get_accessible_node in wptrunner and TestDriver. For wptrunner, this is currently only supported in executormarionette (Gecko), as there is no implementation yet in any other browser to support.
  2. Add AriaUtils.verifyAccessibilityTree for simpler tree verification using a simple JS object.
  3. Add simple WPT tests exercising and demonstrating these new methods.

@jcsteh
Support for testing additional accessibility properties beyond name a…
bff2df5 
…nd role.

1. Implement get_element_accessible_node and get_accessible_node in wptrunner and TestDriver.
    For wptrunner, this is currently only supported in executormarionette (Gecko), as there is no implementation yet in any other browser to support.
2. Add AriaUtils.verifyAccessibilityTree for simpler tree verification using a simple JS object.
3. Add simple WPT tests exercising and demonstrating these new methods.
@jcsteh jcsteh requested a review from a team as a code owner October 31, 2025 03:21
@wpt-pr-bot wpt-pr-bot added infra testdriver.js wai-aria wg-wai_pf wptrunner The automated test runner, commonly called through ./wpt run labels Oct 31, 2025
@jcsteh
Copy link
Contributor Author

jcsteh commented Oct 31, 2025 •
edited
Loading

There are a number of blocking details to sort out here, but I figured it was worth posting this to get things moving and get some initial feedback.

  1. There is ongoing, unresolved discussion in Tentative testdriver methods rfcs#226 as to whether these TestDriver methods need to be prefixed with tentative_. In short, RFC 127 just says "an existing WebDriver endpoint", which isn't explicit about a formal spec or where it needs to be specified. If an explainer is sufficient, we have Explainer: WebDriver Extension for Accessible Nodes, etc. (potential solution for #197) WICG/aom#203.
  2. In https://github.com/web-platform-tests/rfcs/pull/226/files#r2411669032, @cookiecrook questioned whether we should be removing accessible children/parent ids from this first pass in order to unblock it sooner. I feel this is a fundamental detail we need to sort out because it is a primary goal of the work here. However, pragmatism is important, so I'm happy to go with the consensus here. I guess that would also involve removing verifyAccessibilityTree from this first pass.
  3. executormarionette being the only implementation in this PR is a bit odd. On the other hand, it's difficult to implement (and verify the implementation of) this in executorwebdriver because we don't have a browser implementation yet to use.

@jcsteh jcsteh mentioned this pull request Nov 4, 2025
Open
@cookiecrook cookiecrook mentioned this pull request Nov 4, 2025
Open
25 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cookiecrook cookiecrook Awaiting requested review from cookiecrook

@DanielRyanSmith DanielRyanSmith Awaiting requested review from DanielRyanSmith

@foolip foolip Awaiting requested review from foolip

@jgraham jgraham Awaiting requested review from jgraham

@jnurthen jnurthen Awaiting requested review from jnurthen

@jonathan-j-lee jonathan-j-lee Awaiting requested review from jonathan-j-lee

@spectranaut spectranaut Awaiting requested review from spectranaut

At least 1 approving review is required to merge this pull request.

Assignees

@cookiecrook cookiecrook

Labels

infra testdriver.js wai-aria wg-wai_pf wptrunner The automated test runner, commonly called through ./wpt run

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jcsteh @cookiecrook @wpt-pr-bot