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 f93a867
Showing 1 changed file with 7 additions and 12 deletions.
19 changes: 7 additions & 12 deletions .github/workflows/ci-web.yml
Original file line number Diff line number Diff line change
Expand Up @@ -62,12 +62,17 @@ 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'
if: steps.should-run-web-jobs.outputs.run == 'true'
run: 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 +82,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 f93a867

Please sign in to comment.