Skip to content

Commit 5f250e3

Browse files
committed
test(pdf-server): mark tombstone e2e as fixme — needs basic-host iframe-reload replay support
1 parent b1f5fd3 commit 5f250e3

1 file changed

Lines changed: 10 additions & 1 deletion

File tree

tests/e2e/pdf-incremental-load.spec.ts

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,16 @@ test.describe("PDF Server — incremental loading", () => {
158158
// range-counting fixture is the convenient place; it does not use any
159159
// byte-accounting features.
160160
test.describe("PDF Server — annotation tombstone preservation", () => {
161-
test("deleted native annotation tombstone survives a persist before its page is scanned", async ({
161+
// FIXME: the load-bearing scenario requires the viewer to re-seed
162+
// restoredRemovedIds from localStorage with page 2 unscanned, which means
163+
// an iframe reload that keeps the same toolId. basic-host doesn't re-supply
164+
// the cached tool result on inner-iframe reload, so the viewer never gets
165+
// data and the canvas never appears. A fresh display_pdf call gets a new
166+
// toolId → new storage key → restore misses. Unblocking needs either a
167+
// basic-host hook to replay the tool result, or a URL-keyed (not
168+
// toolId-keyed) annotation store. The fix itself is covered by the
169+
// computeDiff/serializeDiff contract tests in src/pdf-annotations.test.ts.
170+
test.fixme("deleted native annotation tombstone survives a persist before its page is scanned", async ({
162171
page,
163172
}) => {
164173
// Regression for the lazy baseline scan: restoredRemovedIds must be

0 commit comments

Comments
 (0)