Skip to content

ci: fail the pytest job when an integration cannot be imported - #259

Merged
lesnik512 merged 1 commit into
mainfrom
ci/pytest-require-integrations
Oct 3, 2026
Merged

lesnik512 merged 1 commit into
mainfrom
ci/pytest-require-integrations

Conversation

@lesnik512

Copy link
Copy Markdown
Member

Summary

Follow-up to #258, which added REQUIRE_INTEGRATIONS and set it only in the floors job. The pytest job also installs every integration (just install runs uv sync --all-extras, and the dev group has litestar and faststream[nats]). That means a failed import there would also skip a whole integration suite and the job would stay green. This matters most on the daily scheduled run: it resolves the newest releases, so it is where a new faststream, pydantic or litestar release that breaks the import would show up first.

Changes

  • The pytest job sets REQUIRE_INTEGRATIONS=1 on its test step, the same as the floors job.

Coverage is unaffected: import_or_skip is a single expression, so the line is covered either way.

Checklist

  • Lint and format pass (ruff): no Python changed
  • Type check passes (mypy and pyrefly): no Python changed
  • Tests pass and new behavior is covered: both matrix variants (faststream<0.6.0, faststream>=0.6.0) replayed locally on 3.12 with the variable set: 458 passed, 0 skipped, coverage 100%
  • Build succeeds (uv build) if packaging or build config changed: not applicable
  • Docs updated if behavior or public API changed: not applicable
  • Repo metadata stays consistent across the three surfaces: not applicable

@codecov

codecov Bot commented Oct 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag Coverage Δ
unittests 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lesnik512
lesnik512 merged commit 1f485a0 into main Oct 3, 2026
18 checks passed
@lesnik512
lesnik512 deleted the ci/pytest-require-integrations branch October 3, 2026 09:49
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