Skip to content

Commit fd8acb7

Browse files
committed
Run tests sequentially: cargo test -j 1
I think it is possible that our signal handler set in the test can be overwritten by integration tests (under dir 'test'), which means the `SIGNALED` variable would never be set. We have a lock to prevent this issue in integration tests, but since this is a doc test, we cannot employ it here. ```rust /// Any test that alters signal handling must grab this mutex. pub static SIGNAL_MTX: Mutex<()> = Mutex::new(()); ``` So let's run tests sequentially and see if it will work.
1 parent 3931b02 commit fd8acb7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/actions/test/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,4 +34,4 @@ runs:
3434
3535
- name: test
3636
shell: bash
37-
run: ${{ inputs.SUDO }} $(which ${{ inputs.TOOL }}) test --target ${{ inputs.TARGET }}
37+
run: ${{ inputs.SUDO }} $(which ${{ inputs.TOOL }}) test --target ${{ inputs.TARGET }} -j 1

0 commit comments

Comments
 (0)