Skip to content

Commit a612118

Browse files
authored
[ci] revise wasm CI (microsoft#24825)
### Description <!-- Describe your changes. --> revise WASM CI to run test as later step than publishing artifacts. This allows download the binary to diagnose test failures. ### Motivation and Context <!-- - Why is this change required? What problem does it solve? - If it fixes an open issue, please link to the issue here. -->
1 parent f57db79 commit a612118

File tree

1 file changed

+22
-21
lines changed

1 file changed

+22
-21
lines changed

.github/workflows/linux-wasm-ci-build-and-test-workflow.yml

Lines changed: 22 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -65,27 +65,6 @@ jobs:
6565
--skip_tests
6666
working-directory: ${{ github.workspace }}
6767

68-
- name: Test (Node.js) (simd + threads)
69-
# onnxruntime_test_all is currently only supported in Debug build because it requires exception, which is disabled in Release build.
70-
if: ${{ inputs.build_config == 'Debug' }}
71-
run: |
72-
python ./tools/ci_build/build.py \
73-
${{ env.common_build_args }} \
74-
--build_dir ${{ github.workspace }}/build/wasm_inferencing \
75-
--test
76-
working-directory: ${{ github.workspace }}
77-
78-
- name: Test (browser) (simd + threads)
79-
# onnxruntime_test_all is currently only supported in Debug build because it requires exception, which is disabled in Release build.
80-
if: ${{ inputs.build_config == 'Debug' }}
81-
run: |
82-
python ./tools/ci_build/build.py \
83-
${{ env.common_build_args }} \
84-
--build_dir ${{ github.workspace }}/build/wasm_inferencing \
85-
--wasm_run_tests_in_browser \
86-
--test
87-
working-directory: ${{ github.workspace }}
88-
8968
- name: Build (simd + threads + JSEP)
9069
if: ${{ inputs.build_jsep == true }}
9170
run: |
@@ -143,6 +122,28 @@ jobs:
143122
name: ${{ inputs.build_config }}_wasm_webgpu
144123
path: ${{ github.workspace }}/artifacts/wasm_webgpu
145124

125+
- name: Test (Node.js) (simd + threads)
126+
# onnxruntime_test_all is currently only supported in Debug build because it requires exception, which is disabled in Release build.
127+
if: ${{ inputs.build_config == 'Debug' }}
128+
run: |
129+
python ./tools/ci_build/build.py \
130+
${{ env.common_build_args }} \
131+
--build_dir ${{ github.workspace }}/build/wasm_inferencing \
132+
--test
133+
working-directory: ${{ github.workspace }}
134+
135+
- name: Test (browser) (simd + threads)
136+
# onnxruntime_test_all is currently only supported in Debug build because it requires exception, which is disabled in Release build.
137+
if: ${{ inputs.build_config == 'Debug' }}
138+
run: |
139+
python ./tools/ci_build/build.py \
140+
${{ env.common_build_args }} \
141+
--build_dir ${{ github.workspace }}/build/wasm_inferencing \
142+
--wasm_run_tests_in_browser \
143+
--target onnxruntime_test_all \
144+
--update --build --test
145+
working-directory: ${{ github.workspace }}
146+
146147
- name: Publish test results
147148
if: ${{ always() && inputs.build_config == 'Debug' }}
148149
uses: actions/upload-artifact@v4

0 commit comments

Comments
 (0)