steam-backlog-enforcer/steam_backlog_enforcer/tests/_fake_proc.py
Krzysztof kuhy Rudnicki 47f77f5f1e
Some checks are pending
dependency freshness / dependencies on newest stable (push) Waiting to run
md-naming / markdown naming convention (push) Waiting to run
pre-commit / pre-commit (push) Waiting to run
Tests / test (push) Waiting to run
Make serve re-runnable, and cover it to 100%
The serve rework was sitting untracked: a /proc-based port guard, the
start-or-defer decision, and a stale-bundle rebuild. It had no tests, and it
changed cmd_serve(_config, state) to cmd_serve(argv) without updating the one
test that called it, so the suite was red and coverage was 251 lines short.

Tests, all against fakes:
- _fake_proc builds a synthetic /proc. _NET_TCP is derived from _PROC at
  import time, so patch_proc redirects both; patching _PROC alone would have
  had these tests parse the host's real process table.
- _serve_guard is split across two test files to stay under the 250-line cap.
  os is replaced wholesale in every termination test, so no signal can reach a
  real process even if the guard's self-protection were removed.
- _web_build mocks subprocess entirely and redirects every path constant into
  tmp_path. _no_subprocess now blocks _web_build.subprocess.run too: an
  unguarded test would fire a real `npm run build`, rewriting web/dist and
  perturbing frontend_is_stale() for every later test in the run.

Source changes, both required rather than incidental:
- _serve_guard's wildcard match was a `# noqa: S104` on a "0.0.0.0" literal,
  which the no-noqa hook rejects outright. It is now the all-zero hex column
  /proc actually prints, so the dotted-quad literal never appears.
- main/__init__.py's serve dispatch arm was itself uncovered; it now has a
  test asserting the raw argv passes through unparsed.

Also drops the document.hidden guard from useBudgetPoll: it skipped the very
first load, so the budget panel sat on "Reading budget state…" forever in a
background tab, which is every tab under browser automation.

Verified: 1712 python tests, 100% branch coverage; 128 vitest, 100%. The
guard's /proc parser was checked read-only against a real listener on :8000
and resolved the same pid as `ss`.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MKFzwzwRU1JD7SFLHtQBsz
2026-08-29 18:52:11 +02:00

109 lines
3.5 KiB
Python

"""Builders for a fake ``/proc`` tree used by the _serve_guard tests.
``_serve_guard`` was written to resolve port ownership through ``/proc`` rather
than ``ss`` precisely so it could be graded against a synthetic tree instead of
the host's real process table. This module is that tree.
Note that ``_NET_TCP`` is derived from ``_PROC`` at import time, so redirecting
the guard needs *both* patched — see :func:`patch_proc`.
"""
from __future__ import annotations
from contextlib import contextmanager
from typing import TYPE_CHECKING
from unittest.mock import patch
from steam_backlog_enforcer import _serve_guard
if TYPE_CHECKING:
from collections.abc import Iterator, Sequence
from pathlib import Path
# A real /proc/net/tcp row has 17 whitespace-separated columns. The guard reads
# only 1 (local address), 3 (state) and 9 (inode), but rejects short rows, so
# the surrounding columns have to be present and correctly positioned.
_REM_ADDRESS = "00000000:0000"
_QUEUES = "00000000:00000000 00:00000000 00000000"
_UID_TIMEOUT = "1000 0"
_ROW_TAIL = "1 0000000000000000 100 0 0 10 0"
def make_proc(root: Path) -> Path:
"""Create an empty fake ``/proc`` under *root* and return it."""
proc = root / "proc"
(proc / "net").mkdir(parents=True)
for name in ("tcp", "tcp6"):
write_net_tcp(proc, [], v6=name == "tcp6")
return proc
def tcp_row(local: str, inode: object = 4242, state: str = "0A") -> str:
"""Render one ``/proc/net/tcp`` row for a socket at *local*.
*inode* is deliberately untyped so a test can plant a non-numeric value in
the column the guard parses with ``int()``.
"""
return (
f" 0: {local} {_REM_ADDRESS} {state} {_QUEUES} "
f"{_UID_TIMEOUT} {inode} {_ROW_TAIL}"
)
def write_net_tcp(proc: Path, rows: Sequence[str], *, v6: bool = False) -> None:
"""Write *rows* into ``net/tcp`` (or ``net/tcp6``) with a header line."""
name = "tcp6" if v6 else "tcp"
header = " sl local_address rem_address st ...\n"
(proc / "net" / name).write_text(header + "\n".join(rows), encoding="utf-8")
def add_pid(
proc: Path,
pid: int,
*,
inodes: Sequence[int] = (),
cmdline: Sequence[str] = (),
ppid: int | None = None,
) -> Path:
"""Create ``/proc/<pid>`` with the requested sockets, argv and parent."""
pid_dir = proc / str(pid)
pid_dir.mkdir()
fd_dir = pid_dir / "fd"
fd_dir.mkdir()
for index, inode in enumerate(inodes):
(fd_dir / str(index)).symlink_to(f"socket:[{inode}]")
if cmdline:
(pid_dir / "cmdline").write_bytes("\0".join(cmdline).encode("utf-8") + b"\0")
if ppid is not None:
# comm may contain spaces and parentheses; the guard splits on ") ".
(pid_dir / "stat").write_text(
f"{pid} (py thon) S {ppid} 1 1 0", encoding="utf-8"
)
return pid_dir
@contextmanager
def patch_proc(proc: Path) -> Iterator[None]:
"""Point the guard at *proc*, including the derived _NET_TCP tuple."""
with (
patch.object(_serve_guard, "_PROC", proc),
patch.object(
_serve_guard,
"_NET_TCP",
(proc / "net" / "tcp", proc / "net" / "tcp6"),
),
):
yield
def add_opaque_pid(proc: Path, pid: int) -> Path:
"""Create a ``/proc/<pid>`` whose ``fd/`` cannot be opened.
This is what another user's process looks like from here: it holds the
port, but nothing about it can be attributed.
"""
pid_dir = proc / str(pid)
pid_dir.mkdir()
(pid_dir / "fd").write_text("", encoding="utf-8")
return pid_dir