pyproject-api 1.11.3__tar.gz → 1.11.4__tar.gz

{pyproject_api-1.11.3 → pyproject_api-1.11.4}/.pre-commit-config.yaml

@@ -15,11 +15,11 @@ repos:
       - id: codespell
         additional_dependencies: ["tomli>=2.4.1"]
   - repo: https://github.com/tox-dev/tox-toml-fmt
-    rev: "v1.10.3"
+    rev: "v1.11.0"
     hooks:
       - id: tox-toml-fmt
   - repo: https://github.com/tox-dev/pyproject-fmt
-    rev: "v2.29.4"
+    rev: "v2.30.1"
     hooks:
       - id: pyproject-fmt
   - repo: https://github.com/astral-sh/ruff-pre-commit

{pyproject_api-1.11.3 → pyproject_api-1.11.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: pyproject-api
-Version: 1.11.3
+Version: 1.11.4
 Summary: API to interact with the python pyproject.toml based projects
 Project-URL: Changelog, https://github.com/tox-dev/pyproject-api/releases
 Project-URL: Homepage, https://pyproject-api.readthedocs.io

{pyproject_api-1.11.3 → pyproject_api-1.11.4}/src/pyproject_api/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '1.11.3'
-__version_tuple__ = version_tuple = (1, 11, 3)
+__version__ = version = '1.11.4'
+__version_tuple__ = version_tuple = (1, 11, 4)
 
 __commit_id__ = commit_id = None

{pyproject_api-1.11.3 → pyproject_api-1.11.4}/src/pyproject_api/_via_fresh_subprocess.py

@@ -73,6 +73,7 @@ class SubprocessFrontend(Frontend):
             stderr=PIPE,
             stdin=PIPE,
             universal_newlines=True,
+            errors="backslashreplace",
             cwd=self._root,
             env=env,
         )

{pyproject_api-1.11.3 → pyproject_api-1.11.4}/tests/test_frontend.py

@@ -42,8 +42,45 @@ def test_missing_backend(local_builder: Callable[[str], Path]) -> None:
     assert "ModuleNotFoundError: No module named " in exc.err
 
 
-@pytest.mark.parametrize(("stream", "output"), [("stdout", "out"), ("stderr", "err")])
-def test_large_command_with_backend_output(stream: str, output: str, local_builder: Callable[[str], Path]) -> None:
+@pytest.fixture
+def isolated_build_wheel() -> Callable[[Path, str, int], tuple[str, str]]:
+    script = dedent("""
+        import sys
+        from pathlib import Path
+        from pyproject_api import SubprocessFrontend
+
+        root = Path(sys.argv[1])
+        frontend = SubprocessFrontend(*SubprocessFrontend.create_args_from_folder(root)[:-1])
+        result = frontend.build_wheel(root / "dist", config_settings={"payload": "y" * int(sys.argv[3])})
+        print(result.wheel.name)
+        print(getattr(result, sys.argv[2]), end="")
+        """)
+
+    def _f(root: Path, output: str, payload_size: int) -> tuple[str, str]:
+        # Run the frontend in a child process so a hang fails on the timeout instead of stalling pytest.
+        result = subprocess.run(
+            [sys.executable, "-X", "utf8", "-c", script, str(root), output, str(payload_size)],
+            capture_output=True,
+            encoding="utf-8",
+            timeout=30,
+            check=True,
+        )
+        assert not result.stderr
+        wheel, captured = result.stdout.split("\n", 1)
+        return wheel, captured
+
+    return _f
+
+
+@pytest.mark.parametrize(
+    ("stream", "output"), [pytest.param("stdout", "out", id="stdout"), pytest.param("stderr", "err", id="stderr")]
+)
+def test_large_command_with_backend_output(
+    stream: str,
+    output: str,
+    local_builder: Callable[[str], Path],
+    isolated_build_wheel: Callable[[Path, str, int], tuple[str, str]],
+) -> None:
     tmp_path = local_builder(f"""
         import sys
 
@@ -54,27 +91,30 @@ def test_large_command_with_backend_output(stream: str, output: str, local_build
             assert config_settings == {{"payload": "y" * 1048576}}
             return "demo-1.0-py3-none-any.whl"
         """)
-    # Isolate the frontend so a pipe deadlock fails with a timeout instead of hanging pytest.
-    script = dedent("""
+    wheel, captured = isolated_build_wheel(tmp_path, output, 1048576)
+    assert (wheel, "x" * 1048576 in captured) == ("demo-1.0-py3-none-any.whl", True)
+
+
+@pytest.mark.parametrize(
+    ("stream", "output"), [pytest.param("stdout", "out", id="stdout"), pytest.param("stderr", "err", id="stderr")]
+)
+def test_backend_output_with_invalid_bytes(
+    stream: str,
+    output: str,
+    local_builder: Callable[[str], Path],
+    isolated_build_wheel: Callable[[Path, str, int], tuple[str, str]],
+) -> None:
+    tmp_path = local_builder(f"""
         import sys
-        from pathlib import Path
-        from pyproject_api import SubprocessFrontend
 
-        root = Path(sys.argv[1])
-        frontend = SubprocessFrontend(*SubprocessFrontend.create_args_from_folder(root)[:-1])
-        result = frontend.build_wheel(root / "dist", config_settings={"payload": "y" * 1048576})
-        print(result.wheel.name)
-        print("x" * 1048576 in getattr(result, sys.argv[2]))
+        sys.{stream}.buffer.write(b"before\\xffafter")
+        sys.{stream}.flush()
+
+        def build_wheel(wheel_directory, config_settings=None, metadata_directory=None):
+            return "demo-1.0-py3-none-any.whl"
         """)
-    result = subprocess.run(
-        [sys.executable, "-c", script, str(tmp_path), output],
-        capture_output=True,
-        text=True,
-        timeout=30,
-        check=True,
-    )
-    assert result.stdout.splitlines() == ["demo-1.0-py3-none-any.whl", "True"]
-    assert not result.stderr
+    wheel, captured = isolated_build_wheel(tmp_path, output, 0)
+    assert (wheel, "before\\xffafter" in captured) == ("demo-1.0-py3-none-any.whl", True)
 
 
 def test_large_command_with_failed_backend(local_builder: Callable[[str], Path]) -> None: