Fix loadscope handling of colons in parameters - #1382
LouisDeconinck wants to merge 3 commits into
Conversation
pytest-dev/pytest#14758 (main, upcoming 9.2) replaced Node._nodeid with a structured NodeId stored in Node._id, so assigning item._nodeid no longer affects item.nodeid and xdist_group suffixes were silently dropped. When Node._id is present, override the NodeId's cached string form so item.nodeid (and reports, which share the object) carry the @group suffix as before.
|
Pushed a fix for the The failing The commit detects Note this affects |
Fixes #1335.
LoadScopeScheduling._split_scopepreviously split on the last::, so a parametrized value containing::could be mistaken for the test scope. Parse the module boundary first and only consider::separators before parametrization, preserving class-level grouping.Added regression coverage for colon-containing parameters and parameterized class methods.
Checks:
pytest -q testing(224 passed, 6 skipped, 10 xfailed)ruff checkandruff format --checkgit diff --check