Skip to content

Add pytest markers to separate fast tests from network/credential tests - #117

Closed
toakinrele wants to merge 1 commit into
CIROH-UA:mainfrom
toakinrele:pytest-fast-tests-115
Closed

toakinrele wants to merge 1 commit into
CIROH-UA:mainfrom
toakinrele:pytest-fast-tests-115

Conversation

@toakinrele

Copy link
Copy Markdown
Collaborator

Closes #115

Adds network and aws_creds pytest markers so the fast unit tests can be run separately from tests that need network access or AWS/GCS credentials.

  • Registered markers in pyproject.toml
  • conftest.py auto-tags tests based on which fixtures they use (download_gpkg, clean_s3_test, etc.), and auto-skips aws_creds
    tests when no AWS credentials are detected
  • Explicitly marked the handful of tests that hit the network directly without a fixture (test_hf2ds.py, some of test_trouterestarts.py)
  • Added a Makefile with test-fast / test-network / test-all targets

Local fast loop: make test-fast - 7 tests in ~3s instead of the full ~30 min suite, no credentials required.

@toakinrele toakinrele closed this Sep 4, 2026
@toakinrele
toakinrele deleted the pytest-fast-tests-115 branch September 4, 2026 18:27
@toakinrele

Copy link
Copy Markdown
Collaborator Author

Closing in favor of a new PR opened directly from CIROH-UA/forcingprocessor (branch pushed to upstream to allow AWS/GCS credential checks to run).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Short test suite

1 participant