From 0774f3b58a69076dbfaed50a02c10e5df2d0b629 Mon Sep 17 00:00:00 2001 From: firelight flagboy Date: Fri, 21 Apr 2023 09:01:08 +0200 Subject: [PATCH] Re-enable web unit test The unit-test seems to work just fine Closes #4394 --- .github/workflows/ci-web.yml | 27 ++++++++++++++------------- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/.github/workflows/ci-web.yml b/.github/workflows/ci-web.yml index 5981ae75417..8838df9bc42 100644 --- a/.github/workflows/ci-web.yml +++ b/.github/workflows/ci-web.yml @@ -62,12 +62,23 @@ jobs: timeout-minutes: 5 - name: Install web bindings - # Currently disable because `E2E tests & Unit tests` are. - if: false && steps.should-run-web-jobs.outputs.run == 'true' - run: npm run build:ci + if: steps.should-run-web-jobs.outputs.run == 'true' + run: | + # Execute 'npm clean-install' until success, + # This is done that way because sometime some CDN response with 503 + until npm clean-install; do + echo "Failed clean-install, retrying ..."; + done + npm run build:ci working-directory: oxidation/bindings/web timeout-minutes: 15 + - name: Unit tests + if: steps.should-run-web-jobs.outputs.run == 'true' + run: npm run test:unit + working-directory: oxidation/client + timeout-minutes: 10 + - name: E2E tests # Currently disable because of #4393 if: false && steps.should-run-web-jobs.outputs.run == 'true' @@ -77,13 +88,3 @@ jobs: run: VUE_CLI_TEST=1 npm run test:e2e:headless working-directory: oxidation/client timeout-minutes: 10 - - # Need to run e2e first, as it will trigger compilation of the - # libparsec web plugin that is also needed for the unit tests (but - # which doesn't trigger it compilation...) - - name: Unit tests - # Currently disabled because of #3494 - if: false && steps.should-run-web-jobs.outputs.run == 'true' - run: npm run test:unit - working-directory: oxidation/client - timeout-minutes: 10