-
Notifications
You must be signed in to change notification settings - Fork 451
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[tests] Improve Cosmos EndToEnd test #5639
Conversation
…d of 1s to reduce noise in the logs
…tching log texts in the tests
Related: #5641 |
What do you think about removing this test from the EndToEnd and putting it in its own FunctionalTests? Like we did for the other integrations. I think this is just left over from that work. Then we wouldn't need to clutter up the EndToEnd tests with this change. |
Makes sense. cc @sebastienros as he has been adding the functional tests. |
Added |
Wait for the texts indicating that the db, and service are ready for requests before sending the http request. This helps with unnecessary attempts, and wait times in the retry pipeline.
Microsoft Reviewers: Open in CodeFlow