test(a11y): playwright coverage for explore - graph controls -bed-8633

[JayC-Specter]

Description

Adds accessibility tests for the Explore - graph controls to verify compliance with accessibility standards.

Motivation and Context

Resolves bed-8633

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

This change was needed to ensure the Explore - graph controls 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:

• I have met the contributing prerequisites
  • Assigned myself to this PR
  • Added the appropriate labels
  • Associated an issue: Issues and Pull Requests README #672
  • Read the Contributing guide: https://github.com/SpecterOps/BloodHound/wiki/Contributing
• I have ensured that related documentation is up-to-date
  • Open API docs
  • Code comments (GoDocs / JSDocs)
• I have followed proper test practices
  • Added/updated tests to cover my changes
  • All new and existing tests passed

Summary by CodeRabbit

• Tests
  • Added accessibility coverage for Explore graph controls, including Hide Labels, Layout, Export, and Search.
  • Added checks for search results and no-results states to help prevent accessibility regressions.

[JayC-Specter]

All contributors have signed the CLA

[coderabbitai]

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: 05e63f76-bbb0-4453-ad49-51b14072e05c

📥 Commits

Reviewing files that changed from the base of the PR and between f6ecae4 and 0aef79c.

📒 Files selected for processing (1)

• cmd/ui/tests/a11y/Explore/explore-graph-controls.a11y.spec.ts

---

📝 Walkthrough

Walkthrough

The PR adds a Playwright accessibility test suite for Explore graph controls. The suite mocks graph data, intercepts the Cypher API, loads Explore, and checks expanded control menus and graph search states with Axe.

Changes

Explore graph controls

Layer / File(s)	Summary
Control and search accessibility coverage cmd/ui/tests/a11y/Explore/explore-graph-controls.a11y.spec.ts	Adds graph fixtures, route interception, Explore page setup, and helpers for expanding graph search. Axe checks cover Hide Labels, Layout, Export, Search results, and no-results states.

Estimated code review effort: 3 (Moderate) | ~15–30 minutes

Possibly related PRs

• SpecterOps/BloodHound#2774: Adds accessibility tests for the same Explore graph-control label menus.
• SpecterOps/BloodHound#3034: Adds Explore search accessibility coverage for results and no-results states.
• SpecterOps/BloodHound#3036: Introduces Explore graph-control interactions covered by these tests.

Suggested reviewers: thenando

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the new Playwright accessibility coverage for Explore graph controls.
Description check	✅ Passed	The description covers the changes, motivation, ticket, testing, change type, and checklist.
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 jcarver/explore-graph-controls-a11y

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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