Skip to content

test: comprehensive unit tests for export-timeline - #142

Closed
TerminalGravity wants to merge 5 commits into
mainfrom
test/export-timeline-coverage
Closed

test: comprehensive unit tests for export-timeline#142
TerminalGravity wants to merge 5 commits into
mainfrom
test/export-timeline-coverage

Conversation

@TerminalGravity

Copy link
Copy Markdown
Collaborator

Adds 20 unit tests for the export-timeline tool's pure functions (computeStats, generateMarkdownReport, parseRelativeDate). Previously only had 1 smoke test checking the export existed.

Exports three internal functions for direct testing — no behavioral changes.

Export computeStats, generateMarkdownReport, and parseRelativeDate
for direct testing. Add 20 tests covering:
- parseRelativeDate with days/weeks/months/singular/passthrough
- computeStats aggregation, empty input, missing timestamps
- generateMarkdownReport sections, formatting, conditional rendering
@TerminalGravity

Copy link
Copy Markdown
Collaborator Author

Export-timeline tests look comprehensive. Good edge case coverage.

@TerminalGravity

Copy link
Copy Markdown
Collaborator Author

Superseded by #154

@TerminalGravity
TerminalGravity deleted the test/export-timeline-coverage branch March 8, 2026 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant