Skip to content

test: make tests more stable#601

Merged
mhuisi merged 1 commit intomasterfrom
mhuisi/racey-test
Apr 8, 2025
Merged

test: make tests more stable#601
mhuisi merged 1 commit intomasterfrom
mhuisi/racey-test

Conversation

@mhuisi
Copy link
Copy Markdown
Collaborator

@mhuisi mhuisi commented Apr 7, 2025

This PR adjusts tests that check the contents of 'All Messages' to check the content of 'Messages' instead, which (when initializing the InfoView for the first time) is sometimes subject to a race condition that makes it not display any messages. This is a known, albeit minor, issue.

@mhuisi mhuisi force-pushed the mhuisi/racey-test branch from 6f58abb to da42144 Compare April 8, 2025 13:40
@mhuisi mhuisi force-pushed the mhuisi/racey-test branch from da42144 to 48367a2 Compare April 8, 2025 13:40
@mhuisi mhuisi changed the title test: remove racey test test: make tests more stable Apr 8, 2025
@mhuisi mhuisi merged commit 389bb4d into master Apr 8, 2025
2 checks passed
@mhuisi mhuisi deleted the mhuisi/racey-test branch October 21, 2025 12:37
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