Skip to content

test: fix Python 3.10 Tests CI (surge namespace-shadow test)#38

Merged
qian-harvard merged 1 commit into
mainfrom
ci/fix-py310-surge-test
Jun 11, 2026
Merged

test: fix Python 3.10 Tests CI (surge namespace-shadow test)#38
qian-harvard merged 1 commit into
mainfrom
ci/fix-py310-surge-test

Conversation

@qian-harvard

Copy link
Copy Markdown
Contributor

Fixes the failing Tests workflow on the py3.10 matrix leg.

test_namespace_shadow_not_false_positive expected surge to report red (missing), but on Python 3.10/3.11 _dep_status short-circuits to the needs Python 3.12-3.14 gate (yellow) before reaching the namespace-shadow probe - so the assertion failed on 3.10 while passing on 3.12. The test now bypasses the gate via monkeypatch so it exercises the probe path on every Python version. The generic probe_installed namespace logic remains covered by test_runner.py.

?? Generated with Claude Code

test_namespace_shadow_not_false_positive asserted surge reports "missing"
(red), but on Python 3.10/3.11 _dep_status short-circuits to the
"needs Python 3.12-3.14" gate (yellow) before reaching the namespace-shadow
probe, so the test failed on the py3.10 matrix leg. Bypass the gate via
monkeypatch so the probe path is exercised on every Python version; the
generic probe_installed namespace logic stays covered by test_runner.py.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@qian-harvard qian-harvard merged commit af8162f into main Jun 11, 2026
@qian-harvard qian-harvard deleted the ci/fix-py310-surge-test branch June 11, 2026 00:13
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.

1 participant