Skip to content

Commit 706f17a

Browse files
wingologanek
authored andcommitted
upload_test_binaries clears testsuite before downloading artifact
The previous state was that the build_assemblyscript, build_c, and build_rust workflows upload tests/*/testsuite as an artifact, and then upload_test_binaries runs once for each of {rust,assemblyscript,c}, removing any wasm file in tests/*lang*/testsuite, downloading the artifact, unpacking it to tests/*lang*/testsuite, and adding and committing to prod/testsuite/base if any file differed. This PR implements a more simple and robust approach, which is to just have upload_test_binaries remove tests/*lang*/testsuite before unpacking the artifact.
1 parent 694cec5 commit 706f17a

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.github/workflows/compile-tests.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ jobs:
153153
run: git merge ${{ github.sha }}
154154

155155
- name: Remove existing binaries
156-
run: git rm --ignore-unmatch tests/${{ matrix.suite }}/testsuite/*.wasm
156+
run: rm -rf tests/${{ matrix.suite }}/testsuite
157157

158158
- name: Download ${{ matrix.suite }} test binaries
159159
uses: actions/download-artifact@v4
@@ -164,6 +164,6 @@ jobs:
164164
- name: Publish changes to consumer branch
165165
shell: bash
166166
run: |
167-
git add tests/${{ matrix.suite }}/testsuite/*.wasm -f
168-
git diff --quiet --cached || git commit -m "Update test binaries for ${{ matrix.suite }} test suite"
167+
git add -f tests/${{ matrix.suite }}/testsuite
168+
git diff --quiet || git commit -m "Update test binaries for ${{ matrix.suite }} test suite"
169169
git push

0 commit comments

Comments
 (0)