Skip to content

Commit

Permalink
added note context on increased timeout in autoconnect sentry test
Browse files Browse the repository at this point in the history
  • Loading branch information
taddes committed Apr 17, 2024
1 parent 807dd5c commit 48f698c
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions tests/integration/test_integration_all_rust.py
Original file line number Diff line number Diff line change
Expand Up @@ -650,6 +650,12 @@ async def test_sentry_output_autoconnect(self):
await httpx_client.get(f"http://localhost:{CONNECTION_PORT}/v1/err/crit", timeout=30)

event1 = MOCK_SENTRY_QUEUE.get(timeout=5)
# NOTE: this timeout increased to 5 seconds as was yielding
# errors in CI due to potential race condition and lingering
# artifact in the MOCK_SENTRY_QUEUE. It could cause test_sentry_output_autoendpoint
# to have two LogCheck entires and test_no_sentry_output would have data
# when it should not.

# new autoconnect emits 2 events
try:
MOCK_SENTRY_QUEUE.get(timeout=5)
Expand Down

0 comments on commit 48f698c

Please sign in to comment.