Skip to content

Development/resolve test failures - #315

Merged
smanes0213 merged 12 commits into
masterfrom
development/resolve_test_failures
Jul 29, 2026
Merged

Development/resolve test failures#315
smanes0213 merged 12 commits into
masterfrom
development/resolve_test_failures

Conversation

@smanes0213

@smanes0213 smanes0213 commented Jul 29, 2026

Copy link
Copy Markdown
Contributor
  • Increase RPC socket close timeout from 1s to 5s for graceful shutdown
  • Increase CTest timeout from 1500s to 3600s (60 min)
  • Add 500ms SetUp() sleep between tests to allow socket draining

Fixes cascading failures when tests take 2-3s with async operations.

The root cause: when tests like TestEvents take 2-3s waiting for async
notifications, the RPC socket is still draining. Next test's Register() call
hits the internal RPC timeout (2-6s) before socket is ready, causing ERROR_TIMEOUT
(error code 2147483659) and subsequent test failures.

This allows the test suite to gracefully handle slow CI without affecting
developers testing on fast machines.

@smanes0213
smanes0213 requested a review from MFransen69 July 29, 2026 10:20
@smanes0213
smanes0213 merged commit 26471c5 into master Jul 29, 2026
93 checks passed
@smanes0213
smanes0213 deleted the development/resolve_test_failures branch July 29, 2026 10:52
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants