Add an integration fixture for the folded named-line overlay #54

Open
opened 2026-07-03 20:10:21 +00:00 by toph · 0 comments
Owner

The folded-view construction overlay in tools/fold2svg.mjs — named lines clipped per-face and drawn on the folded form — has no integration fixture. Every committed example whose named lines reach the folded view has them as crease-duplicates, which the overlay skips, so tools/test/foldview.test.mjs never exercises the lineToFace clip path end-to-end — only the lineToFace unit.

This surfaced in #31: the per-frame clip fix (bcb2c10) rests on the lineToFace unit test plus a structural argument (the folded loop now clips against each frame's own table polygon, so no crease-pattern index can leak in), but a regression in the overlay itself would fail no test. Add a fixture that combines a genuine auxiliary (non-crease) named line with multiple --step frames, and assert the drawn construction line lands within the selected frame's faces.

For #31.

The folded-view construction overlay in `tools/fold2svg.mjs` — named lines clipped per-face and drawn on the folded form — has no integration fixture. Every committed example whose named lines reach the folded view has them as crease-duplicates, which the overlay skips, so `tools/test/foldview.test.mjs` never exercises the `lineToFace` clip path end-to-end — only the `lineToFace` unit. This surfaced in #31: the per-frame clip fix (`bcb2c10`) rests on the `lineToFace` unit test plus a structural argument (the folded loop now clips against each frame's own table polygon, so no crease-pattern index can leak in), but a regression in the overlay itself would fail no test. Add a fixture that combines a genuine auxiliary (non-crease) named line with multiple `--step` frames, and assert the drawn construction line lands within the selected frame's faces. For #31.
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
toph/beloch#54
No description provided.