Skip to content

Comments

Add automated tests for all scripts#40

Merged
nhorton merged 3 commits intomainfrom
claude/add-script-tests-ej4zp
Jan 15, 2026
Merged

Add automated tests for all scripts#40
nhorton merged 3 commits intomainfrom
claude/add-script-tests-ej4zp

Conversation

@nhorton
Copy link
Contributor

@nhorton nhorton commented Jan 15, 2026

  • Add tests for user_prompt_submit.sh hook
  • Add tests for capture_prompt_work_tree.sh helper
  • Add tests for make_new_job.sh utility
  • Add JSON format validation tests enforcing Claude Code hooks response format
  • Tests verify that hook scripts return valid JSON with correct structure

- Add tests for user_prompt_submit.sh hook
- Add tests for capture_prompt_work_tree.sh helper
- Add tests for make_new_job.sh utility
- Add JSON format validation tests enforcing Claude Code hooks response format
- Tests verify that hook scripts return valid JSON with correct structure
@nhorton nhorton force-pushed the claude/add-script-tests-ej4zp branch from 779b855 to ac9156e Compare January 15, 2026 20:23
- 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
@nhorton nhorton merged commit c95f3e5 into main Jan 15, 2026
2 checks passed
@nhorton nhorton deleted the claude/add-script-tests-ej4zp branch January 15, 2026 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants