Skip to content

Use active interpreter for module entrypoint version test#56

Merged
srichs merged 1 commit intomainfrom
codex/fix-test_cli.py-failure-in-test_module_entrypoint_supports_v
Feb 17, 2026
Merged

Use active interpreter for module entrypoint version test#56
srichs merged 1 commit intomainfrom
codex/fix-test_cli.py-failure-in-test_module_entrypoint_supports_v

Conversation

@srichs
Copy link
Owner

@srichs srichs commented Feb 17, 2026

Motivation

  • Replace the hard-coded python executable in the module-entrypoint version test so the subprocess invocation uses the same interpreter running the tests and avoids platform-specific resolution failures.

Description

  • Add import sys and change the subprocess invocation in tests/test_cli.py::test_module_entrypoint_supports_version_flag to use sys.executable instead of the literal "python".

Testing

  • Ran pytest -q tests/test_cli.py::test_module_entrypoint_supports_version_flag tests/test_cli.py::test_version_flag_prints_version which passed.
  • Ran pytest -q which completed successfully with all tests passing (93 passed).

Codex Task

@srichs srichs merged commit 3a1ef48 into main Feb 17, 2026
9 checks passed
@srichs srichs deleted the codex/fix-test_cli.py-failure-in-test_module_entrypoint_supports_v branch February 17, 2026 03:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant