Merged
Conversation
…ry, server Final tests to push project coverage past 85%. UI: - ui/src/App.test.tsx (12 tests) — App.tsx 0% → 92.75% lines / 100% branches. Covers route resolution, provider mounting, login gate, QueryClient retry path. Mocks SSE/route-heavy children consistent with the existing repo pattern. - ui/src/components/QuotaStrip.test.tsx (11 tests) — QuotaStrip.tsx 0% → 100% (lines, branches, funcs). Covers all quota-band states and time-formatting branches with frozen system time. Go: - internal/session/state_extra_test.go — state.go 62.8% → 82.0%. UpdateHealth, UpdateAttached, UpdateMode, Names, Delete, Rename (incl. conflict + invalid name), Backup-on-empty, DeleteAll edge cases. - internal/serve/api/feed_history_extra_test.go — feed_history.go 72.8% → 85.3%. extractTS variants, nestedBool branches, summariseHistoryInput / summariseHistoryResponse paths, truncateHistory, splitIDExt + idLessThanExt + synthEvent. - internal/serve/server_more_test.go — server.go 82.5% → 86.3%. costSourceAdapter Range/Totals errors, logsUUIDResolver guards, ResolveName / ResolveUUID workdir-fallback paths, sessionSourceAdapter LastCheckpointAt with a real git fixture, New defaults branches with a sandboxed HOME, port-in-use-by-non-ctm detection. Skipped (still integration-bound, won't be tested as units): - Spawn / SendInitialPrompt (live tmux + claude subprocess). - quotaEnricher Attention/Tokens success branches (need ingest + attention engine event-loop integration). - Run loop's 30s ticker / orphan-UUID adoption details (timing- dependent + project-mounted fs walks). Verification: - 918 Go tests pass across 27 packages with -tags sqlite_fts5 (was 869). - 206 UI tests pass (was 183). Co-Authored-By: Claude Opus 4.7 (1M context) <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
Last tests needed to clear the 85% project-coverage target.
Coverage uplift
ui/src/App.tsxui/src/components/QuotaStrip.tsxinternal/session/state.gointernal/serve/api/feed_history.gointernal/serve/server.goVerification
Expected effect
coverage: 82.8% → >85% ✅ goal.🤖 Generated with Claude Code