Skip to content

test(tui): xfail test_repository_table_updates on darwin_amd64/CI - #7

Merged
livingstaccato merged 1 commit into
mainfrom
fix/xfail-table-updates-darwin
Apr 25, 2026
Merged

test(tui): xfail test_repository_table_updates on darwin_amd64/CI#7
livingstaccato merged 1 commit into
mainfrom
fix/xfail-table-updates-darwin

Conversation

@livingstaccato

Copy link
Copy Markdown
Member

Same Textual-pilot timing flakiness pattern as the 3 already-xfailed tests (#4). StateUpdate posted via post_message isn't observed in the DataTable within pilot.pause() on macOS Intel CI runners (asserts 2 rows, sees 0). Brings the FLAKY_PILOT exemption count to 4.

Same Textual-pilot timing flakiness as the other 3 already-xfailed tests.
StateUpdate posted via app.post_message() isn't observed in the table
within pilot.pause() on macOS Intel CI runners (asserts 2 rows, sees 0).
Pre-existing pattern; reason matches FLAKY_PILOT marker.
@livingstaccato
livingstaccato merged commit 66c706c into main Apr 25, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request marks the test_repository_table_updates unit test as an expected failure (xfail) when the FLAKY_PILOT condition is met. This change addresses intermittent timing issues in the Textual pilot observed on macOS Intel and CI Linux environments. I have no feedback to provide.

@livingstaccato
livingstaccato deleted the fix/xfail-table-updates-darwin branch April 25, 2026 03:27
livingstaccato added a commit that referenced this pull request Aug 22, 2026
Same Textual-pilot timing flakiness as the other 3 already-xfailed tests.
StateUpdate posted via app.post_message() isn't observed in the table
within pilot.pause() on macOS Intel CI runners (asserts 2 rows, sees 0).
Pre-existing pattern; reason matches FLAKY_PILOT marker.
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