Properly disable testBuildCompleteMessage on Linux for SwiftBuild (#8444)

bkhouri · web-flow · commit b175dd8036c3 · 2025-04-02T17:11:57.000-07:00
diff --git a/Tests/CommandsTests/BuildCommandTests.swift b/Tests/CommandsTests/BuildCommandTests.swift
@@ -941,9 +941,10 @@ class BuildCommandSwiftBuildTests: BuildCommandTestCases {
 
     override func testBuildCompleteMessage() async throws {
         #if os(Linux)
-        try XCTSkip("SWBINTTODO: Need to properly set LD_LIBRARY_PATH on linux")
-        #endif
+        throw XCTSkip("SWBINTTODO: Need to properly set LD_LIBRARY_PATH on linux")
+        #else
         try await super.testBuildCompleteMessage()
+        #endif
     }
 
 }

Original file line number	Diff line number	Diff line change
`@@ -941,9 +941,10 @@ class BuildCommandSwiftBuildTests: BuildCommandTestCases {`
`941`	`941`
`942`	`942`	`override func testBuildCompleteMessage() async throws {`
`943`	`943`	`#if os(Linux)`
`944`		`- try XCTSkip("SWBINTTODO: Need to properly set LD_LIBRARY_PATH on linux")`
`945`		`- #endif`
	`944`	`+ throw XCTSkip("SWBINTTODO: Need to properly set LD_LIBRARY_PATH on linux")`
	`945`	`+ #else`
`946`	`946`	`try await super.testBuildCompleteMessage()`
	`947`	`+ #endif`
`947`	`948`	`}`
`948`	`949`
`949`	`950`	`}`