verifaied 0.34.0.dev84__tar.gz → 0.34.1.dev85__tar.gz

{verifaied-0.34.0.dev84 → verifaied-0.34.1.dev85}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: verifaied
-Version: 0.34.0.dev84
+Version: 0.34.1.dev85
 Summary: Find what's untested in your code — locally, no account required
 Project-URL: Homepage, https://pypi.org/project/verifaied/
 Author: Kyle Richards

{verifaied-0.34.0.dev84 → verifaied-0.34.1.dev85}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "verifaied"
-version = "0.34.0.dev84"
+version = "0.34.1.dev85"
 description = "Find what's untested in your code — locally, no account required"
 readme = "README.md"
 requires-python = ">=3.10"

{verifaied-0.34.0.dev84 → verifaied-0.34.1.dev85}/src/verifaied/record.py

@@ -774,6 +774,17 @@ def _final_flush(
     return True
 
 
+def _sleep(seconds: float) -> None:
+    """The session loop's own clock.
+
+    Its own name rather than ``time.sleep`` so a test can script it without
+    scripting every other sleep in the process — ``subprocess`` polls a
+    child it waits on with ``time.sleep``, and so does anything else
+    running beside the loop.
+    """
+    time.sleep(seconds)
+
+
 def _interactive_session(
     collector: CoverageCollector,
     *,
@@ -794,7 +805,7 @@ def _interactive_session(
     try:
         while True:
             if stop is None:
-                time.sleep(interval)
+                _sleep(interval)
             elif stop.wait(interval):
                 console.print("[dim]window closed — finishing.[/dim]")
                 break
@@ -925,7 +936,7 @@ def _run_wrapped_child(
     interrupted = False
     try:
         while child.poll() is None:
-            time.sleep(interval)
+            _sleep(interval)
             _interval_flush(
                 collector,
                 console=console,

{verifaied-0.34.0.dev84 → verifaied-0.34.1.dev85}/tests/test_record.py

@@ -8,13 +8,14 @@ loop with a mocked client.
 
 Every bare session here runs against ``FakeWindow`` rather than the real
 sidecar — the window is Node and Chromium, and neither belongs in a unit
-test. The fake keeps ``time.sleep`` as the loop's clock, which is how the
+test. The fake keeps the loop's own ``_sleep`` as its clock, which is how the
 tests below script a session's end with a ``KeyboardInterrupt``.
 """
 
 from __future__ import annotations
 
 import json
+import subprocess
 import sys
 import threading
 import time
@@ -26,6 +27,7 @@ import httpx
 import pytest
 from typer.testing import CliRunner
 
+from verifaied import record as record_module
 from verifaied.cli import app
 from verifaied.client import ApiError, UploadResult
 from verifaied.collect_browser import CollectBrowserError
@@ -61,13 +63,14 @@ runner = CliRunner()
 
 
 class _SleepingEvent(threading.Event):
-    """An Event whose wait is ``time.sleep``, so the loop tests' scripted
-    ``time.sleep`` side effects still end a session that has a window."""
+    """An Event whose wait is the session loop's own sleep, so the loop
+    tests' scripted ``_sleep`` side effects still end a session that has a
+    window."""
 
     def wait(self, timeout: float | None = None) -> bool:
         if self.is_set():
             return True
-        time.sleep(timeout or 0)
+        record_module._sleep(timeout or 0)
         return self.is_set()
 
 
@@ -492,7 +495,7 @@ def test_flush_upload_defaults_to_the_manual_dimension(project: Path):
 
 
 def _run_loop(sleep_side_effect, flush_side_effect):
-    """Drive run_record with time.sleep scripted to end the session.
+    """Drive run_record with the loop's ``_sleep`` scripted to end the session.
 
     The ledger calls are stubbed out here — they're exercised on their own
     below, and leaving them live would make every loop test depend on how
@@ -502,7 +505,7 @@ def _run_loop(sleep_side_effect, flush_side_effect):
     with (
         patch("verifaied.record.create_session", return_value=(uuid4(), None)),
         patch("verifaied.record.finalize_session"),
-        patch("verifaied.record.time.sleep", side_effect=sleep_side_effect),
+        patch("verifaied.record._sleep", side_effect=sleep_side_effect),
         patch("verifaied.record.flush_upload", side_effect=flush_side_effect) as flush,
     ):
         code = run_record(
@@ -520,6 +523,27 @@ def _run_loop(sleep_side_effect, flush_side_effect):
     return code, flush, console, err_console
 
 
+def test_a_sleep_elsewhere_never_spends_the_loops_script():
+    """``subprocess`` polls a child it waits on with ``time.sleep`` — the git
+    lookup before the loop does exactly that on a busy machine — so the
+    script ending a session is the loop's own ``_sleep`` and nothing else's.
+    Here every subprocess the session starts sleeps first, as a slow one
+    would."""
+    real_run = subprocess.run
+
+    def slow_run(*args, **kwargs):
+        time.sleep(0.01)
+        return real_run(*args, **kwargs)
+
+    with patch("verifaied.suite_report.subprocess.run", side_effect=slow_run):
+        code, flush, _, _ = _run_loop(
+            sleep_side_effect=[None, KeyboardInterrupt()],
+            flush_side_effect=None,
+        )
+    assert code == 0
+    assert flush.call_count == 2
+
+
 def test_loop_uploads_on_change_and_flushes_on_ctrl_c():
     # Two ticks then Ctrl-C; the interrupt triggers one final flush.
     code, flush, console, _ = _run_loop(
@@ -607,7 +631,7 @@ def test_ctrl_c_still_uploads_after_a_failed_interval_upload(project: Path):
         patch("verifaied.record.create_session", return_value=(uuid4(), None)),
         patch("verifaied.record.finalize_session"),
         patch("verifaied.record.make_server", side_effect=capture_server),
-        patch("verifaied.record.time.sleep", side_effect=sleep_script),
+        patch("verifaied.record._sleep", side_effect=sleep_script),
         patch("verifaied.record.upload", side_effect=flaky),
     ):
         code = run_record(
@@ -774,7 +798,7 @@ def test_wrapper_interrupt_stops_the_child_and_still_flushes():
     child = [sys.executable, "-c", "import time; time.sleep(30)"]
     with (
         patch("verifaied.record.flush_upload", side_effect=flush),
-        patch("verifaied.record.time.sleep", side_effect=KeyboardInterrupt()),
+        patch("verifaied.record._sleep", side_effect=KeyboardInterrupt()),
     ):
         code = run_record(
             api_url="http://api.test",
@@ -1001,7 +1025,7 @@ def test_record_serves_a_real_round_trip_through_run_record():
 
     with (
         patch("verifaied.record.make_server", side_effect=capture_server),
-        patch("verifaied.record.time.sleep", side_effect=sleep_then_post),
+        patch("verifaied.record._sleep", side_effect=sleep_then_post),
         patch("verifaied.record.flush_upload", side_effect=fake_flush),
     ):
         code = run_record(
@@ -1267,7 +1291,7 @@ def _requested_port(*, command: list[str] | None) -> tuple[int, int]:
     with (
         patch("verifaied.record.make_server", side_effect=capture_server),
         patch(
-            "verifaied.record.time.sleep",
+            "verifaied.record._sleep",
             side_effect=None if command else [KeyboardInterrupt()],
         ),
         patch("verifaied.record.flush_upload", side_effect=lambda *a, **k: None),
@@ -1339,7 +1363,7 @@ def _record_with_ledger(
         ),
         patch("verifaied.record.flush_upload", flush),
         patch(
-            "verifaied.record.time.sleep",
+            "verifaied.record._sleep",
             side_effect=None if command else KeyboardInterrupt(),
         ),
     ):
@@ -2220,7 +2244,7 @@ def test_a_bare_session_never_tracks_a_suite():
         patch("verifaied.record.create_session", MagicMock(return_value=(None, None))),
         patch("verifaied.record.finalize_session", MagicMock()),
         patch("verifaied.record.flush_upload", MagicMock(return_value=None)),
-        patch("verifaied.record.time.sleep", side_effect=KeyboardInterrupt()),
+        patch("verifaied.record._sleep", side_effect=KeyboardInterrupt()),
     ):
         run_record(
             api_url="http://api.test",
@@ -2267,7 +2291,7 @@ def test_run_record_starts_the_proxy_and_closes_it(tmp_path: Path):
     with (
         patch("verifaied.record.create_session", return_value=(uuid4(), None)),
         patch("verifaied.record.finalize_session"),
-        patch("verifaied.record.time.sleep", side_effect=KeyboardInterrupt()),
+        patch("verifaied.record._sleep", side_effect=KeyboardInterrupt()),
         patch("verifaied.record.flush_upload", return_value=None),
         patch("verifaied.record.start_proxy", return_value=handle) as start,
     ):
@@ -2332,7 +2356,7 @@ def test_no_proxy_means_start_proxy_is_never_called(tmp_path: Path):
     with (
         patch("verifaied.record.create_session", return_value=(uuid4(), None)),
         patch("verifaied.record.finalize_session"),
-        patch("verifaied.record.time.sleep", side_effect=KeyboardInterrupt()),
+        patch("verifaied.record._sleep", side_effect=KeyboardInterrupt()),
         patch("verifaied.record.flush_upload", return_value=None),
         patch("verifaied.record.start_proxy") as start,
     ):
@@ -2490,7 +2514,7 @@ def test_run_record_stops_on_a_plan_limit_without_recording(tmp_path: Path):
             side_effect=ApiError(403, "Your plan covers 3 branches per repository."),
         ),
         patch("verifaied.record.flush_upload") as flush,
-        patch("verifaied.record.time.sleep") as sleep,
+        patch("verifaied.record._sleep") as sleep,
     ):
         code = run_record(
             api_url="http://api.test",
@@ -2579,7 +2603,7 @@ def _bare_session(fake_window, *, browser: bool = True, url=None, proxy=False, *
         patch("verifaied.record.create_session", opened),
         patch("verifaied.record.finalize_session"),
         patch(
-            "verifaied.record.time.sleep",
+            "verifaied.record._sleep",
             side_effect=extra.pop("sleep_side_effect", [KeyboardInterrupt()]),
         ),
         patch("verifaied.record.flush_upload", side_effect=lambda *a, **k: None),

{verifaied-0.34.0.dev84 → verifaied-0.34.1.dev85}/uv.lock

@@ -1242,7 +1242,7 @@ wheels = [
 
 [[package]]
 name = "verifaied"
-version = "0.34.0"
+version = "0.34.1"
 source = { editable = "." }
 dependencies = [
     { name = "httpx" },