Skip to content

Commit

Permalink
Update CSOT test definition.
Browse files Browse the repository at this point in the history
  • Loading branch information
vbabanin committed Feb 23, 2024
1 parent 4b7d757 commit 71ca227
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
4 changes: 2 additions & 2 deletions source/client-side-encryption/tests/legacy/timeoutMS.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

10 changes: 6 additions & 4 deletions source/client-side-encryption/tests/legacy/timeoutMS.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,20 +38,22 @@ tests:
command_name: listCollections

# Test that timeoutMS applies to the sum of all operations done for client-side encryption. This is done by blocking
# listCollections and find for 20ms each and running an insertOne with timeoutMS=50. There should be two
# listCollections commands and one "find" command, so the sum should take more than timeoutMS.
# listCollections and find for 30ms each and running an insertOne with timeoutMS=50. There should be one
# listCollections command and one "find" command, so the sum should take more than timeoutMS. A second listCollections
# event doesn't occur due to the internal MongoClient lacking configured auto encryption, plus libmongocrypt holds the
# collection schema in cache for a minute.
#
# This test does not include command monitoring expectations because the exact command sequence is dependent on the
# amount of time taken by mongocryptd communication. In slow runs, mongocryptd communication can breach the timeout
# and result in the final "find" not being sent.
- description: "remaining timeoutMS applied to find to get keyvault data"
failPoint:
configureFailPoint: failCommand
mode: { times: 3 }
mode: { times: 2 }
data:
failCommands: ["listCollections", "find"]
blockConnection: true
blockTimeMS: 20
blockTimeMS: 30
clientOptions:
autoEncryptOpts:
kmsProviders:
Expand Down

0 comments on commit 71ca227

Please sign in to comment.