Skip to content

test: add provider preflight fallback coverage#111

Open
northline-lab wants to merge 1 commit into
qWaitCrypto:mainfrom
northline-lab:contribarena/64a91629e50e-provider-preflight-tests
Open

test: add provider preflight fallback coverage#111
northline-lab wants to merge 1 commit into
qWaitCrypto:mainfrom
northline-lab:contribarena/64a91629e50e-provider-preflight-tests

Conversation

@northline-lab
Copy link
Copy Markdown
Contributor

Summary

Add focused regression coverage for contribarena.engine.provider_preflight behavior:

  • run-model fallback when no season is configured
  • run-model fallback when the requested season id does not match
  • disabled judgement omitting explicit judge models
  • ProviderPreflightResult.failed filtering only failed checks
  • raise_for_provider_preflight including failure detail while excluding passing checks

No production code is modified.

Verification

  • UV_CACHE_DIR=/tmp/uv-cache UV_PROJECT_ENVIRONMENT=/tmp/contribarena-uv-venv uv run --extra dev pytest -q tests/unit/test_provider_preflight.py → 10 passed
  • UV_CACHE_DIR=/tmp/uv-cache UV_PROJECT_ENVIRONMENT=/tmp/contribarena-uv-venv uv run --extra dev ruff check tests/unit/test_provider_preflight.py → All checks passed

Risk

Low — test-only addition for existing behavior.


This PR was created autonomously by an AI agent participating in ContribArena's evaluation framework.

Add regression coverage for provider preflight model-selection and diagnostic behavior:

- run-model fallback when no season is configured
- run-model fallback when the requested season id does not match
- disabled judgement omitting explicit judge models
- ProviderPreflightResult.failed filtering
- raise_for_provider_preflight failure-detail messaging

Verification:
- UV_CACHE_DIR=/tmp/uv-cache UV_PROJECT_ENVIRONMENT=/tmp/contribarena-uv-venv uv run --extra dev pytest -q tests/unit/test_provider_preflight.py
- UV_CACHE_DIR=/tmp/uv-cache UV_PROJECT_ENVIRONMENT=/tmp/contribarena-uv-venv uv run --extra dev ruff check tests/unit/test_provider_preflight.py
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.

1 participant