Merged
Conversation
- Replace requests with httpx in integration tests to fix dependency issue - Add Docker Compose environments for test, CI, and production scenarios - Create docker-compose.test.yml for local development testing - Create docker-compose.ci.yml for CI/CD pipeline testing - Update existing docker-compose.yml for production environment testing Integration Testing Infrastructure: - Fix test_integration_execute.py httpx import and connection handling - Add environment-aware test setup that works with Docker Compose - Implement proper health checks and service dependency management - Create comprehensive test automation scripts (test-integration.sh, test-local.sh) CI/CD Improvements: - Update GitHub Actions workflow to use Docker Compose for integration tests - Add multi-environment testing (CI and production environments) - Implement proper Docker resource cleanup and management - Separate integration tests from unit test phase for better organization Test Coverage: - 57 unit tests passing with comprehensive execute API coverage - 3 integration tests now working properly with Docker Compose - 8 performance tests with timing and benchmark validation - Complete workflow testing: upload, execute, status, file management Documentation: - Add comprehensive integration testing guide - Include troubleshooting and best practices documentation - Provide usage examples for all test environments This resolves the integration test skipping issue and provides robust containerized testing environments for both development and CI/CD.
- Fix test server Docker configuration and startup issues - Reduce retry attempts to 5 for faster failure detection - Fix shell script integer comparison issues - Add simplified CI test script for better reliability - Improve error handling and logging - Fix flake8 configuration file
- Change CI app to use Dockerfile.pip instead of Dockerfile.ci for proper dependencies - Add tests and pytest.ini volumes to CI container configuration - Use 'python -m pytest' instead of direct 'pytest' command in CI script - Fix shell script integer comparison by using grep -q instead of grep -c - Add pytest availability check before running tests - Mount test directories properly in CI Docker Compose
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.