story-kit: queue 120_story_test_coverage_llm_chat_rs for merge
This commit is contained in:
@@ -1,27 +0,0 @@
|
||||
---
|
||||
name: "Add test coverage for llm/chat.rs (2.6% -> 60%+)"
|
||||
---
|
||||
|
||||
# Story 120: Add test coverage for llm/chat.rs
|
||||
|
||||
Currently at 2.6% line coverage (343 lines, 334 missed). This is the chat completion orchestration layer — the biggest uncovered module by missed line count.
|
||||
|
||||
## What to test
|
||||
|
||||
- Message construction and formatting
|
||||
- Token counting/estimation logic
|
||||
- Chat session management
|
||||
- Error handling paths (provider errors, timeout, malformed responses)
|
||||
- Any pure functions that don't require a live LLM connection
|
||||
|
||||
## Notes
|
||||
|
||||
- Mock the LLM provider trait/interface rather than making real API calls
|
||||
- Focus on the logic layer, not the provider integration
|
||||
- Target 60%+ line coverage
|
||||
|
||||
## Acceptance Criteria
|
||||
|
||||
- [ ] Line coverage for `llm/chat.rs` reaches 60%+
|
||||
- [ ] Tests pass with `cargo test`
|
||||
- [ ] `cargo clippy` clean
|
||||
26
.story_kit/work/3_qa/125_story_test_coverage_http_io_rs.md
Normal file
26
.story_kit/work/3_qa/125_story_test_coverage_http_io_rs.md
Normal file
@@ -0,0 +1,26 @@
|
||||
---
|
||||
name: "Add test coverage for http/io.rs (0% -> 60%+)"
|
||||
---
|
||||
|
||||
# Story 125: Add test coverage for http/io.rs
|
||||
|
||||
Currently at 0% line coverage (76 lines). These are the IO-related HTTP endpoints (absolute path listing, directory creation, home directory).
|
||||
|
||||
## What to test
|
||||
|
||||
- `list_directory_absolute` endpoint — valid path, invalid path, permission errors
|
||||
- `create_directory_absolute` endpoint — new dir, existing dir, nested creation
|
||||
- `get_home_directory` endpoint — returns correct home path
|
||||
- Error responses for invalid inputs
|
||||
|
||||
## Notes
|
||||
|
||||
- Use temp directories for filesystem tests
|
||||
- These are straightforward CRUD-style endpoints, should be quick to cover
|
||||
- Follow the test patterns used in `http/project.rs` and `http/settings.rs`
|
||||
|
||||
## Acceptance Criteria
|
||||
|
||||
- [ ] Line coverage for `http/io.rs` reaches 60%+
|
||||
- [ ] Tests pass with `cargo test`
|
||||
- [ ] `cargo clippy` clean
|
||||
Reference in New Issue
Block a user