Skip to content

test(a11y): admin file ingest, data quality coverage - BED-8639 - #3100

Merged
JayC-Specter merged 3 commits into
mainfrom
BED-8639-administration-a11y-pr1
Aug 7, 2026
Merged

test(a11y): admin file ingest, data quality coverage - BED-8639#3100
JayC-Specter merged 3 commits into
mainfrom
BED-8639-administration-a11y-pr1

Conversation

@JayC-Specter

@JayC-Specter JayC-Specter commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Adds accessibility tests for the Administration file ingest and data quality pages, to verify compliance with accessibility standards.

Motivation and Context

Resolves bed-8639

Why is this change required? What problem does it solve?

This change was needed to ensure the Administration file ingest and data quality pages adhere to accessibility standards.

How Has This Been Tested?

The tests were run locally.

Screenshots (optional):

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

Summary by CodeRabbit

  • Tests
    • Added accessibility coverage for the Administration Data Quality page, including empty and populated states.
    • Added accessibility coverage for the Administration File Ingest page, including history states, ingest details, and filtering.
    • Added checks to help ensure key page areas and dialogs meet accessibility standards across common user workflows.

@JayC-Specter JayC-Specter self-assigned this Aug 5, 2026
@JayC-Specter JayC-Specter added enhancement New feature or request user interface A pull request containing changes affecting the UI code. javascript Pull requests that update javascript code labels Aug 5, 2026
@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 9443f8ff-39a3-4e6e-940c-81beb66e2fd7

📥 Commits

Reviewing files that changed from the base of the PR and between 80bb7ab and 3ae21c7.

📒 Files selected for processing (2)
  • cmd/ui/tests/a11y/Administration/data-quality.a11y.spec.ts
  • cmd/ui/tests/a11y/Administration/file-ingest.a11y.spec.ts
🚧 Files skipped from review as they are similar to previous changes (2)
  • cmd/ui/tests/a11y/Administration/file-ingest.a11y.spec.ts
  • cmd/ui/tests/a11y/Administration/data-quality.a11y.spec.ts

📝 Walkthrough

Walkthrough

Added Playwright accessibility suites for Administration Data Quality and File Ingest pages. The tests mock API responses, cover empty and populated states, and run axe checks on page content.

Changes

Administration accessibility tests

Layer / File(s) Summary
Data Quality accessibility scenarios
cmd/ui/tests/a11y/Administration/data-quality.a11y.spec.ts
Covers empty and populated Data Quality states with mocked responses, UI assertions, and axe checks.
File Ingest test setup
cmd/ui/tests/a11y/Administration/file-ingest.a11y.spec.ts
Adds response fixtures and feature-flag mocking for File Ingest accessibility tests.
File Ingest accessibility scenarios
cmd/ui/tests/a11y/Administration/file-ingest.a11y.spec.ts
Covers empty history, populated history, completed and failed ingests, parse-error details, and the filter dialog.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the accessibility test coverage added for the Administration file ingest and data quality pages.
Description check ✅ Passed The description includes the required sections, ticket reference, testing summary, change type, and completed checklist items.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch BED-8639-administration-a11y-pr1

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@JayC-Specter
JayC-Specter marked this pull request as draft August 5, 2026 20:41
@JayC-Specter

Copy link
Copy Markdown
Contributor Author

All contributors have signed the CLA

@TheNando TheNando changed the title test(a11y): playwright coverage for administration - file ingest and data quality pages - bed-8639 test(a11y): administration file ingest, data quality coverage - BED-8639 Aug 7, 2026
@TheNando TheNando changed the title test(a11y): administration file ingest, data quality coverage - BED-8639 test(a11y): admin file ingest, data quality coverage - BED-8639 Aug 7, 2026
Comment thread cmd/ui/tests/a11y/Administration/data-quality.a11y.spec.ts Outdated
Comment thread cmd/ui/tests/a11y/Administration/data-quality.a11y.spec.ts Outdated
Comment thread cmd/ui/tests/a11y/Administration/data-quality.a11y.spec.ts Outdated
Comment thread cmd/ui/tests/a11y/Administration/data-quality.a11y.spec.ts
Comment thread cmd/ui/tests/a11y/Administration/data-quality.a11y.spec.ts Outdated
Comment thread cmd/ui/tests/a11y/Administration/file-ingest.a11y.spec.ts Outdated
Comment thread cmd/ui/tests/a11y/Administration/file-ingest.a11y.spec.ts Outdated
Comment thread cmd/ui/tests/a11y/Administration/file-ingest.a11y.spec.ts Outdated
@JayC-Specter
JayC-Specter merged commit 0862158 into main Aug 7, 2026
14 checks passed
@JayC-Specter
JayC-Specter deleted the BED-8639-administration-a11y-pr1 branch August 7, 2026 20:24
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request javascript Pull requests that update javascript code user interface A pull request containing changes affecting the UI code.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants