Refactor shell script tests based on code review

- Add shared fixtures to conftest.py (git_repo, git_repo_with_policy, etc.)
- Add shared run_shell_script helper function
- Remove duplicate fixture definitions from test files
- Add assertion to test_rejects_name_with_spaces
- Remove redundant TestCapturePromptWorkTreeJsonFormat class
- Apply ruff formatting to all test files

Author: claude

tests/shell_script_tests/conftest.py

@@ -0,0 +1,115 @@
+"""Shared fixtures for shell script tests."""
+
+import json
+import os
+import subprocess
+from pathlib import Path
+
+import pytest
+from git import Repo
+
+
+@pytest.fixture
+def git_repo(tmp_path: Path) -> Path:
+    """Create a basic git repo for testing."""
+    repo = Repo.init(tmp_path)
+
+    readme = tmp_path / "README.md"
+    readme.write_text("# Test Project\n")
+    repo.index.add(["README.md"])
+    repo.index.commit("Initial commit")
+
+    return tmp_path
+
+
+@pytest.fixture
+def git_repo_with_policy(tmp_path: Path) -> Path:
+    """Create a git repo with policy that will fire."""
+    repo = Repo.init(tmp_path)
+
+    readme = tmp_path / "README.md"
+    readme.write_text("# Test Project\n")
+    repo.index.add(["README.md"])
+    repo.index.commit("Initial commit")
+
+    # Policy that triggers on any Python file
+    policy_file = tmp_path / ".deepwork.policy.yml"
+    policy_file.write_text(
+        """- name: "Python File Policy"
+  trigger: "**/*.py"
+  compare_to: prompt
+  instructions: |
+    Review Python files for quality.
+"""
+    )
+
+    # Empty baseline so new files trigger
+    deepwork_dir = tmp_path / ".deepwork"
+    deepwork_dir.mkdir(exist_ok=True)
+    (deepwork_dir / ".last_work_tree").write_text("")
+
+    return tmp_path
+
+
+@pytest.fixture
+def policy_hooks_dir() -> Path:
+    """Return the path to the policy hooks scripts directory."""
+    return (
+        Path(__file__).parent.parent.parent
+        / "src"
+        / "deepwork"
+        / "standard_jobs"
+        / "deepwork_policy"
+        / "hooks"
+    )
+
+
+@pytest.fixture
+def jobs_scripts_dir() -> Path:
+    """Return the path to the jobs scripts directory."""
+    return (
+        Path(__file__).parent.parent.parent / "src" / "deepwork" / "standard_jobs" / "deepwork_jobs"
+    )
+
+
+def run_shell_script(
+    script_path: Path,
+    cwd: Path,
+    args: list[str] | None = None,
+    hook_input: dict | None = None,
+    env_extra: dict[str, str] | None = None,
+) -> tuple[str, str, int]:
+    """
+    Run a shell script and return its output.
+
+    Args:
+        script_path: Path to the shell script
+        cwd: Working directory to run the script in
+        args: Optional list of arguments to pass to the script
+        hook_input: Optional JSON input to pass via stdin
+        env_extra: Optional extra environment variables
+
+    Returns:
+        Tuple of (stdout, stderr, return_code)
+    """
+    env = os.environ.copy()
+    env["PYTHONPATH"] = str(Path(__file__).parent.parent.parent / "src")
+    if env_extra:
+        env.update(env_extra)
+
+    cmd = ["bash", str(script_path)]
+    if args:
+        cmd.extend(args)
+
+    stdin_data = json.dumps(hook_input) if hook_input else ""
+
+    result = subprocess.run(
+        cmd,
+        cwd=cwd,
+        capture_output=True,
+        text=True,
+        input=stdin_data,
+        env=env,
+    )
+
+    return result.stdout, result.stderr, result.returncode