claude-dev-env 4.3.0 → 7.1.0

package/hooks/blocking/test_session_edit_stage_gate_support.py

@@ -1,204 +0,0 @@
-"""Production-neutral fixtures for session-edit stage gate tests."""
-
-from __future__ import annotations
-
-import json
-import os
-import subprocess
-import sys
-from pathlib import Path
-
-_SUPPORT_DIRECTORY = Path(__file__).resolve().parent
-_HOOKS_TREE = _SUPPORT_DIRECTORY.parent
-if str(_HOOKS_TREE) not in sys.path:
-    sys.path.insert(0, str(_HOOKS_TREE))
-
-from hooks_constants.session_edit_stage_gate_constants import (  # noqa: E402
-    ALL_EDITED_FILE_PATHS_KEY,
-    SESSION_EDIT_FILE_PREFIX,
-    SESSION_EDIT_FILE_SUFFIX,
-)
-
-
-def clean_git_environment() -> dict[str, str]:
-    """Return the environment without inherited repository overrides."""
-    return {
-        each_key: each_value
-        for each_key, each_value in os.environ.items()
-        if not each_key.startswith("GIT_")
-    }
-
-
-def run_git(repository_root: Path, *git_arguments: str) -> None:
-    """Run one Git command in a test repository."""
-    subprocess.run(
-        ["git", "-C", str(repository_root), *git_arguments],
-        check=True,
-        capture_output=True,
-        env=clean_git_environment(),
-    )
-
-
-def initialize_repository(repository_root: Path) -> None:
-    """Create a Git repository on a non-protected test branch."""
-    run_git(repository_root, "init")
-    run_git(repository_root, "checkout", "-b", "test-session-stage")
-    run_git(repository_root, "config", "user.email", "tests@example.com")
-    run_git(repository_root, "config", "user.name", "Gate Tests")
-
-
-def prepare_repository_with_unstaged_edit(repository_root: Path) -> Path:
-    """Create one tracked file with an unstaged edit."""
-    initialize_repository(repository_root)
-    tracked_file = repository_root / "widget.py"
-    tracked_file.write_text("x = 1\n", encoding="utf-8")
-    run_git(repository_root, "add", "widget.py")
-    run_git(repository_root, "commit", "-m", "add widget")
-    tracked_file.write_text("x = 2\n", encoding="utf-8")
-    return tracked_file
-
-
-def prepare_repository_with_two_unstaged_edits(repository_root: Path) -> tuple[Path, Path]:
-    """Create two tracked files with separate unstaged edits."""
-    initialize_repository(repository_root)
-    tracked_file = repository_root / "widget.py"
-    second_file = repository_root / "second_widget.py"
-    tracked_file.write_text("x = 1\n", encoding="utf-8")
-    second_file.write_text("y = 1\n", encoding="utf-8")
-    run_git(repository_root, "add", "widget.py", "second_widget.py")
-    run_git(repository_root, "commit", "-m", "add widgets")
-    tracked_file.write_text("x = 2\n", encoding="utf-8")
-    second_file.write_text("y = 2\n", encoding="utf-8")
-    return tracked_file, second_file
-
-
-def make_test_directories(tmp_path: Path) -> tuple[Path, Path, Path]:
-    """Create repository, tracker, and home directories for one test."""
-    repository_root = tmp_path / "repo"
-    repository_root.mkdir()
-    temp_directory = tmp_path / "tmp"
-    temp_directory.mkdir()
-    home_directory = tmp_path / "home"
-    home_directory.mkdir()
-    return repository_root, temp_directory, home_directory
-
-
-def make_process_repository(tmp_path: Path) -> Path:
-    """Create a separate Git repository for the dispatcher process directory."""
-    process_repository = tmp_path / "process-repository"
-    process_repository.mkdir()
-    initialize_repository(process_repository)
-    return process_repository
-
-
-def prepare_process_child_repository(process_repository: Path) -> None:
-    """Create the relative command directory inside process repository B."""
-    process_child_repository = process_repository / "child"
-    process_child_repository.mkdir()
-    initialize_repository(process_child_repository)
-
-
-def build_hook_environment(
-    temp_directory: Path,
-    home_directory: Path,
-) -> dict[str, str]:
-    """Build isolated environment variables for production hooks."""
-    environment = clean_git_environment()
-    environment["TMP"] = str(temp_directory)
-    environment["TEMP"] = str(temp_directory)
-    environment["TMPDIR"] = str(temp_directory)
-    environment["USERPROFILE"] = str(home_directory)
-    environment["HOME"] = str(home_directory)
-    return environment
-
-
-def run_production_hook(
-    hook_path: Path,
-    payload: str,
-    repository_root: Path,
-    temp_directory: Path,
-    home_directory: Path,
-    process_directory: Path | None = None,
-) -> subprocess.CompletedProcess[str]:
-    """Run one hook script as a production subprocess."""
-    return subprocess.run(
-        [sys.executable, str(hook_path)],
-        check=False,
-        input=payload,
-        capture_output=True,
-        text=True,
-        cwd=str(process_directory or repository_root),
-        env=build_hook_environment(temp_directory, home_directory),
-        timeout=60,
-    )
-
-
-def run_tracker_event(
-    tracked_file: Path,
-    session_id: str,
-    repository_root: Path,
-    temp_directory: Path,
-    home_directory: Path,
-) -> subprocess.CompletedProcess[str]:
-    """Run the production PostToolUse tracker for one edited file."""
-    payload = json.dumps(
-        {
-            "tool_name": "Edit",
-            "session_id": session_id,
-            "tool_input": {"file_path": str(tracked_file)},
-        }
-    )
-    return run_production_hook(
-        _HOOKS_TREE / "observability" / "session_file_edit_tracker.py",
-        payload,
-        repository_root,
-        temp_directory,
-        home_directory,
-    )
-
-
-def run_bash_dispatcher(
-    bash_command: str,
-    session_id: str,
-    repository_root: Path,
-    temp_directory: Path,
-    home_directory: Path,
-    process_directory: Path | None = None,
-) -> subprocess.CompletedProcess[str]:
-    """Run the production Bash PreToolUse dispatcher for one command."""
-    payload = json.dumps(
-        {
-            "tool_name": "Bash",
-            "cwd": str(repository_root),
-            "session_id": session_id,
-            "tool_input": {"command": bash_command},
-        }
-    )
-    return run_production_hook(
-        _SUPPORT_DIRECTORY / "bash_pre_tool_use_dispatcher.py",
-        payload,
-        repository_root,
-        temp_directory,
-        home_directory,
-        process_directory,
-    )
-
-
-def read_tracker_paths(temp_directory: Path, session_id: str) -> list[str]:
-    """Read the recorded paths from one production tracker file."""
-    tracker_file = (
-        temp_directory / f"{SESSION_EDIT_FILE_PREFIX}{session_id}{SESSION_EDIT_FILE_SUFFIX}"
-    )
-    tracker_payload = json.loads(tracker_file.read_text(encoding="utf-8"))
-    return tracker_payload[ALL_EDITED_FILE_PATHS_KEY]
-
-
-def parse_dispatcher_decision(stdout_text: str) -> tuple[str, str]:
-    """Return the dispatcher decision and reason from its stdout payload."""
-    if not stdout_text.strip():
-        return "", ""
-    hook_specific_output = json.loads(stdout_text)["hookSpecificOutput"]
-    return (
-        hook_specific_output.get("permissionDecision", ""),
-        hook_specific_output.get("permissionDecisionReason", ""),
-    )

package/hooks/blocking/test_session_edit_stage_gate_tracker.py

@@ -1,116 +0,0 @@
-"""Production-path tracker event-order tests for the session-edit stage gate."""
-
-from __future__ import annotations
-
-import sys
-from pathlib import Path
-
-import pytest
-
-_TEST_DIRECTORY = Path(__file__).resolve().parent
-if str(_TEST_DIRECTORY) not in sys.path:
-    sys.path.insert(0, str(_TEST_DIRECTORY))
-
-from test_session_edit_stage_gate_support import (  # noqa: E402
-    make_test_directories,
-    make_process_repository,
-    parse_dispatcher_decision,
-    prepare_process_child_repository,
-    prepare_repository_with_unstaged_edit,
-    prepare_repository_with_two_unstaged_edits,
-    read_tracker_paths,
-    run_bash_dispatcher,
-    run_tracker_event,
-)
-
-SESSION_ID = "tracker-production-session"
-
-
-def test_tracker_events_precede_dispatcher_stage_decision(tmp_path: Path) -> None:
-    """The dispatcher sees every edit recorded before the commit event."""
-    repository_root, temp_directory, home_directory = make_test_directories(tmp_path)
-    tracked_file, second_file = prepare_repository_with_two_unstaged_edits(repository_root)
-    for each_file in (tracked_file, second_file):
-        tracker_run = run_tracker_event(
-            each_file, SESSION_ID, repository_root, temp_directory, home_directory
-        )
-        assert tracker_run.returncode == 0
-
-    assert read_tracker_paths(temp_directory, SESSION_ID) == [
-        str(tracked_file.resolve()),
-        str(second_file.resolve()),
-    ]
-
-    dispatcher_run = run_bash_dispatcher(
-        "git commit -m update",
-        SESSION_ID,
-        repository_root,
-        temp_directory,
-        home_directory,
-    )
-    assert dispatcher_run.returncode == 0
-    decision, reason = parse_dispatcher_decision(dispatcher_run.stdout)
-    assert decision == "deny"
-    assert "widget.py" in reason
-    assert "second_widget.py" in reason
-
-
-def test_dispatcher_uses_event_cwd_when_process_runs_elsewhere(tmp_path: Path) -> None:
-    """The dispatcher passes event cwd so the gate inspects the event repository."""
-    repository_root, temp_directory, home_directory = make_test_directories(tmp_path)
-    process_repository = make_process_repository(tmp_path)
-    tracked_file = prepare_repository_with_unstaged_edit(repository_root)
-    tracker_run = run_tracker_event(
-        tracked_file, SESSION_ID, repository_root, temp_directory, home_directory
-    )
-    assert tracker_run.returncode == 0
-
-    dispatcher_run = run_bash_dispatcher(
-        "git commit -m update",
-        SESSION_ID,
-        repository_root,
-        temp_directory,
-        home_directory,
-        process_directory=process_repository,
-    )
-    assert dispatcher_run.returncode == 0
-    decision, reason = parse_dispatcher_decision(dispatcher_run.stdout)
-    assert decision == "deny"
-    assert "widget.py" in reason
-
-
-@pytest.mark.parametrize(
-    "bash_command",
-    [
-        "cd child && git commit -m update",
-        "pushd child && git commit -m update",
-        "git -C child commit -m update",
-    ],
-)
-def test_dispatcher_anchors_relative_command_directory_to_event_cwd(
-    tmp_path: Path,
-    bash_command: str,
-) -> None:
-    """Relative commit directories resolve from event cwd across command forms."""
-    repository_root, temp_directory, home_directory = make_test_directories(tmp_path)
-    process_repository = make_process_repository(tmp_path)
-    prepare_process_child_repository(process_repository)
-    (repository_root / "child").mkdir()
-    tracked_file = prepare_repository_with_unstaged_edit(repository_root)
-    tracker_run = run_tracker_event(
-        tracked_file, SESSION_ID, repository_root, temp_directory, home_directory
-    )
-    assert tracker_run.returncode == 0
-
-    dispatcher_run = run_bash_dispatcher(
-        bash_command,
-        SESSION_ID,
-        repository_root,
-        temp_directory,
-        home_directory,
-        process_directory=process_repository,
-    )
-    assert dispatcher_run.returncode == 0
-    decision, reason = parse_dispatcher_decision(dispatcher_run.stdout)
-    assert decision == "deny"
-    assert "widget.py" in reason

package/hooks/blocking/test_session_handoff_blocker.py

@@ -1,318 +0,0 @@
-"""Tests for session_handoff_blocker hook response shape."""
-
-import importlib.util
-import json
-import os
-import subprocess
-import sys
-
-HOOK_SCRIPT_PATH = os.path.join(os.path.dirname(__file__), "session_handoff_blocker.py")
-_HOOKS_DIR = os.path.dirname(HOOK_SCRIPT_PATH)
-_HOOKS_ROOT = os.path.join(_HOOKS_DIR, "..")
-_HOOK_CONFIG_DIR = os.path.join(_HOOKS_ROOT, "hooks_constants")
-if _HOOKS_DIR not in sys.path:
-    sys.path.insert(0, _HOOKS_DIR)
-if _HOOKS_ROOT not in sys.path:
-    sys.path.insert(0, _HOOKS_ROOT)
-import session_handoff_blocker
-from hooks_constants.messages import USER_FACING_CONTEXT_REASSURANCE_NOTICE
-from hooks_constants.text_stripping import strip_code_and_quotes
-
-
-def test_blocker_uses_shared_strip_code_and_quotes() -> None:
-    assert session_handoff_blocker.strip_code_and_quotes is strip_code_and_quotes
-
-
-NEW_SESSION_PROPOSAL_MESSAGE = (
-    "I recommend we continue this in a fresh session to keep things manageable."
-)
-RUNNING_LOW_ON_CONTEXT_MESSAGE = (
-    "We are running low on context, so let me summarize where things stand."
-)
-SHORT_ON_TOKENS_MESSAGE = "I'm getting short on tokens, so I'll wrap up here."
-CONSERVE_CONTEXT_MESSAGE = "To conserve context, let me stop and hand off the remaining work."
-CONTEXT_WINDOW_HANDOFF_MESSAGE = (
-    "The context window is filling up, so I'll wrap up and we can continue later."
-)
-BENIGN_TOPICAL_MESSAGE = "The function accepts a context manager and a token string."
-CLEAN_MESSAGE = "The parser handles every fixture and returns a deduplicated list."
-TECHNICAL_TERMINAL_SESSION_MESSAGE = (
-    "Consider starting a new session in your terminal to pick up the env vars."
-)
-LOAD_TEST_SESSION_MESSAGE = "We can spin up a fresh session for the load test."
-DATABASE_SESSION_MESSAGE = "Open a new database session before running the query."
-HANDOFF_NEW_SESSION_MESSAGE = (
-    "Let's wrap up and continue this in a fresh session to pick this up later."
-)
-LOW_ON_CONTEXT_WITHOUT_CUE_MESSAGE = (
-    "I am low on context for this edge case in the parser."
-)
-SAVE_TOKENS_REPORT_MESSAGE = "To save tokens, I inlined the constant."
-LOW_ON_CONTEXT_WITH_HANDOFF_CUE_MESSAGE = (
-    "I'm low on context, so let me wrap up and hand off."
-)
-NEW_SESSION_TOKEN_MESSAGE = (
-    "To continue this task I need the new session token from the API."
-)
-RESUME_TRAFFIC_FRESH_SESSION_POOL_MESSAGE = (
-    "After deploy, resume traffic in a fresh session pool."
-)
-THIRD_PERSON_RUNS_LOW_DESIGN_MESSAGE = (
-    "The agent should stop when it runs low on context, per the old design."
-)
-DOCUMENT_RUNNING_LOW_MESSAGE = (
-    "Document that agents running low on tokens should not stop."
-)
-USER_ADVICE_NEW_SESSION_MESSAGE = (
-    "To save tokens, you could start a new session for the unrelated task."
-)
-CONTEXT_WINDOW_SUMMARIZE_MESSAGE = (
-    "The context window is getting full, so let me summarize where we are."
-)
-BENIGN_SUMMARIZE_REPORT_MESSAGE = "I should summarize the findings for the report."
-BENIGN_STOP_PLUS_UNRELATED_RUNNING_LOW_MESSAGE = (
-    "Let me pause the animation timer. The job fails when it is running low on tokens."
-)
-BENIGN_STOP_PLUS_USER_DIRECTED_NEW_SESSION_MESSAGE = (
-    "Let me stop the timer. The user can continue this in a fresh session."
-)
-EMPTY_MESSAGE = ""
-
-
-def run_hook_with_message(assistant_message: str) -> subprocess.CompletedProcess:
-    hook_input_payload = json.dumps({"last_assistant_message": assistant_message})
-    return subprocess.run(
-        [sys.executable, HOOK_SCRIPT_PATH],
-        input=hook_input_payload,
-        capture_output=True,
-        text=True,
-        check=False,
-    )
-
-
-def run_hook_with_payload(hook_input_payload: dict) -> subprocess.CompletedProcess:
-    return subprocess.run(
-        [sys.executable, HOOK_SCRIPT_PATH],
-        input=json.dumps(hook_input_payload),
-        capture_output=True,
-        text=True,
-        check=False,
-    )
-
-
-def test_user_facing_notice_matches_config_messages_module():
-    config_messages_path = os.path.join(_HOOK_CONFIG_DIR, "messages.py")
-    specification = importlib.util.spec_from_file_location("messages", config_messages_path)
-    module = importlib.util.module_from_spec(specification)
-    specification.loader.exec_module(module)
-
-    assert module.USER_FACING_CONTEXT_REASSURANCE_NOTICE == USER_FACING_CONTEXT_REASSURANCE_NOTICE
-    assert (
-        session_handoff_blocker.USER_FACING_CONTEXT_REASSURANCE_NOTICE
-        == module.USER_FACING_CONTEXT_REASSURANCE_NOTICE
-    )
-
-
-def test_new_session_proposal_emits_block_with_short_user_notice():
-    completed_process = run_hook_with_message(NEW_SESSION_PROPOSAL_MESSAGE)
-
-    assert completed_process.returncode == 0
-    parsed_response = json.loads(completed_process.stdout)
-
-    assert parsed_response["decision"] == "block"
-    assert parsed_response["systemMessage"] == USER_FACING_CONTEXT_REASSURANCE_NOTICE
-    assert parsed_response["suppressOutput"] is True
-    assert "ample context remaining" in parsed_response["reason"]
-    assert "long-horizon-autonomy" in parsed_response["reason"]
-
-
-def test_running_low_on_context_emits_block():
-    completed_process = run_hook_with_message(RUNNING_LOW_ON_CONTEXT_MESSAGE)
-
-    assert completed_process.returncode == 0
-    parsed_response = json.loads(completed_process.stdout)
-
-    assert parsed_response["decision"] == "block"
-    assert parsed_response["systemMessage"] == USER_FACING_CONTEXT_REASSURANCE_NOTICE
-
-
-def test_short_on_tokens_emits_block():
-    completed_process = run_hook_with_message(SHORT_ON_TOKENS_MESSAGE)
-
-    assert completed_process.returncode == 0
-    parsed_response = json.loads(completed_process.stdout)
-
-    assert parsed_response["decision"] == "block"
-
-
-def test_conserve_context_emits_block():
-    completed_process = run_hook_with_message(CONSERVE_CONTEXT_MESSAGE)
-
-    assert completed_process.returncode == 0
-    parsed_response = json.loads(completed_process.stdout)
-
-    assert parsed_response["decision"] == "block"
-
-
-def test_context_window_co_occurring_handoff_cue_emits_block():
-    completed_process = run_hook_with_message(CONTEXT_WINDOW_HANDOFF_MESSAGE)
-
-    assert completed_process.returncode == 0
-    parsed_response = json.loads(completed_process.stdout)
-
-    assert parsed_response["decision"] == "block"
-
-
-def test_benign_topical_mention_passes_through_with_no_output():
-    completed_process = run_hook_with_message(BENIGN_TOPICAL_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_technical_terminal_session_passes_through_with_no_output():
-    completed_process = run_hook_with_message(TECHNICAL_TERMINAL_SESSION_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_load_test_session_passes_through_with_no_output():
-    completed_process = run_hook_with_message(LOAD_TEST_SESSION_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_database_session_passes_through_with_no_output():
-    completed_process = run_hook_with_message(DATABASE_SESSION_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_new_session_with_handoff_framing_emits_block():
-    completed_process = run_hook_with_message(HANDOFF_NEW_SESSION_MESSAGE)
-
-    assert completed_process.returncode == 0
-    parsed_response = json.loads(completed_process.stdout)
-
-    assert parsed_response["decision"] == "block"
-    assert parsed_response["systemMessage"] == USER_FACING_CONTEXT_REASSURANCE_NOTICE
-
-
-def test_low_on_context_without_handoff_cue_passes_through_with_no_output():
-    completed_process = run_hook_with_message(LOW_ON_CONTEXT_WITHOUT_CUE_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_save_tokens_work_report_passes_through_with_no_output():
-    completed_process = run_hook_with_message(SAVE_TOKENS_REPORT_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_low_on_context_with_handoff_cue_emits_block():
-    completed_process = run_hook_with_message(LOW_ON_CONTEXT_WITH_HANDOFF_CUE_MESSAGE)
-
-    assert completed_process.returncode == 0
-    parsed_response = json.loads(completed_process.stdout)
-
-    assert parsed_response["decision"] == "block"
-    assert parsed_response["systemMessage"] == USER_FACING_CONTEXT_REASSURANCE_NOTICE
-
-
-def test_new_session_token_passes_through_with_no_output():
-    completed_process = run_hook_with_message(NEW_SESSION_TOKEN_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_resume_traffic_fresh_session_pool_passes_through_with_no_output():
-    completed_process = run_hook_with_message(RESUME_TRAFFIC_FRESH_SESSION_POOL_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_third_person_runs_low_description_passes_through_with_no_output():
-    completed_process = run_hook_with_message(THIRD_PERSON_RUNS_LOW_DESIGN_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_document_running_low_instruction_passes_through_with_no_output():
-    completed_process = run_hook_with_message(DOCUMENT_RUNNING_LOW_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_user_directed_new_session_advice_passes_through_with_no_output():
-    completed_process = run_hook_with_message(USER_ADVICE_NEW_SESSION_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_context_window_summarize_handoff_emits_block():
-    completed_process = run_hook_with_message(CONTEXT_WINDOW_SUMMARIZE_MESSAGE)
-
-    assert completed_process.returncode == 0
-    parsed_response = json.loads(completed_process.stdout)
-
-    assert parsed_response["decision"] == "block"
-    assert parsed_response["systemMessage"] == USER_FACING_CONTEXT_REASSURANCE_NOTICE
-
-
-def test_benign_summarize_report_passes_through_with_no_output():
-    completed_process = run_hook_with_message(BENIGN_SUMMARIZE_REPORT_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_benign_stop_with_unrelated_running_low_sentence_passes_through_with_no_output():
-    completed_process = run_hook_with_message(
-        BENIGN_STOP_PLUS_UNRELATED_RUNNING_LOW_MESSAGE
-    )
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_benign_stop_with_user_directed_new_session_sentence_passes_through_with_no_output():
-    completed_process = run_hook_with_message(
-        BENIGN_STOP_PLUS_USER_DIRECTED_NEW_SESSION_MESSAGE
-    )
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_clean_message_passes_through_with_no_output():
-    completed_process = run_hook_with_message(CLEAN_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_empty_message_passes_through_with_no_output():
-    completed_process = run_hook_with_message(EMPTY_MESSAGE)
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""
-
-
-def test_stop_hook_active_short_circuits_with_no_output():
-    completed_process = run_hook_with_payload(
-        {"last_assistant_message": NEW_SESSION_PROPOSAL_MESSAGE, "stop_hook_active": True}
-    )
-
-    assert completed_process.returncode == 0
-    assert completed_process.stdout == ""