Skip to content

Commit bf617f2

Browse files
kingpanther13claude
andcommitted
test: retire the precedence rule from two test docstrings
Both tests survived the pool merge on their assertions but still taught the deleted named-over-synthesized ranking: the upper-bound test now states the one-admitted-probe rule its innocence actually rests on (the floor is one probe of four), and the inclusive-ceiling test matches the source's wording, which no longer calls that probe "named" (Patch76 review). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QiuyTYBQ2opBVPp8xzMq3r
1 parent d001cc1 commit bf617f2

1 file changed

Lines changed: 10 additions & 10 deletions

File tree

tests/src/unit/test_dependency_diagnostics.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -406,22 +406,22 @@ def test_wildcard_floor_still_probes(self):
406406
assert requirement_forces_conflict("mcp==1.9.0", wildcard)
407407

408408
def test_upper_bound_admitting_the_named_floor_is_innocent(self):
409-
"""Patch76 review on #2245: a named probe outranks a synthesized one.
409+
"""Patch76 review on #2245: one admitted probe proves innocence.
410410
411-
``mcp<=1.24.0`` admits the violated spec's own floor, so enforcing
412-
it can never hold the package below that floor — blaming it for
413-
rejecting the invented successor 1.24.0.0.1 pointed the user at an
414-
integration that cannot be the culprit.
411+
``mcp<=1.24.0`` admits the violated spec's own floor — one probe of
412+
the four that survive the filter — so enforcing it can never hold
413+
the package below that floor, and blaming it would point the user
414+
at an integration that cannot be the culprit.
415415
"""
416416
assert not requirement_forces_conflict("mcp<=1.24.0", self._VIOLATION)
417417

418418
def test_inclusive_ceiling_probe_cannot_convict_alone(self):
419-
"""Patch76 on #2245: an inclusive ceiling survives as a named probe.
419+
"""Patch76 on #2245: an inclusive ceiling survives as a probe.
420420
421-
Under ``mcp<=2.0,>=1.24.0`` the probes are 1.24.0 AND 2.0; demanding
422-
both blamed ``mcp<=1.24.0`` for the ceiling although it admits the
423-
floor and can never hold the package below it. One admitted probe
424-
proves innocence.
421+
Under ``mcp<=2.0,>=1.24.0`` the probes include 1.24.0 AND 2.0;
422+
demanding every one blamed ``mcp<=1.24.0`` for the ceiling although
423+
it admits the floor and can never hold the package below it. One
424+
admitted probe proves innocence.
425425
"""
426426
inclusive_ceiling = DependencyViolation(
427427
package="mcp",

0 commit comments

Comments
 (0)