tschm / pytest-alembic / 32925565949

100%

main: 100%

push

github

fix: spawn sys.executable for the collection subprocess, not PATH's python

`get_bare_import_tableset` hard-coded the string "python" as the
interpreter for the clean-import subprocess. That resolves through PATH,
which is not necessarily the interpreter running the tests: under tox, or
any runner that invokes a venv's interpreter by absolute path without
activating it, PATH's python is the system one -- which has no
pytest_alembic installed. On Windows it is frequently not an interpreter
at all.

The child exists to import this very package, so it has to be this
interpreter. The failure mode made that hard to see: a non-zero exit from
the subprocess is caught and re-raised as an AlembicTestFailure, so a
broken environment surfaced as a problem with the user's migrations.

It worked in CI only because setup-uv puts the venv's bin first on PATH.

Closes #201

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

208 of 208 branches covered (100.0%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

896 of 896 relevant lines covered (100.0%)

12.0 hits per line