Add unit tests to improve test coverage #1
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 comprehensive tests for formatRelativeTime (0% -> 100%) - Add tests for getProjectLastUpdate (80% -> 100%) - Add tests for parseMessage edge cases (76.7% -> 86%) - Add tests for DiscoverProjects edge cases (81% -> 90.5%) - Add tests for ParseSession with CWD and summary fallback (63.8% -> 87.2%) Overall coverage improved from 58.8% to 62.2%
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.
Summary
This PR adds unit tests to increase the overall test coverage from 58.8% to 62.2%.
Changes
New Tests in
cmd_test.goTestFormatRelativeTime: Comprehensive tests for time difference formatting (seconds, minutes, hours, days, weeks)TestFormatRelativeTimeBoundaries: Tests for boundary conditions between time intervalsNew Tests in
parser_test.goTestGetProjectLastUpdate: Tests project last update time extraction with various session combinationsTestParseMessage_EdgeCases: Tests empty message content handlingTestParseMessage_NullParentUUID: Tests nil parent UUID handlingTestParseMessage_WithParentUUID: Tests valid parent UUID assignmentTestParseMessage_AlternateTimestampFormat: Tests RFC3339 timestamp parsingTestParseMessage_InvalidTimestamp: Tests fallback behavior for invalid timestampsTestParseSession_WithCWD: Tests CWD extraction from sessionsTestParseSession_SummaryFallback: Tests using first user message as summary fallbackTestParseSession_SummaryFallbackTruncation: Tests long message truncationTestDiscoverProjects_SkipsNonEncodedPaths: Tests filtering of non-encoded directoriesTestDiscoverProjects_NotADirectory: Tests error handling for file pathsCoverage Improvements
formatRelativeTimegetProjectLastUpdateparseMessageDiscoverProjectsParseSessionTesting
All tests pass:
@hancengiz can click here to continue refining the PR