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

Use Roborazzi A11y checks #2463

Merged
merged 7 commits into from
Nov 21, 2024
Merged

Use Roborazzi A11y checks #2463

merged 7 commits into from
Nov 21, 2024

Conversation

yschimke
Copy link
Collaborator

WHAT

This change updates the handling of accessibility checks for off-screen elements in the UI tests. It suppresses accessibility errors for elements which are off-screen but are intended to be scrolled on-screen.

This improves the accuracy and efficiency of the accessibility tests by ignoring false-positives related to these elements.

Additionally, it removes unnecessary custom shadow classes that were previously used for accessibility testing. These shadows caused conflicts and were replaced by existing Roborazzi features. This simplifies the testing setup and relies on external libraries for better maintainability.

Finally, it updates the version of the Roborazzi library to bring in the new accessibility checking capabilities. This allows the tests to use newer, more robust and accurate accessibility checking features.

WHY

HOW

Checklist 📋

  • Add explicit visibility modifier and explicit return types for public declarations
  • Run spotless check
  • Run tests
  • Update metalava's signature text files

This change updates the handling of accessibility checks for off-screen elements in the UI tests. It suppresses accessibility errors for elements which are off-screen but are intended to be scrolled on-screen.

This improves the accuracy and efficiency of the accessibility tests by ignoring false-positives related to these elements.

Additionally, it removes unnecessary custom shadow classes that were previously used for accessibility testing.
These shadows caused conflicts and were replaced by existing Roborazzi features.
This simplifies the testing setup and relies on external libraries for better maintainability.

Finally, it updates the version of the Roborazzi library to bring in the new accessibility checking capabilities.
This allows the tests to use newer, more robust and accurate accessibility checking features.
@yschimke yschimke requested a review from kul3r4 November 21, 2024 10:46
@yschimke
Copy link
Collaborator Author

View with bounds: [24,438][430,542]: This clickable item's speakable text: "Playlists" is identical to that of 1 other item(s). Reported by com.google.android.apps.common.testing.accessibility.framework.checks.DuplicateSpeakableTextCheck

image

@yschimke yschimke requested a review from garanj November 21, 2024 12:17
@yschimke
Copy link
Collaborator Author

@garanj landing as test only, will follow up on review comments.

@yschimke yschimke merged commit 7785bae into google:main Nov 21, 2024
6 checks passed
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

Successfully merging this pull request may close these issues.

1 participant