Skip to content

Commit d6fd5f9

Browse files
rootclaude
andcommitted
ci: actually run pytest
The workflow is named "Tests" and has never run one — it installed ruff and pyright, linted and type-checked tests/, and stopped there. tests/test_base_url.py has existed since 0.3.x without ever executing in CI. #18 asks for a fixture round-trip 'so model drift fails CI'. That only works if the suite runs. Also fixes the pyright failure this PR surfaced: 'import pytest could not be resolved', because the test extra was never installed. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 parent f996fec commit d6fd5f9

1 file changed

Lines changed: 9 additions & 1 deletion

File tree

.github/workflows/test.yml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,19 @@ jobs:
2222

2323
- name: Install dependencies
2424
run: |
25-
pip install -e ".[pandas]"
25+
pip install -e ".[pandas,test]"
2626
pip install ruff pyright
2727
2828
- name: Ruff check
2929
run: ruff check
3030

3131
- name: Pyright
3232
run: pyright
33+
34+
# This workflow is named "Tests" but never ran any until now — `tests/` has
35+
# existed since 0.3.x and CI only linted and type-checked it. #18 shipped two
36+
# model/wire mismatches that a single parse of a real payload would have
37+
# caught, so the fixture round-trip it asks for is only worth adding if it can
38+
# actually fail the build.
39+
- name: Pytest
40+
run: pytest -q

0 commit comments

Comments
 (0)