test(dashboard): add and improve tests to achieve >80% code coverage [#326]#399
Merged
youngtaekiim merged 5 commits intoeclipse-pullpiri:mainfrom Dec 22, 2025
Merged
Conversation
- Added comprehensive unit tests for Overview.tsx and ImageWithFallback.tsx - Enhanced test coverage for src/tools/dashboard components - Ensured all critical paths and error handling are covered - Code coverage for src/tools/dashboard now exceeds 80%
Contributor
Author
|
Hey @daeyoung-jeong-lge @youngtaekiim @leeeunkoo |
Contributor
|
@chaehee-lim This PR is for testing Dashboard which is built with React. So, could you check this PR? Thanks in advance. :-) |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds comprehensive unit tests to the dashboard component to achieve >80% code coverage. The testing infrastructure is set up using Vitest and React Testing Library, with coverage thresholds enforced for lines, statements, functions, and branches.
Key Changes:
- Added testing infrastructure and dependencies (Vitest, React Testing Library, Jest types)
- Documented test commands and coverage requirements in README
- Established 80% coverage thresholds across all metrics
Reviewed changes
Copilot reviewed 2 out of 112 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/tools/dashboard/README.md | Added documentation for running tests and coverage reporting |
| package.json | Added testing dependencies for Vitest and React Testing Library |
Contributor
|
@youngtaekiim I've confirmed. Merge please. |
youngtaekiim
approved these changes
Dec 22, 2025
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
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.
📝 PR Description
test(dashboard): add and improve tests to achieve >80% code coverage
🔗 Related Issue
Closes #326
🧪 Test Method
cd src/tools/dashboard
npm install
npm run test:coverage
Screenshot:
✅ Checklist
[✅] Code conventions are followed
[✅] Tests are added/modified
[✅] Documentation is updated (if necessary)