Skip to content

Add isolated CI coverage for extras and guard optional-dependency tests#895

Merged
ftnext merged 2 commits into
Uberi:masterfrom
ftnext:codex/extra-ci-verification
Apr 23, 2026
Merged

Add isolated CI coverage for extras and guard optional-dependency tests#895
ftnext merged 2 commits into
Uberi:masterfrom
ftnext:codex/extra-ci-verification

Conversation

@ftnext
Copy link
Copy Markdown
Collaborator

@ftnext ftnext commented Apr 22, 2026

Summary

  • split unit test CI into a base job, an all-extras regression job, and per-extra contract checks
  • verify each extra in a fresh environment so missing transitive dependencies are not masked by other extras
  • make optional-dependency tests skip cleanly when their extra is not installed

Background

This follows the dependency mismatch discussed in #855 and fixed in #856.

That issue was easy to miss because CI mainly exercised an all-extras environment. In that setup, dependencies installed by one extra can mask missing dependencies in another extra. In the faster-whisper case, soundfile could be present because of other installs, so the faster-whisper packaging contract was not being verified in isolation.

What This Changes

  • add a base job that installs only .[dev]
  • keep the existing all-extras regression coverage
  • add extra-contracts jobs that create a fresh environment per extra and run targeted verification
  • update optional-dependency tests to skip cleanly when their extra is not installed

Testing

  • git diff --check
  • python3 -m compileall tests speech_recognition

A full local pytest run in a fresh CI-like environment was not possible from this worktree, so the new workflow is intended to validate the matrix in CI.

@ftnext
Copy link
Copy Markdown
Collaborator Author

ftnext commented Apr 23, 2026

codex

The changes add CI coverage for base and per-extra installations and make optional-dependency tests skip when their corresponding extras are not installed.
I did not find any discrete issues in the diff that would break existing tests or workflows.

@ftnext ftnext merged commit 8bcaf0b into Uberi:master Apr 23, 2026
21 checks passed
@ftnext ftnext deleted the codex/extra-ci-verification branch April 23, 2026 14:54
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