mirror of
https://github.com/storytold/designcraft.git
synced 2026-10-24 10:16:59 -06:00
skylar taylor-barrick
c43a63f71e
The merge compiled, but six tests showed where the two branches meant different things. Fixes, in order of consequence: - Format version: both branches had independently defined version 2 — here for retained EPS sources, on main for pinned Keep Lines Together and drop cap defaults, with an `upgrade` that runs below 2. Writing 1 for documents without EPS then re-ran that upgrade on load and rewrote their paragraph styles, so saving and reloading changed the document. EPS now asks for version 3 and everything else stays at 2. - place.drop into a frame: main turned Auto-Fit off so that resizing a frame crops rather than refits, and this branch had carried the older FillProportionally along while moving the geometry out of the edit. Main's behaviour is the intended one. - save() compacts unreferenced assets now, so the format test has to place its EPS for the version sniff to see it at all. - EPUB entries are named from the asset's file stem (asset_path), not its bare id: images/logo-<id>.png. - Command parameters are validated strictly, so the PDF page test sends each command only the parameters it takes. - The relink batch's "commits once" check read a process-wide revision counter, which a test running beside it can advance; the single undo step is what actually shows it committed once. The EPS place tests move to cmd/file/eps_place_tests.rs: they write real files, and main's file-system guard reads src files without seeing inline #[cfg(test)] modules, but exempts _tests.rs ones.