chore(e2e): test auto-update from COMPASS-8536 #6608
+219
−44
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,31 @@ | ||
| set -e | ||
|
|
||
| HAS_DOCKER=false | ||
|
|
||
| DOCKER_BUILD="" | ||
|
|
||
| echo "Checking if docker is available ..." | ||
|
|
||
| if docker version &>/dev/null; then | ||
| echo " docker is available, checking docker build" | ||
| if docker compose version &>/dev/null; then | ||
| echo " docker build is available" | ||
| HAS_DOCKER=true | ||
| DOCKER_BUILD="docker build" | ||
| else | ||
| echo " docker compose could not be found, trying standalone docker-compose as a fallback" | ||
| if docker-build version &>/dev/null; then | ||
| HAS_DOCKER=true | ||
| DOCKER_BUILD="docker-build" | ||
| else | ||
| echo " docker build not found" | ||
| fi | ||
| fi | ||
| fi | ||
|
|
||
| if [ "$HAS_DOCKER" = true ]; then | ||
| $DOCKER_BUILD -t compass/web ./ | ||
| docker run -p 8080:8080 -t -e PORT=8080 -e UPDATE_CHECKER_ALLOW_DOWNGRADES=true -d compass/web | ||
| else | ||
| echo " docker not found" | ||
| fi |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,14 +8,20 @@ export function execute( | |
| ): Promise<void> { | ||
| return new Promise((resolve, reject) => { | ||
| console.log(command, ...args); | ||
|
|
||
| const p = spawn(command, args, { | ||
| stdio: 'inherit', | ||
| ...options, | ||
| }); | ||
|
|
||
| p.on('error', (err: any) => { | ||
| reject(err); | ||
| }); | ||
|
|
||
| p.on('close', (code: number | null, signal: NodeJS.Signals | null) => { | ||
| process.off('exit', killChild); | ||
| process.off('SIGINT', interruptChild); | ||
|
|
||
| if (code !== null) { | ||
| if (code === 0) { | ||
| resolve(); | ||
|
|
@@ -41,5 +47,11 @@ export function execute( | |
| } | ||
| } | ||
| }); | ||
|
|
||
| // Exit child process if main process exits | ||
|
Contributor
Author
There was a problem hiding this comment. Copied from https://github.com/kraenhansen/bufout which we should probably just use. |
||
| const killChild = () => p.kill(); | ||
| const interruptChild = () => p.kill('SIGINT'); | ||
| process.once('exit', killChild); | ||
| process.once('SIGINT', interruptChild); | ||
| }); | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,51 @@ | ||
| import { execute } from '../installers/helpers'; | ||
| import type { Package } from '../installers/types'; | ||
|
|
||
| function testInstalledApp( | ||
| pkg: Package, | ||
| appPath: string, | ||
| env: Record<string, string> | ||
| ): Promise<void> { | ||
| return execute( | ||
| 'npm', | ||
| [ | ||
| 'run', | ||
| '--unsafe-perm', | ||
| 'test-packaged', | ||
| '--workspace', | ||
| 'compass-e2e-tests', | ||
| '--', | ||
| '--test-filter=auto-update', | ||
| ], | ||
| { | ||
| env: { | ||
| ...process.env, | ||
| HADRON_AUTO_UPDATE_ENDPOINT_OVERRIDE: 'http://localhost:8080', | ||
| AUTO_UPDATE_UPDATABLE: pkg.updatable.toString(), | ||
| COMPASS_APP_NAME: pkg.appName, | ||
| COMPASS_APP_PATH: appPath, | ||
| ...env, | ||
| }, | ||
| } | ||
| ); | ||
| } | ||
|
|
||
| export async function testAutoUpdateFrom(pkg: Package) { | ||
| // install the app | ||
| console.log(`installing ${pkg.packageFilepath}`); | ||
| const { appPath, uninstall } = await pkg.installer({ | ||
| appName: pkg.appName, | ||
| filepath: pkg.packageFilepath, | ||
| }); | ||
|
|
||
| console.log(appPath); | ||
|
|
||
| try { | ||
| await testInstalledApp(pkg, appPath, { | ||
| AUTO_UPDATE_FROM: 'true', | ||
| }); | ||
| } finally { | ||
| // remove the app | ||
| await uninstall(); | ||
| } | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,75 @@ | ||
| import { expect } from 'chai'; | ||
| import { | ||
| init, | ||
| cleanup, | ||
| Selectors, | ||
| screenshotPathName, | ||
| } from '../helpers/compass'; | ||
|
|
||
| describe('Auto-update', function () { | ||
| it('auto-update from', async function () { | ||
| if (!process.env.AUTO_UPDATE_FROM) { | ||
| // we don't want this test to execute along with all the others under | ||
| // normal circumstances because it is destructive - it overwrites Compass | ||
| // itself | ||
| this.skip(); | ||
| } | ||
|
|
||
| // run the app and wait for it to auto-update | ||
| const compass = await init('auto-update from', { firstRun: true }); | ||
| const { browser } = compass; | ||
| try { | ||
| await browser.$(Selectors.AutoUpdateToast).waitForDisplayed(); | ||
|
|
||
| if (process.env.AUTO_UPDATE_UPDATABLE === 'true') { | ||
| const restartButton = browser.$(Selectors.AutoUpdateRestartButton); | ||
| await restartButton.waitForDisplayed(); | ||
|
|
||
| // We could click the restart button to apply the update and restart the | ||
| // app, but restarting the app confuses webdriverio or at least our test | ||
| // helpers. So we're going to just restart the app manually. | ||
| } else { | ||
| // When auto-update is not supported the toast contains a link to | ||
| // download | ||
| const linkElement = browser.$(Selectors.AutoUpdateDownloadLink); | ||
| await linkElement.waitForDisplayed(); | ||
| expect(await linkElement.getAttribute('href')).to.equal( | ||
| 'https://www.mongodb.com/try/download/compass' | ||
| ); | ||
| } | ||
| } finally { | ||
| await browser.screenshot(screenshotPathName('auto-update-from')); | ||
| await cleanup(compass); | ||
| } | ||
|
|
||
| if (process.env.AUTO_UPDATE_UPDATABLE === 'true') { | ||
| // run the app again and check that the version changed | ||
| const compass = await init('auto-update from restart', { | ||
| firstRun: false, | ||
| }); | ||
| const { browser } = compass; | ||
| try { | ||
| await browser.$(Selectors.AutoUpdateToast).waitForDisplayed(); | ||
| await browser | ||
| .$(Selectors.AutoUpdateReleaseNotesLink) | ||
| .waitForDisplayed(); | ||
| } finally { | ||
| await browser.screenshot( | ||
| screenshotPathName('auto-update-from-restart') | ||
| ); | ||
| await cleanup(compass); | ||
| } | ||
| } | ||
| }); | ||
|
|
||
| it('auto-update to', function () { | ||
| if (!process.env.AUTO_UPDATE_TO) { | ||
| // we don't want this test to execute along with all the others under | ||
| // normal circumstances because it is destructive - it overwrites Compass | ||
| // itself | ||
| this.skip(); | ||
| } | ||
|
|
||
| // TODO | ||
| }); | ||
| }); |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
This is to make it easy to copy/paste these into your local shell when working on the smoketests. Otherwise you have to transform the string manually every time.