the_og/better-claude-config
1
0
Fork 0
Code Issues 8 Pull Requests 3 Actions Packages Projects Releases 4 Wiki Activity

test: dep-check probe command must exist on Windows too
CI / Lint (ruff) (pull_request) Successful in 6s
Details
CI / Tests (py3.12 / windows-latest) (pull_request) Successful in 17s
Details
CI / Tests (py3.10 / ubuntu-latest) (pull_request) Successful in 8s
Details
CI / Tests (py3.12 / ubuntu-latest) (pull_request) Successful in 8s
Details
CI / Tests (py3.13 / ubuntu-latest) (pull_request) Successful in 8s
Details

Browse Source 
python3 is not a command on a stock Windows install; caught by the new
windows-latest CI job. Probe 'python' there and accept warn (found on
augmented PATH) as proof of resolution.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
Cowork Supervisor
2026-07-12 13:05:04 -04:00
parent 62c8a2ea65
commit 87303809b8
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/test_core.py
+5 -2
View File
@@ -227,8 +227,11 @@ def test_diagnostics_redacts_token_args():
# --------------------------------------------------------------------------- # # --------------------------------------------------------------------------- #
# 6. Dependency / PATH checking # 6. Dependency / PATH checking
# --------------------------------------------------------------------------- # # --------------------------------------------------------------------------- #
def test_dep_check_finds_python3(): def test_dep_check_finds_python():
assert c.check_dependency({"command": "python3"})["status"] == "ok" # "python3" does not exist on a stock Windows install; "warn" (found only
# on an augmented PATH) still proves resolution works on a real machine.
cmd = "python" if os.name == "nt" else "python3"
assert c.check_dependency({"command": cmd})["status"] in ("ok", "warn")
def test_dep_check_flags_missing(): def test_dep_check_flags_missing():