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

Enable missed '.ex-generic' orphaned role tests #45174

Merged

Conversation

nmlapre
Copy link
Contributor

@nmlapre nmlapre commented Mar 18, 2024

Closes web-platform-tests/interop-accessibility#111

This commit adds calls to verifyGenericRolesBySelector in three test files, where those calls are missing. There are orphaned role tests sitting in each file that use the 'ex-generic' class, but since we're not calling verifyGenericRolesBySelector, they aren't running. This commit fixes that problem.

This commit adds calls to verifyGenericRolesBySelector in three test
files, where those calls are missing. There are orphaned role tests
sitting in each file that use the 'ex-generic' class, but since we're
not calling verifyGenericRolesBySelector, they aren't running. This
commit fixes that problem.
@cookiecrook
Copy link
Contributor

cookiecrook commented Mar 28, 2024

Closing and reopening to kick the CI bots.

@cookiecrook cookiecrook reopened this Mar 28, 2024
@cookiecrook
Copy link
Contributor

I don't see anything obvious in the CI logs that would explain the failure as being test-specific. Try to close/open tomorrow to kick the CI bots or file the failure as a standalone WPT harness issue?

@cookiecrook cookiecrook merged commit bd93e91 into web-platform-tests:master Mar 29, 2024
19 checks passed
@nmlapre nmlapre deleted the enable-orphaned-role-tests branch April 10, 2024 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Task: Need to add in the verifyGenericRolesBySelector() call missing from a few files
4 participants