Skip to content

Commit

Permalink
Re-enable web unit test
Browse files Browse the repository at this point in the history
The unit-test seems to work just fine

Closes #4394
  • Loading branch information
FirelightFlagboy committed Apr 21, 2023
1 parent 15543dc commit 0774f3b
Showing 1 changed file with 14 additions and 13 deletions.
27 changes: 14 additions & 13 deletions .github/workflows/ci-web.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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'
Expand All @@ -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

0 comments on commit 0774f3b

Please sign in to comment.