Skip to content

Conversation

@dluffy56
Copy link

@dluffy56 dluffy56 commented Feb 11, 2026

Issue: #1531
Previously PRs: #1597, #1599

Description

Problem

The test complete-flow spec was flaky because it used broad page.locator("tbody") selectors to check table content. When multiple invoice rows were present, assertions were matching the entire table body instead of a specific row, causing unreliable failures.

Additionally, the date picker (React Aria spinbutton) sometimes did not fully commit the year value. In some runs, Dec 1, 2024 was being saved/rendered as Dec 1, 202, which caused the assertion to fail intermittently.

Solution

Changed from broad tbody selectors to row-specific selectors using page.locator("tbody tr").filter({ hasText: "CUSTOM-X" }). This ensures assertions are targeted to the specific invoice row instead of the entire table body.

Also stabilized the fillDatePicker helper by clearing the field first, typing sequentially with delay, blurring the input to trigger commit, and asserting the final value to guarantee the full year is saved.


Test Results

Tested with --repeat-each=20 --workers=1 - all 20 test runs passed

Screenshot from 2026-02-06 09-41-05

Checklist


AI Disclosure

Used Claude Opus 4.5 to understand the issue and implement the fix. Code was reviewed and tested manually.

@dluffy56 dluffy56 changed the title Fix/complete flow fix: complete-flow spec flaky test Feb 11, 2026
@dluffy56 dluffy56 closed this Feb 11, 2026
@dluffy56 dluffy56 reopened this Feb 11, 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