Skip to content

Commit c1048f7

Browse files
committed
Limit test suite to essential data-loss safeguards and update README
- Remove large/integration test modules that were flaky or non-essential: - Delete tests/brain_cli (CLI integration tests) and its __init__. - Delete notes_search_client and template_generator test modules. - Narrow brain_core tests to focus on critical safety checks: - Keep only essential config validation (missing DIARY_PATH / PLANNER_PATH). - Keep EntryManager/DiaryEntry tests that verify filenames, separate planner path, and basic write/read cycles. - Remove many higher-level and integration-style assertions to speed up and stabilize CI. - Update README Testing section to document the minimal test suite and adjusted coverage summary. Reason: reduce test noise and flakiness while retaining coverage for operations that could cause data loss or corruption.
1 parent 4aec9d5 commit c1048f7

File tree

7 files changed

+22
-1089
lines changed

7 files changed

+22
-1089
lines changed

README.md

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -168,13 +168,18 @@ After writing, run `brain diary link today` to add semantic backlinks:
168168

169169
## Testing
170170

171+
Minimal test suite focused on preventing data loss:
172+
171173
```bash
172174
uv run pytest # Run all tests
173175
uv run pytest --cov # Run with coverage
174-
uv run pytest tests/brain_core/ # Run specific module
175176
```
176177

177-
**Coverage**: 74 tests with 45% overall coverage, 96% on critical paths (plan_commands.py)
178+
**Coverage**: 7 essential tests covering:
179+
- Configuration validation (missing paths)
180+
- File naming (reflection vs. plan entries)
181+
- Write/read cycles (prevent data loss)
182+
- Path separation (diary vs. planner)
178183

179184
## License
180185

tests/brain_cli/__init__.py

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)