[DO NOT MERGE] Comparison branch for Antithesis test changes #355
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…357) Moved common starter signal `"Consumed N output records" from e2e test driver to generic utils class so we can regex match for the common Logger prefix instead of looking for test-specific class names like E2eTestDriver Also added e2e error injection to the regression test topologies (for the responsive app only)
Co-authored-by: Jason Gustafson <[email protected]>
…htlies (#359) Two new Antithesis custom params: -database_type just determines which nodes get fault injection (mongo vs scylla) -test_type is just used to gate test-specific assertions until we can migrate fully to the Java SDK
Wanted to be able to set a higher connection idle timeout. Might as well take the opportunity to allow configuration of all the Mongo params via the connection string, rather than adding them one by one.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Not a real PR -- these changes have already been merged to main. This PR is comparing
mainto an older branch from before these changes to get the regression test running in Antithesis, and is just showing these changes together for review purposes