docs(bf-3oy5): verify SemanticNarrativePanel tests - already fixed
Verified that the 3 failing tests in SemanticNarrativePanel.test.ts were
already fixed in commit 62370103. The issue was test mocks missing required
fields (workerId, events, startTime, endTime, durationMs) added to the
SemanticNarrative type.
All 59 tests now pass. The implementation methods (refresh, updateFromWorker,
updateAggregated) were always correct - the failure was purely test data.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
parent
634aaa7498
commit
3a3d136342
1 changed files with 36 additions and 0 deletions
36
notes/bf-3oy5.md
Normal file
36
notes/bf-3oy5.md
Normal file
|
|
@ -0,0 +1,36 @@
|
|||
# BF-3OY5: SemanticNarrativePanel Tests - Already Fixed
|
||||
|
||||
## Issue
|
||||
3 tests in `src/tui/components/SemanticNarrativePanel.test.ts` were reported as failing:
|
||||
- "should refresh narrative from manager"
|
||||
- "should generate narrative for worker and set it"
|
||||
- "should generate aggregated narrative and set it"
|
||||
|
||||
## Root Cause
|
||||
The tests were failing due to missing required fields in mock narrative objects. When the `SemanticNarrative` type was updated to require:
|
||||
- `workerId: string`
|
||||
- `events: LogEvent[]`
|
||||
- `startTime: number`
|
||||
- `endTime: number`
|
||||
- `durationMs: number`
|
||||
|
||||
The test mocks were not updated accordingly, causing type mismatches and runtime errors.
|
||||
|
||||
## Fix Applied
|
||||
The fix was already applied in commit `62370103685d6f629e7335d397c6dadb1c4cfe23`:
|
||||
- Added all required fields to `createMockNarrative()` helper
|
||||
- Added missing fields to all inline narrative segment mocks throughout tests
|
||||
- Changed `entities.workers` array (incorrect) to `workerId` string (correct)
|
||||
|
||||
## Verification
|
||||
All 59 tests in `SemanticNarrativePanel.test.ts` now pass:
|
||||
```
|
||||
✓ src/tui/components/SemanticNarrativePanel.test.ts (59 tests) 37ms
|
||||
```
|
||||
|
||||
The implementation methods (`refresh()`, `updateFromWorker()`, `updateAggregated()`) were always correct - they properly:
|
||||
1. Call the SemanticNarrative manager methods
|
||||
2. Pass results to `setNarrative()`
|
||||
3. Trigger render via `this.box.screen.render()`
|
||||
|
||||
The failure was purely a test data issue, not an implementation bug.
|
||||
Loading…
Add table
Reference in a new issue