Fix tests on Windows.

Author: tobiasraabe

src/_pytask/build.py

@@ -275,7 +275,7 @@ def build(  # noqa: C901, PLR0912, PLR0913
             session.exit_code = ExitCode.FAILED
 
         except Exception:  # noqa: BLE001
-            console.print(Traceback(sys.exc_info()))
+            console.print(Traceback(sys.exc_info(), show_locals=True))
             session.exit_code = ExitCode.FAILED
 
         session.hook.pytask_unconfigure(session=session)

tests/conftest.py

@@ -121,15 +121,22 @@ class Result(NamedTuple):
     stderr: str
 
 
-def run_in_subprocess(
-    cmd: tuple[str, ...], cwd: Path | None = None, env: dict[str, str] | None = None
-) -> Result:
+def run_in_subprocess(cmd: tuple[str, ...], cwd: Path | None = None) -> Result:
     """Run a command in a subprocess and return the output."""
-    result = subprocess.run(cmd, cwd=cwd, check=False, capture_output=True, env=env)
+    kwargs = (
+        {
+            "env": os.environ | {"PYTHONIOENCODING": "utf-8", "TERM": "unknown"},
+            "encoding": "utf-8",
+        }
+        if sys.platform == "win32"
+        else {}
+    )
+
+    result = subprocess.run(
+        cmd, cwd=cwd, check=False, capture_output=True, text=True, **kwargs
+    )
     return Result(
-        exit_code=result.returncode,
-        stdout=result.stdout.decode("utf-8", "replace").replace("\r\n", "\n"),
-        stderr=result.stderr.decode("utf-8", "replace").replace("\r\n", "\n"),
+        exit_code=result.returncode, stdout=result.stdout, stderr=result.stderr
     )
 
 

tests/test_config.py

@@ -120,7 +120,7 @@ def test_paths_are_relative_to_configuration_file(tmp_path):
     session = build(paths=[Path("src")])
     """
     tmp_path.joinpath("script.py").write_text(textwrap.dedent(source))
-    result = run_in_subprocess(("python", "script.py"), cwd=tmp_path)
+    result = run_in_subprocess(("uv", "run", "python", "script.py"), cwd=tmp_path)
     assert result.exit_code == ExitCode.OK
     assert "1  Succeeded" in result.stdout
 

tests/test_execute.py

@@ -20,13 +20,16 @@
 from pytask import build
 from pytask import cli
 from tests.conftest import enter_directory
+from tests.conftest import run_in_subprocess
 
 
-@pytest.mark.xfail(sys.platform == "win32", reason="See #293.")
 @pytest.mark.end_to_end
 def test_python_m_pytask(tmp_path):
     tmp_path.joinpath("task_module.py").write_text("def task_example(): pass")
-    subprocess.run(["python", "-m", "pytask", tmp_path.as_posix()], check=False)
+    result = run_in_subprocess(
+        ("uv", "run", "python", "-m", "pytask", tmp_path.as_posix())
+    )
+    assert result.exit_code == ExitCode.OK
 
 
 @pytest.mark.end_to_end
@@ -657,10 +660,10 @@ def task2() -> None: pass
         sys.exit(session2.exit_code)
     """
     tmp_path.joinpath("task_module.py").write_text(textwrap.dedent(source))
-    result = subprocess.run(
-        ("python", tmp_path.joinpath("task_module.py").as_posix()), check=False
+    result = run_in_subprocess(
+        ("uv", "run", "python", tmp_path.joinpath("task_module.py").as_posix())
     )
-    assert result.returncode == ExitCode.OK
+    assert result.exit_code == ExitCode.OK
 
 
 @pytest.mark.end_to_end
@@ -765,14 +768,14 @@ def task_example() -> Annotated[str, Path("file.txt")]:
     """
     tmp_path.joinpath("task_example.py").write_text(textwrap.dedent(source))
 
-    result = subprocess.run(("pytask"), cwd=tmp_path)  # noqa: PLW1510
-    assert result.returncode == ExitCode.OK
+    result = run_in_subprocess(("pytask",), cwd=tmp_path)
+    assert result.exit_code == ExitCode.OK
 
     hashes = json.loads(tmp_path.joinpath(".pytask", "file_hashes.json").read_text())
     assert len(hashes) == 2
 
-    result = subprocess.run(("pytask"), cwd=tmp_path)  # noqa: PLW1510
-    assert result.returncode == ExitCode.OK
+    result = run_in_subprocess(("pytask",), cwd=tmp_path)
+    assert result.exit_code == ExitCode.OK
 
     hashes_ = json.loads(tmp_path.joinpath(".pytask", "file_hashes.json").read_text())
     assert hashes == hashes_

tests/test_hook_module.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import os
+from pathlib import Path
 import subprocess
 import sys
 import textwrap
@@ -41,6 +42,8 @@ def pytask_extend_command_line_interface(cli):
 
     if module_name:
         args = (
+            "uv",
+            "run",
             "python",
             "-m",
             "pytask",
@@ -50,7 +53,15 @@ def pytask_extend_command_line_interface(cli):
             "--help",
         )
     else:
-        args = ("pytask", "build", "--hook-module", "hooks/hooks.py", "--help")
+        args = (
+            "uv",
+            "run",
+            "pytask",
+            "build",
+            "--hook-module",
+            "hooks/hooks.py",
+            "--help",
+        )
 
     result = run_in_subprocess(args, cwd=tmp_path)
     assert result.exit_code == ExitCode.OK
@@ -89,9 +100,9 @@ def pytask_extend_command_line_interface(cli):
     tmp_path.joinpath("hooks", "hooks.py").write_text(textwrap.dedent(hooks))
 
     if module_name:
-        args = ("python", "-m", "pytask", "build", "--help")
+        args = ("uv", "run", "--no-project", "python", "-m", "pytask", "build", "--help")
     else:
-        args = ("pytask", "build", "--help")
+        args = ("uv", "run", "--no-project", "pytask", "build", "--help")
 
     result = run_in_subprocess(args, cwd=tmp_path)
     assert result.exit_code == ExitCode.OK

tests/test_warnings.py

@@ -127,7 +127,6 @@ def task_example():
 
 
 @pytest.mark.end_to_end
-@pytest.mark.xfail(sys.platform == "win32", reason="See #293.")
 @pytest.mark.parametrize("warning", ["DeprecationWarning", "PendingDeprecationWarning"])
 def test_deprecation_warnings_are_not_captured(tmp_path, warning):
     path_to_warn_module = tmp_path.joinpath("warning.py")
@@ -156,7 +155,7 @@ def warn_now():
     path_to_warn_module.write_text(textwrap.dedent(warn_module))
 
     # Cannot use runner since then warnings are not ignored by default.
-    result = run_in_subprocess(("pytask"), cwd=tmp_path)
+    result = run_in_subprocess(("uv", "run", "pytask"), cwd=tmp_path)
     assert result.exit_code == ExitCode.OK
     assert "Warnings" not in result.stdout
     assert "warning!!!" not in result.stdout