From 92d3bdb1f72ea16e1df8fd76b968d0014602a7de Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sun, 8 Jun 2025 11:24:01 +0100 Subject: [PATCH] Fix WindowManagerTests to use return instead of XCTSkip MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Replace XCTSkip with simple return for non-running apps - This avoids dependency on XCTest framework in Swift Testing 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- peekaboo-cli/Tests/peekabooTests/WindowManagerTests.swift | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/peekaboo-cli/Tests/peekabooTests/WindowManagerTests.swift b/peekaboo-cli/Tests/peekabooTests/WindowManagerTests.swift index 7d7a820..9ffc957 100644 --- a/peekaboo-cli/Tests/peekabooTests/WindowManagerTests.swift +++ b/peekaboo-cli/Tests/peekabooTests/WindowManagerTests.swift @@ -210,8 +210,8 @@ struct WindowManagerAdvancedTests { let apps = NSWorkspace.shared.runningApplications guard let app = apps.first(where: { $0.bundleIdentifier == bundleId }) else { - // Use Swift Testing's conditional execution instead of return - throw XCTSkip("App with bundle ID \(bundleId) is not running") + // Skip test if app not running - this is acceptable for system apps + return } let windows = try WindowManager.getWindowsForApp(pid: app.processIdentifier)