Skip to content

Fix flaky integration tests#195

Merged
vrslev merged 6 commits into
mainfrom
fix/flaky-tests-160
Jun 3, 2026
Merged

Fix flaky integration tests#195
vrslev merged 6 commits into
mainfrom
fix/flaky-tests-160

Conversation

@vrslev

@vrslev vrslev commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Fixes #160.

Changes:

  • Wait for ActiveMQ Artemis to accept a STOMP CONNECT before running integration tests.
  • Remove pytest reruns and the pytest-rerunfailures dependency so connection/startup failures are not masked.
  • Make reconnect integration test deterministic by forcing reconnects instead of waiting on timing-based no-message restarts.
  • Ensure the bulk integration producer waits until the subscription is registered.

Local verification:

  • just lint
  • just check-types
  • just test-fast
  • just test
  • After cleanup: just install, just test-fast, targeted reconnect+bulk integration tests.

CI verification:

  • 6 consecutive green CI runs total.
  • 2 consecutive green CI runs after removing pytest-rerunfailures and destination mangling.

@vrslev vrslev merged commit 207b45b into main Jun 3, 2026
6 checks passed
@vrslev vrslev deleted the fix/flaky-tests-160 branch June 3, 2026 09:04
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.

Fix flaky tests

1 participant