Skip to content

Honor exact pytest nodes in the single-test workflow - #803

Open
xiehuanyi wants to merge 1 commit into
amoffat:developfrom
xiehuanyi:fix/exact-test-selection
Open

xiehuanyi wants to merge 1 commit into
amoffat:developfrom
xiehuanyi:fix/exact-test-selection

Conversation

@xiehuanyi

Copy link
Copy Markdown

The README's make test='FunctionalTests.test_background' test_one command passes a dotted unittest name to pytest, which exits with “file or directory not found.” Replacing it with a valid pytest node alone still collects the entire suite because tox prepends tests/ to every argument list.

Use tests/ as tox's default only when no positional arguments are supplied. Document the exact pytest node for the single-test target and the current uv sync development setup. The default full-suite target continues to select tests/.

Validation on Python 3.14.6/Linux: the original README selector exits 4; a valid exact node with the old prefix collects all 190 tests. The updated representative py314-locale-c-poller-poll tox invocation selects and passes exactly one requested test. Its full default run passes 187 tests with 3 skips. README rstcheck, Ruff lint/format and git diff --check pass. The other Python/platform/locale/backend combinations were not exercised; existing pytest warnings remain.

AI disclosure: OpenAI Codex assisted with investigation, implementation, command verification, and this description.

AI-assisted implementation and verification using OpenAI Codex.
Copilot AI balanced review requested due to automatic review settings October 10, 2026 18:50

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

This branch has not been deployed

No deployments
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.

2 participants