Skip to content

Commit befe9c0

Browse files
committed
test(super-editor): synthesize ResolvedLayout in PresentationEditor mock (SD-2836)
rebuildRegions now iterates resolvedLayout.pages (was layout.pages). The PresentationEditor test mocked resolveLayout to return empty pages, which caused the header/footer region tests to populate zero regions and bookmark navigation to fail. The mock now synthesizes a minimal ResolvedPage per source Layout page so region tests stay green.
1 parent 54c94bc commit befe9c0

1 file changed

Lines changed: 28 additions & 1 deletion

File tree

packages/super-editor/src/editors/v1/core/presentation-editor/tests/PresentationEditor.test.ts

Lines changed: 28 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,34 @@ const {
182182
getActiveEditorHost: vi.fn(() => null),
183183
destroy: vi.fn(),
184184
})),
185-
mockResolveLayout: vi.fn(() => ({ version: 1, flowMode: 'paginated', pageGap: 0, pages: [] })),
185+
// SD-2836: rebuildRegions now iterates resolvedLayout.pages, so the mock
186+
// must synthesize a ResolvedPage per source Layout page to keep header/footer
187+
// region tests from going empty.
188+
mockResolveLayout: vi.fn(
189+
({ layout }: { layout: { pages: Array<Record<string, unknown>>; pageSize: { w: number; h: number } } }) => ({
190+
version: 1,
191+
flowMode: 'paginated',
192+
pageGap: 0,
193+
pages: (layout?.pages ?? []).map((p, i) => ({
194+
id: `page-${i}`,
195+
index: i,
196+
number: (p.number as number) ?? i + 1,
197+
width: ((p.size as { w?: number } | undefined)?.w ?? layout.pageSize?.w) as number,
198+
height: ((p.size as { h?: number } | undefined)?.h ?? layout.pageSize?.h) as number,
199+
items: [],
200+
margins: p.margins,
201+
sectionRefs: p.sectionRefs,
202+
sectionIndex: p.sectionIndex,
203+
numberText: p.numberText,
204+
footnoteReserved: p.footnoteReserved,
205+
vAlign: p.vAlign,
206+
baseMargins: p.baseMargins,
207+
orientation: p.orientation,
208+
columns: p.columns,
209+
columnRegions: p.columnRegions,
210+
})),
211+
}),
212+
),
186213
mockFlowBlockCacheInstances,
187214
MockFlowBlockCache,
188215
};

0 commit comments

Comments
 (0)