Skip to content

Commit 47e8027

Browse files
authored
ci: run all unit tests in the mocked-API job (#45)
The mocked-API CI job invoked pytest with an explicit file list (`tests/tests_mocked_api.py tests/test_batch_job_builder.py`), which silently omits any new test module added to the suite and has needed hand-editing more than once. Replace it with a directory invocation that runs everything under `tests/` except the live-AWS suite (which needs real credentials): ``` pytest tests/ --ignore=tests/tests_true_api.py -v ``` All workflow services and environment variables are preserved; only the pytest selection changes. On `main` today this selects the same two files — it future-proofs the invocation against the recurring list-rot. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Tests** * Updated the continuous integration test run to execute the full test suite, while skipping one API-dependent test file. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 3b26b40 commit 47e8027

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/ci_mocked_api.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ jobs:
102102
env:
103103
SNAKEMAKE_AWS_BATCH_JOB_QUEUE: "${{ secrets.SNAKEMAKE_AWS_BATCH_JOB_QUEUE }}"
104104
SNAKEMAKE_AWS_BATCH_JOB_ROLE: "${{ secrets.SNAKEMAKE_AWS_BATCH_JOB_ROLE }}"
105-
run: poetry run coverage run -m pytest tests/tests_mocked_api.py tests/test_batch_job_builder.py -v
105+
run: poetry run coverage run -m pytest tests/ --ignore=tests/tests_true_api.py -v
106106

107107
- name: Run Coverage
108108
run: poetry run coverage report -m

0 commit comments

Comments
 (0)