ci(NODE-6505): CI Setup for Encryption Support #15139
Conversation
lint typo lint
aditi-khare-mongoDB
force-pushed
the
NODE-6505/ci-setup
branch
4 times, most recently
from
8ec6cef to
7cd83f6
Compare
readd dependency readd dependency lint readd dependency readd dependency fix dep fix dep2 fix dep2 fix dep3 fix dep4 fix dep5 fix dep6
aditi-khare-mongoDB
force-pushed
the
NODE-6505/ci-setup
branch
from
e8fa19e to
06ffdea
Compare
package.json
Outdated
| @@ -104,6 +104,7 @@ | |||
| "test-deno": "deno run --allow-env --allow-read --allow-net --allow-run --allow-sys --allow-write ./test/deno.js", | |||
| "test-rs": "START_REPLICA_SET=1 mocha --timeout 30000 --exit ./test/*.test.js", | |||
| "test-tsd": "node ./test/types/check-types-filename && tsd", | |||
| "test-encryption": "bash scripts/configure-cluster-with-encryption.sh && mocha --exit ./test/encryption/*.test.js && npm uninstall mongodb-client-encryption > /dev/null", | |||
There was a problem hiding this comment.
Can we make this command just run mocha and move the configure-cluster-with-encryption.sh script to a setup-encryption-test script? Sometimes you don't want to re-run configure-cluster-with-encryption.sh when running locally, and also it would be great be able to pass command line flags to Mocha.
There was a problem hiding this comment.
Sounds good, made the change!
|
@vkarpov15 What errors and hacks did you need to get this working locally? Ideally it's trivial for anybody to set up and run just from our tooling 🙂 |
|
@baileympearson I don't recall exactly, there were some issues with finding a python3 install and needing to set up venv. Nothing is trivial when installing python is involved :) If I run into issues again, I'll do a better job writing them down and let you know. |
|
Merging this into 8.10. |
Descriptions
This testing framework is being added in to allow for ease of encryption testing in PRs to come.
Add CI configuration to ensure that FLE can be end-to-end tested in the mongoose repo.
Add scripting so users can test locally and understand how the CI configuration works.
Summary of Changes