Merged
Conversation
Add Playwright E2E tests for the assessment tracker accessed via the project view frameworks tab. Covers page rendering, Controls/ Assessments toggle, progress stats, topic navigation, and accessibility. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add Playwright E2E tests for the compliance tracker embedded in the project view frameworks tab. Covers progress stats, control categories, expand/collapse, framework tab switching, filters, and accessibility. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add Playwright E2E tests for the super admin pages covering organizations (list, search, sort, create modal), all users (list, search, org/role filters), settings (profile/password tabs), and cross-page navigation. Tests gracefully skip when the user lacks super admin access. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add Playwright E2E tests for the public-facing intake form pages. Covers route resolution (new and legacy formats), form field rendering, contact info fields, submit button, validation errors, math captcha, footer, success page, and accessibility. Uses a custom fixture to discover valid form URLs via authenticated context. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add aria-input-field-name to axe disabled rules (assessment, compliance-tracker) - Add graceful skip for compliance-tracker render when content unavailable - Match "unavailable"/"expired" text on public intake success page - Add skip logic for super-admin users table when content not loaded Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The Super Admin org switcher MuiSelect had no accessible name, causing axe aria-input-field-name violations on every page that renders the banner (~25 e2e a11y failures).
Replaces the brittle getByText(...).locator('..').getByRole('textbox')
chain with the stable #project-title-input selector exposed by
CreateProjectForm. Unblocks vendors, datasets, and risk-management
CRUD specs.
…pages - /register now redirects to /login, so the 'register here' link test is rewritten to assert the redirect. - Registration form tests target /user-reg (the public registration route) instead of the redirecting /register. - Logout test uses a more robust ancestor xpath to locate the MoreVertical IconButton next to the user role label.
Tighten Password/Organization/Profile tab locators to
getByRole('tab').filter({ hasText }) so they match the MUI Tab
exclusively instead of falling back to ambiguous getByText matches
elsewhere on the page. Relax the profile URL regex to accept both
/settings and /settings/.
Match the PageHeaderExtended 'Plugins' heading explicitly instead of the ambiguous getByText(/plugin/i) which could match nav items, tooltips, and body copy under strict mode.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
More high priority e2e