Skip to content
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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

lerouxb
Copy link
Contributor

@lerouxb lerouxb commented Jan 17, 2025

Replaces #6608

Requires #6625, so merge that first.

@lerouxb lerouxb added the no release notes Fix or feature not for release notes label Jan 17, 2025
@github-actions github-actions bot added the feat label Jan 17, 2025
env.UPDATE_CHECKER_ALLOW_DOWNGRADES = 'true';
}

// a git repo that is not published to npm that evergreen clones for us in CI
Copy link
Contributor Author

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') {
Copy link
Contributor Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat no release notes Fix or feature not for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant