correct a stale untested list: two of its four entries already had tests
This commit is contained in:
@@ -978,8 +978,15 @@ they are marked below, because a dead-code list that is itself wrong is the wors
|
||||
`onClearApp` → `null`), but `setApp` is exported from the barrel. Fixed in the same commit.
|
||||
Two tests are deliberately written to the _current_ behaviour and marked `KNOWN GAP` so they fail
|
||||
the day it changes: a move drops `zoom` and `fitContent` (§5.3).
|
||||
**Still untested:** the other eleven files of the framework — `WorkspaceView`, `WorkspaceRenderer`,
|
||||
`PanelSlot`, `normalizeLayout`. §5.5's debounce lost-updates live there, not in `layout-utils`.
|
||||
**The "still untested" list this line used to carry was stale by the time it was written, and two of
|
||||
its four entries were wrong.** `normalizeLayout` is not a separate file — it lives in `layout-utils.ts`
|
||||
and has nine tests of its own (identity-by-reference on a clean tree, `config` dropped along with the
|
||||
app it belonged to, idempotence, nested repair, `null` always allowed). `WorkspaceRenderer` got eight
|
||||
tests in `56ca411`. The Workspace directory is 76 tests across three files and green.
|
||||
**Genuinely still untested: `WorkspaceView` and `PanelSlot`.** But note §5.5's lost updates are no
|
||||
longer what makes that urgent — every mutation in `WorkspaceView` now goes through `onLayoutChange`
|
||||
as an *updater*, never as a computed tree, which is the structural fix; a test there would be
|
||||
guarding the fix rather than finding the bug. Checked, not assumed — `bun test src/workspaces/officerdev/src/components/Workspace/`.
|
||||
|
||||
- [x] **A panel is told where it is, instead of being handed a key to reverse-engineer.** _(`dbe585f` +
|
||||
`717580f` + `585f234`, branch `agent-coordination-mvp`)_ — the first three items of §5.9. The
|
||||
|
||||
Reference in New Issue
Block a user