Legacy lost integration test work... #81
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add Docker-based integration test environment with mocked external services - Create mock scripts for claude, gh, and git commands to isolate tests - Add test coverage for worktree operations, Claude sessions, PR workflows - Refactor handlers to use RegisterRoutes pattern for cleaner organization - Include test data management and debugging tools for development 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Major improvements to the integration testing architecture: **New Architecture:** - Test container runs Catnip server on port 8181 with hot reloading via Air - Integration tests run from host machine making real HTTP calls - Mock scripts and test data baked into Docker image with proper PATH - Container persists between test runs for faster execution **Key Features:** - Self-contained: Mocks/data included in image, works without volume mounts - Development-friendly: Optional volume mounts for iterating on mocks/data - Fast builds: BuildKit cache mounts for Go dependencies - Hot reloading: Server code changes reflected immediately via Air - Port isolation: Test server on 8181 avoids conflicts with dev server on 8080 **Performance Improvements:** - Container stays running between test runs (no startup delay) - BuildKit with cache mounts for faster Go builds - Real HTTP testing eliminates in-memory test setup overhead **Commands:** - `./run_integration_tests.sh start` - Start persistent test container - `./run_integration_tests.sh test` - Run tests (container stays running) - `./run_integration_tests.sh status` - Check container status - `./run_integration_tests.sh clean` - Cleanup when needed Fixes issue where git commands were trying to contact GitHub by ensuring mock scripts are properly mounted and prioritized in PATH. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.