Skip to content

test: migrate getByTestId queries to accessible queries - #3557

Merged
setchy merged 2 commits into
mainfrom
test/sidebar-accessible-queries
Oct 28, 2026
Merged

setchy merged 2 commits into
mainfrom
test/sidebar-accessible-queries

Conversation

@setchy

@setchy setchy commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Replace getByTestId queries with accessible queries (getByRole, getByLabelText, etc.) in tests.
  • Remove data-testid attributes that are no longer used.
  • Fix the Header back button's accessible name (aria-labelledby → aria-label).

Closes #3510

Fix back button accessible name in Header (aria-labelledby -> aria-label).

Refs #3510
@setchy
setchy requested a review from afonsojramos as a code owner October 28, 2026 23:14
@github-actions github-actions Bot added the test Enhancements to test coverage or quality label Oct 28, 2026
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
75.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@setchy
setchy merged commit aab112d into main Oct 28, 2026
22 of 23 checks passed
@setchy
setchy deleted the test/sidebar-accessible-queries branch October 28, 2026 23:18
@github-actions github-actions Bot added this to the Release 8.0.0 milestone Oct 28, 2026
setchy added a commit that referenced this pull request Oct 28, 2026
…h-app-login-to-device-flow

Resolve conflicts with #3557: keep LoginWithOAuthApp removed, keep the
thin LoginWithDeviceFlow wrapper, and use accessible queries for the
Custom OAuth App menu item. Port #3557's unused test-ID cleanup to
DeviceFlowAuthorize and LoginWithCustomOAuthApp, and drop the glass
baselines for the retired oauth-app route.
setchy added a commit that referenced this pull request Oct 29, 2026
Resolve conflicts with #3449, #3557, #3558 and the metric pill changes:
keep main's structure (device-flow wrapper, custom OAuth app, removed
test IDs, chevron directions) and wrap its strings for translation.
Translate the screens main added since this branch was cut
(DeviceFlowAuthorize, LoginWithCustomOAuthApp, device-flow start errors,
ScopeStatusIcon, HelpPopover) and regenerate en.json.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Enhancements to test coverage or quality

Development

Successfully merging this pull request may close these issues.

Migrate remaining getByTestId queries to accessible queries

1 participant