Fix pytest tests to exit with proper exit code (#16681)

This commit is contained in:
Alison Shao
2026-01-07 15:20:46 -08:00
committed by GitHub
parent 6037267f5b
commit 3a4767daa3
8 changed files with 22 additions and 8 deletions

View File

@@ -29,7 +29,7 @@ python3 test_choices.py
- Create new test files under `test/srt` or `test/lang` depending on the type of test.
- For nightly tests, place them in `test/srt/nightly/`. Use the `NightlyBenchmarkRunner` helper class in `nightly_utils.py` for performance benchmarking tests.
- Ensure they are referenced in the respective `run_suite.py` (e.g., `test/srt/run_suite.py`) so they are picked up in CI. For most small test cases, they can be added to the `per-commit-1-gpu` suite. Sort the test cases alphabetically by name.
- Ensure you added `unittest.main()` for unittest and `pytest.main([__file__])` for pytest in the scripts. The CI run them via `python3 test_file.py`.
- Ensure you added `unittest.main()` for unittest and `sys.exit(pytest.main([__file__]))` for pytest in the scripts. The CI run them via `python3 test_file.py`.
- The CI will run some suites such as `per-commit-1-gpu`, `per-commit-2-gpu`, and `nightly-1-gpu` automatically. If you need special setup or custom test groups, you may modify the workflows in [`.github/workflows/`](https://github.com/sgl-project/sglang/tree/main/.github/workflows).
## CI Registry Quick Peek

View File

@@ -78,4 +78,6 @@ def test_unknown_tool_name_forwarded(caplog):
if __name__ == "__main__":
pytest.main([__file__])
import sys
sys.exit(pytest.main([__file__]))

View File

@@ -380,4 +380,6 @@ def test_causal_conv1d_varlen(
if __name__ == "__main__":
pytest.main([__file__])
import sys
sys.exit(pytest.main([__file__]))

View File

@@ -144,4 +144,6 @@ def mixer2_gated_norm_tensor_parallel(
if __name__ == "__main__":
pytest.main([__file__])
import sys
sys.exit(pytest.main([__file__]))

View File

@@ -297,4 +297,6 @@ def test_selective_state_update_with_heads_with_batch_indices(
if __name__ == "__main__":
pytest.main([__file__])
import sys
sys.exit(pytest.main([__file__]))

View File

@@ -607,4 +607,6 @@ def test_mamba_chunk_scan_cont_batch_prefill_chunking(chunk_size, seqlens):
if __name__ == "__main__":
pytest.main([__file__])
import sys
sys.exit(pytest.main([__file__]))

View File

@@ -51,4 +51,6 @@ def test_replace_offsets_with_tokens_per_frame():
if __name__ == "__main__":
pytest.main([__file__])
import sys
sys.exit(pytest.main([__file__]))

View File

@@ -56,4 +56,6 @@ def test_sample_video_frames_lengths(case: Case):
assert result == case.expected_frames
if __name__ == "__main__":
pytest.main([__file__])
import sys
sys.exit(pytest.main([__file__]))