feat: Add unit test workflow and rename service health checks#187
Closed
NotYuSheng wants to merge 3 commits into
Closed
feat: Add unit test workflow and rename service health checks#187NotYuSheng wants to merge 3 commits into
NotYuSheng wants to merge 3 commits into
Conversation
## Changes Made - **Unit Test Workflow**: Created `.github/workflows/unit-tests.yml` using existing `./scripts/test-all-services.sh` - **Disabled Cypress**: Created `.github/workflows/e2e-tests.yml.disabled` since frontend is not ready for E2E testing - **Documentation**: Updated CLAUDE.md to reflect current testing status ## Testing Infrastructure Updates - **Automated Unit Tests**: GitHub Actions will now run all 206+ unit tests across 7 services - **CI Integration**: Unit tests run on push/PR to dev branch - **Resource Management**: Includes disk space cleanup and Docker optimization - **Artifact Upload**: Test logs uploaded on failure for debugging ## Rationale - Replace integration testing approach with comprehensive unit test automation - Preserve Cypress setup for future use when frontend is ready - Maintain consistent testing standards while improving CI efficiency 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Renamed .github/workflows/integration-test.yaml to service-health-checks.yml - Updated workflow name and job name to reflect actual functionality - Removed Node.js and Cypress setup steps since they were placeholder-only - Workflow now focuses purely on service health checks: build, start, and verify each *_service 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Contributor
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
- Updated unit-tests.yml to test services individually with health checks - Each service gets built, dependencies started, health checked, then unit tested - Removed separate service-health-checks.yml workflow to avoid duplication - Added proper resource management with container cleanup between tests - Workflow now efficiently combines health validation and unit testing 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
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
1 participant
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.
Summary
scripts/test-all-services.shChanges Made
.github/workflows/unit-tests.ymlintegration-test.yaml→service-health-checks.ymle2e-tests.ymlto.disabledextensionTest Plan
🤖 Generated with Claude Code