From 2937d2d5e02be50f27d08e8d0e1b900fdce6c21f Mon Sep 17 00:00:00 2001 From: kua-agent Date: Sat, 5 Sep 2026 09:10:43 +0400 Subject: [PATCH] fix: correct recordUncommented test names (were misspelled "commended") MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Two @Test descriptions and function names in ReturnWatcherTests.swift used "recordUncommended" (commended, as in praised) instead of "recordUncommented" (commented, as in has a comment) — a typo introduced when the tests were added. The actual public API (ReturnWatcher.recordUncommented, setRecordUncommented) was already spelled correctly everywhere; only these two test names/descriptions needed fixing. Co-Authored-By: Claude Fable 5.1 Claude-Session: https://claude.ai/code/session_012ZiTXPbPCSjzPVsfoweAbp --- Tests/ShotdeckCoreTests/ReturnWatcherTests.swift | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Tests/ShotdeckCoreTests/ReturnWatcherTests.swift b/Tests/ShotdeckCoreTests/ReturnWatcherTests.swift index d396d1f..3278649 100644 --- a/Tests/ShotdeckCoreTests/ReturnWatcherTests.swift +++ b/Tests/ShotdeckCoreTests/ReturnWatcherTests.swift @@ -231,8 +231,8 @@ func w27_fsEventsCallbackFiresOnRealArrival() async throws { } } -@Test("recordUncommended defaults to true: an unmarked PDF is still recorded (AirDrop behaviour unchanged)") -func recordUncommendedDefaultTrueRecordsAnUnmarkedPDF() async throws { +@Test("recordUncommented defaults to true: an unmarked PDF is still recorded (AirDrop behaviour unchanged)") +func recordUncommentedDefaultTrueRecordsAnUnmarkedPDF() async throws { let (paths, cleanup) = try makeCasePaths() defer { try? FileManager.default.removeItem(at: cleanup) } @@ -250,8 +250,8 @@ func recordUncommendedDefaultTrueRecordsAnUnmarkedPDF() async throws { #expect(all.count == 1) } -@Test("recordUncommended=false: an unmarked PDF is not recorded or returned; marking it up in place gets it recorded") -func recordUncommendedFalseSkipsUnmarkedThenRecordsAfterInPlaceMarkup() async throws { +@Test("recordUncommented=false: an unmarked PDF is not recorded or returned; marking it up in place gets it recorded") +func recordUncommentedFalseSkipsUnmarkedThenRecordsAfterInPlaceMarkup() async throws { let (paths, cleanup) = try makeCasePaths() defer { try? FileManager.default.removeItem(at: cleanup) }