Skip to content

test: stabilize integration tests against dev-server slowness#2362

Draft
jmaeagle99 wants to merge 4 commits into
temporalio:mainfrom
jmaeagle99:test-fixes
Draft

test: stabilize integration tests against dev-server slowness#2362
jmaeagle99 wants to merge 4 commits into
temporalio:mainfrom
jmaeagle99:test-fixes

Conversation

@jmaeagle99

Copy link
Copy Markdown
Contributor

What was changed

  • Consolidate integration test CLI version
  • Consolidate client creation
  • Reorder config callbacks to be applied last
  • Use database file for payload limits tests

Why?

Various connection issues have appeared lately in tests. Some indications show that:

  • The client GetSystemInfoTimeout was too short (fixed by consolidating client creation)
  • Payload limits tests were not using the correct namespace (fixed by reordering config callbacks)
  • Resetting client and worker in payload limits tests is causing contention in database connections (fixed by using database file so connection pooling is enabled, just like main integration test suite).

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