Skip to content

Conversation

@WilsonNet
Copy link
Collaborator

Summary

  • Add data-test-id attributes for complex selectors in tree-details page
  • Update e2e-selectors.ts to use data-test-id instead of complex text-based selectors
  • Fix tests to properly handle navigation and expanded rows
  • All 8 tests now passing

Changes

  • Added data-test-id to BreadcrumbLink, filter drawer buttons, status chart legend buttons, and details buttons
  • Updated test selectors to use data-test-id attributes
  • Fixed breadcrumb URL assertions to handle search parameters
  • Fixed tests table test to expand rows before clicking details button
  • Simplified filter drawer test

- Create tree-details.spec.ts with 10 test cases covering:
  - Navigation to tree details from tree listing
  - Tab switching (Builds/Boots/Tests)
  - Filter interactions
  - Breadcrumb navigation
  - Status filter functionality
  - Search input functionality
  - Detail button clicks (builds/boots/tests)
  - Issue details navigation

- Add TREE_DETAILS_SELECTORS to e2e-selectors.ts

- WIP: 6/10 tests passing, 4 tests failing due to selector issues
  - Failing: commit graph, filter card, build table details, tests table details
  - Need to investigate DOM structure and fix selectors

- Note: Plan to add data-test-id attributes to React components
  for more reliable test selection

- Created WIP_tree-details-e2e.md documenting current state
@WilsonNet WilsonNet changed the title Fix Playwright tests for tree-details page [draft]playwright tests for tree-details page Jan 28, 2026
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