Skip to content

ci: don't run live-prod tests in CI; run pytest locally only#96

Merged
martinkersner merged 1 commit into
mainfrom
worktree-ci-no-prod-tests
Jun 30, 2026
Merged

ci: don't run live-prod tests in CI; run pytest locally only#96
martinkersner merged 1 commit into
mainfrom
worktree-ci-no-prod-tests

Conversation

@martinkersner

Copy link
Copy Markdown
Member

Summary

CI was failing only on the Python 3.14 job — not a 3.14 bug. The Test with pytest step was gated to if: matrix.python-version == '3.14', so 3.14 was the only version that actually ran the suite. Those tests hit live production endpoints and depend on prod data availability, so they fail nondeterministically in CI:

This PR removes the pytest step from .github/workflows/python-package.yml. CI now runs only black + flake8 across the 3.10–3.14 matrix and never touches the production API. The test suite is unchanged and stays runnable locally:

DATAMAXI_API_KEY=... python -m pytest tests/

Tests

  • No tests run in CI by design (this is the point of the change).
  • Locally, the suite still collects and runs against prod as before.

Known failures

  • None introduced. The previously-failing 3.14 job is the failure this removes.
  • The DATAMAXI_API_KEY repo secret is now unused by this workflow; left in place intentionally.

Closes #95

Tests hit live production endpoints and depend on prod data
availability (DEX 404s, premium 'no data found'). Drop the pytest
step from the workflow so CI runs only black + flake8 across the
matrix. Tests stay runnable locally via:
  DATAMAXI_API_KEY=... python -m pytest tests/

Closes #95
@martinkersner martinkersner self-assigned this Jun 30, 2026
@martinkersner martinkersner merged commit a9097a8 into main Jun 30, 2026
7 checks passed
@martinkersner martinkersner deleted the worktree-ci-no-prod-tests branch June 30, 2026 04:41
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.

CI: 3.14 job fails on live-API premium tests missing the _FLAKY_PROD_DATA_XFAIL marker

1 participant