Skip to content

Commit dbce22b

Browse files
lerouxbkraenhansen
andauthored
feat: enable auto-update-to tests, auto-update-from on mac and upload logs COMPASS-9063 COMPASS-8128 (#6749)
* try and enable all the skipped tests * upload logs * longer timeout * also do the no-sandbox thing on linux_tar * remove comment that seems to be invalid * strip down how many auto-update-to tests we run * add a timeout for the test job * disable linux_tar again * Update .github/workflows/test-installers.yml Co-authored-by: Kræn Hansen <kraen.hansen@mongodb.com> * remove linux_tar fixes * bump timeout * bump the timeout where we wait for the update to finish downloading * disable auto-update-from on windows again * Revert "bump the timeout where we wait for the update to finish downloading" This reverts commit 09b13ca. --------- Co-authored-by: Kræn Hansen <kraen.hansen@mongodb.com>
1 parent 81af201 commit dbce22b

File tree

2 files changed

+39
-20
lines changed

2 files changed

+39
-20
lines changed

.github/workflows/test-installers.yml

Lines changed: 38 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@ jobs:
4141
run: echo "[Evergreen Task](${{ github.event.inputs.evergreen_task_url }})" >> $GITHUB_STEP_SUMMARY
4242
test:
4343
name: ${{ matrix.package }} test ${{ matrix.test }} (${{ matrix.hadron-distribution }})
44+
timeout-minutes: 30
4445
strategy:
4546
fail-fast: false
4647
matrix:
@@ -55,9 +56,8 @@ jobs:
5556
- windows_msi
5657
- windows_setup
5758
- linux_deb
58-
# TODO: Re-enable (see https://github.com/mongodb-js/compass/actions/runs/13281152689/job/37079619474)
59-
# - linux_tar
60-
# TODO: Enable (needs a docker container)
59+
# TODO(COMPASS-9061): Fix and enable the linux_tar smoke tests
60+
#- linux_tar
6161
- linux_rpm
6262
hadron-distribution:
6363
- compass
@@ -93,10 +93,11 @@ jobs:
9393
runs-on: ubuntu-latest
9494
arch: x64
9595
hadron-platform: linux
96-
# - package: linux_tar
97-
# runs-on: ubuntu-latest
98-
# arch: x64
99-
# hadron-platform: linux
96+
# TODO(COMPASS-9061): Fix and enable the linux_tar smoke tests
97+
#- package: linux_tar
98+
# runs-on: ubuntu-latest
99+
# arch: x64
100+
# hadron-platform: linux
100101
- package: linux_rpm
101102
runs-on: ubuntu-latest
102103
arch: x64
@@ -120,12 +121,6 @@ jobs:
120121
install-update-server: true
121122

122123
exclude:
123-
# TODO: See https://github.com/mongodb-js/compass/actions/runs/13281152689/job/37079620322
124-
- package: osx_dmg
125-
test: auto-update-from
126-
# TODO: See https://github.com/mongodb-js/compass/actions/runs/13281152689/job/37079621700
127-
- package: windows_setup
128-
test: auto-update-from
129124
# Skip auto-update tests for isolated and readonly distributions for now
130125
# This is not supported by the update server we're using in the test harness
131126
- hadron-distribution: compass-isolated
@@ -136,18 +131,34 @@ jobs:
136131
test: auto-update-from
137132
- hadron-distribution: compass-readonly
138133
test: auto-update-to
139-
# Temporary skip failing auto-update-to tests
134+
135+
136+
# TODO(COMPASS-9067): auto-update-from does not work for windows. Not
137+
# even manually. It downloads the releases file, the toast appears
138+
# that says it is downloading the update, the log appears that says
139+
# thatit is downloading the update, but it never makes the request to
140+
# the update server to actually download the .nupkg file. Strangely
141+
# auto-update-to works. (which starts with the latest release and then
142+
# updates to the newly packaged one) and auto-updating from an old dev
143+
# version also works. it seems to be the fact that it is actually
144+
# downgrading from the new package to the most recent release that
145+
# breaks it? Can't see any other difference.
146+
- test: auto-update-from
147+
package: windows_setup
148+
149+
# Skip auto-update-to tests for anything except the one release
150+
# package we install for that platform and arch, because that's the
151+
# only one that gets installed and therefore updated.
152+
# See https://github.com/10gen/compass-mongodb-com/blob/ae1ce87c54ab8702e94b154ffe2bfb9997b56661/src/routes/update.js#L320-L411
153+
# ie. only leave osx_dmg, windows_setup, linux_deb, linux_rpm
140154
- test: auto-update-to
141-
package: osx_dmg
155+
package: osx_zip
142156
- test: auto-update-to
143157
package: windows_zip
144158
- test: auto-update-to
145159
package: windows_msi
146160
- test: auto-update-to
147-
package: windows_setup
148-
# Waiting for https://github.com/10gen/compass-mongodb-com/pull/122 to be released
149-
- test: auto-update-to
150-
package: linux_deb
161+
package: linux_tar
151162
- test: auto-update-to
152163
package: linux_rpm
153164

@@ -221,3 +232,11 @@ jobs:
221232
working-directory: packages/compass-smoke-tests
222233
# Using --skipUninstalling --skipCleanup because the runners are ephemeral
223234
run: npm start -- --package ${{ matrix.package }} --tests ${{ matrix.test }} --skipUninstall --skipCleanup
235+
236+
- name: 'Upload logs'
237+
uses: actions/upload-artifact@v4
238+
if: always()
239+
with:
240+
name: ${{ matrix.hadron-distribution}}-${{ matrix.hadron-platform}}-${{ matrix.arch }}-${{ matrix.package }}-${{ matrix.test }}-logs
241+
path: packages/compass-e2e-tests/.log/
242+
include-hidden-files: true

packages/compass-e2e-tests/tests/auto-update.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ describe('Auto-update', function () {
7171
console.log(
7272
'pause to make sure the app properly exited before starting again'
7373
);
74-
await wait(10_000);
74+
await wait(60_000);
7575

7676
console.log('starting compass a second time');
7777
// run the app again and check that the version changed

0 commit comments

Comments
 (0)