-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Fix test cleanup to prevent conflicts between tests. #1608
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved, but wondering whether ClientSideEncryptionExternalKeyVaultTest
should also do some cleanup, e.g. drop the collection.
...er-sync/src/test/functional/com/mongodb/client/ClientSideEncryptionExternalKeyVaultTest.java
Show resolved
Hide resolved
JAVA-5761
Let me know why the collection name change had to be reverted. |
Initially, the calculation of bytes to be subtracted from the available bytes for the message in the batch-splitting test was defined as 1122 in the spec tests. After changing the namespace, the test started failing. I reverted the change but later found a way to recalculate it. The test and namespace have now been updated accordingly. |
JAVA-5761