Skip to content

Commit d527257

Browse files
authored
Fix test failures due to collecting 0 tests (#485)
## Summary <!-- Include a short paragraph of the changes introduced in this PR. If this PR requires additional context or rationale, explain why the changes are necessary. --> Fixes the current E2E failures on main. See #482 for more details. ## Test Plan <!-- List the steps needed to test this PR. --> Ran CI with patch applied; [result here](https://github.com/vllm-project/guidellm/actions/runs/19650016859/job/56274540060#step:7:195). ## Related Issues <!-- Link any relevant issues that this PR addresses. --> - Resolves #482 --- - [x] "I certify that all code in this PR is my own, except as noted below." ## Use of AI - [ ] Includes AI-assisted code completion - [ ] Includes code generated by an AI application - [ ] Includes AI-generated tests (NOTE: AI written tests should have a docstring that includes `## WRITTEN BY AI ##`)
2 parents 766e6ef + 904b6a3 commit d527257

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/actions/run-tox/action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ runs:
2323
shell: bash
2424
- name: Run tox
2525
run: |
26-
tox run -e "${{ inputs.tox-env }}" -- ${{ inputs.tox-args }}
26+
tox run -e "${{ inputs.tox-env }}" -- ${{ inputs.tox-args }} || ret=$?
2727
# errno 5 means all tests were filtered out, ignore
28-
ret=$? && [[ $ret -eq 5 ]] && exit 0; exit $ret
28+
[[ "${ret:=0}" -eq 5 ]] && exit 0; exit $ret
2929
shell: bash

0 commit comments

Comments
 (0)