Skip to content

Commit f142fa5

Browse files
committed
Skip POSIX-only PATH-resolution test on Windows
shutil.which() requires a PATHEXT-listed extension on Windows, so the shebang-script fixture used by test_stdio_resolves_bare_command_via_path isn't found. Skip the test on win32; the underlying code path is exercised by the macOS and Linux runners.
1 parent 5beaa53 commit f142fa5

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

tests/test_sdk_copilot_mcp.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@
55

66
from __future__ import annotations
77

8+
import sys
9+
10+
import pytest
11+
812
from seclab_taskflow_agent.sdk.base import MCPServerSpec
913
from seclab_taskflow_agent.sdk.copilot_sdk.mcp import build_mcp_config
1014

@@ -92,6 +96,8 @@ def test_stdio_minimal_defaults():
9296

9397

9498
def test_stdio_resolves_bare_command_via_path(tmp_path, monkeypatch):
99+
if sys.platform == "win32":
100+
pytest.skip("PATHEXT-based shutil.which lookup is POSIX-specific in this test")
95101
bin_dir = tmp_path / "bin"
96102
bin_dir.mkdir()
97103
fake = bin_dir / "fakecmd"

0 commit comments

Comments
 (0)