Skip to content

Commit 6b26058

Browse files
committed
chore(test): clean dead fixture-cache substring reference (#100)
Round-2 follow-up identified by /idd-verify round-2 logic Agent: the fixture-type detection in NoteHTMLConvertTests.swift still referenced the removed `macdoc-note-fixture-cache` substring. Detection still worked because the OR branch on `synthetic-fixture` matched, but the first branch was dead code. Replaced with the canonical synthetic-fixture marker per the new no-cache resolution path: `macdoc-synthetic-fixture-<UUID>`. No behavior change — same fixture-type detection outcome. Refs #100
1 parent 437c2c9 commit 6b26058

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

Tests/MacDocCLITests/NoteHTMLConvertTests.swift

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,10 @@ final class NoteHTMLConvertTests: XCTestCase {
5757
// "media ≥ 1 asset" + ">750 KB index.html" floors per #81 calibration.
5858
// - Synthetic fixtures (NoteFixtureGenerator output, per #100) have no
5959
// media + minimal strokes → only structural assertions apply.
60-
let isSyntheticFixture = fixture.path.contains("macdoc-note-fixture-cache")
61-
|| fixture.lastPathComponent.contains("synthetic-fixture")
60+
// Detection: synthetic fixtures live in $TMPDIR with the
61+
// `macdoc-synthetic-fixture-` prefix (per CLITestHelper.swift, no
62+
// cache subdirectory since #100 round-2 fix).
63+
let isSyntheticFixture = fixture.lastPathComponent.contains("macdoc-synthetic-fixture")
6264

6365
let mediaFiles = try FileManager.default.contentsOfDirectory(
6466
at: mediaURL,

0 commit comments

Comments
 (0)