From 0cf596aa8ac724e3ae4558ecc457a65ce19bda7e Mon Sep 17 00:00:00 2001 From: Safi Date: Sun, 31 May 2026 00:10:19 +0100 Subject: [PATCH] fix test_no_incremental_without_manifest false positive pytest derives tmp_path from the test name, so the path contained "incremental" and the bare-word assertion always failed; check for the actual incremental-mode phrases instead Co-Authored-By: Claude Sonnet 4.6 --- tests/test_incremental.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/test_incremental.py b/tests/test_incremental.py index 15fd9f1a..2e2e902e 100644 --- a/tests/test_incremental.py +++ b/tests/test_incremental.py @@ -54,4 +54,8 @@ def test_no_incremental_without_manifest(tmp_path): """Without manifest.json, full scan message is shown (not incremental).""" docs = _make_docs_corpus(tmp_path) r = _run(["extract", str(docs)], tmp_path) - assert "incremental" not in r.stdout + # Check combined output doesn't contain incremental-mode phrasing. + # Use a phrase rather than a bare word to avoid matching the tmp_path, + # which pytest derives from the test name and contains "incremental". + assert "incremental update" not in r.stdout.lower() + assert "incremental scan" not in r.stdout.lower()