From b8ea201bd3148f5274d27c1d3a3222eaa0e63100 Mon Sep 17 00:00:00 2001 From: Mathias Date: Mon, 5 Oct 2026 20:48:25 +0200 Subject: [PATCH] test(pipeline): the page-swap test's positive control asserts the source ref (#157) Review round 2 on PR #176: the control checked for text that was in the page before BackfillRefs ran, so the test stayed green when no page was read at all. It now asserts the [[s|S]] ref BackfillRefs adds; red when the page read always fails, still red when the read follows the link. Branch rebased onto main. Co-Authored-By: Claude Opus 5.5 --- ingestion/internal/pipeline/root_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ingestion/internal/pipeline/root_test.go b/ingestion/internal/pipeline/root_test.go index f5ca876..b9f7b6c 100644 --- a/ingestion/internal/pipeline/root_test.go +++ b/ingestion/internal/pipeline/root_test.go @@ -162,6 +162,6 @@ func TestBackfillRefsDoesNotCopyAPageSwappedInBehindASymlink(t *testing.T) { return string(b) } require.NoError(t, os.WriteFile(page, []byte(inside), 0o644)) - require.Contains(t, read(), "zqinside", "positive control: the regular page gets the ref") + require.Contains(t, read(), "[[s|S]]", "positive control: the regular page gets the source ref") brainfstest.AssertNoLeakFrom(t, page, inside, target, 3*time.Second, read) }