-
Notifications
You must be signed in to change notification settings - Fork 190
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
feat(smoke-tests): test auto-updating from the version of Compass that was just packaged COMPASS-8536 #6629
base: main
Are you sure you want to change the base?
Conversation
env.UPDATE_CHECKER_ALLOW_DOWNGRADES = 'true'; | ||
} | ||
|
||
// a git repo that is not published to npm that evergreen clones for us in CI |
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.
I don't know how best to cleanly start this, especially in dev. This assumes that compass-mongodb-com
is cloned right next to compass
which is hacky.
My initial plan was to start the server using docker in evergreen so it is already running before we execute the smoke tests, but we don't have docker in every platform where we want to run the smoke tests, so I think we have to start the server manually.
Furthermore we'll also have two of these servers with different configuration that we'll need back-to-back because updating from the most recent release TO this test subject will require telling the auto-update server about this new unreleased copy of compass.
|
||
describe('Auto-update', function () { | ||
it('auto-update from', async function () { | ||
if (process.env.TEST_NAME !== 'AUTO_UPDATE_FROM') { |
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.
Open to ideas for a better name for this env var. I think we'll also have AUTO_UPDATE_TO once we add that test next to this one. These tests kinda have to be the only test you run at a time because they replace the test subject.
c9c0779
to
de07d7f
Compare
Replaces #6608
Requires #6625, so merge that first.