Merge pull request #2160 from aciidb0mb3r/moar-xctest

Support new XCTest overlays

Author: aciidgh

Sources/Commands/SwiftTestTool.swift

@@ -477,8 +477,9 @@ public class SwiftTestTool: SwiftTool<TestToolOptions> {
         var env = try constructTestEnvironment(toolchain: try getToolchain(), options: self.options, buildParameters: self.buildParameters())
         // Add the sdk platform path if we have it. If this is not present, we
         // might always end up failing.
-        if let sdkPlatformFrameworksPath = Destination.sdkPlatformFrameworkPath() {
-            env["DYLD_FRAMEWORK_PATH"] = sdkPlatformFrameworksPath.pathString
+        if let sdkPlatformFrameworksPath = Destination.sdkPlatformFrameworkPaths() {
+            env["DYLD_FRAMEWORK_PATH"] = sdkPlatformFrameworksPath.fwk.pathString
+            env["DYLD_LIBRARY_PATH"] = sdkPlatformFrameworksPath.lib.pathString
         }
         try Process.checkNonZeroExit(arguments: args, environment: env)
         // Read the temporary file's content.

Sources/TestSupport/Resources.swift

@@ -36,7 +36,7 @@ public class Resources: ManifestResourceProvider {
 
   #if os(macOS)
     public var sdkPlatformFrameworksPath: AbsolutePath {
-        return Destination.sdkPlatformFrameworkPath()!
+        return Destination.sdkPlatformFrameworkPaths()!.fwk
     }
   #endif
 

Sources/Workspace/Destination.swift

@@ -103,15 +103,21 @@ public struct Destination {
         }
 
         // Compute common arguments for clang and swift.
-        // This is currently just frameworks path.
-        let commonArgs = Destination.sdkPlatformFrameworkPath(environment: environment).map({ ["-F", $0.pathString] }) ?? []
+        var extraCCFlags: [String] = []
+        var extraSwiftCFlags: [String] = []
+        if let sdkPaths = Destination.sdkPlatformFrameworkPaths(environment: environment) {
+            extraCCFlags += ["-F", sdkPaths.fwk.pathString]
+            extraSwiftCFlags += ["-F", sdkPaths.fwk.pathString]
+            extraSwiftCFlags += ["-I", sdkPaths.lib.pathString]
+            extraSwiftCFlags += ["-L", sdkPaths.lib.pathString]
+        }
 
         return Destination(
             target: hostTargetTriple,
             sdk: sdkPath,
             binDir: binDir,
-            extraCCFlags: commonArgs,
-            extraSwiftCFlags: commonArgs,
+            extraCCFlags: extraCCFlags,
+            extraSwiftCFlags: extraSwiftCFlags,
             extraCPPFlags: ["-lc++"]
         )
       #else
@@ -127,21 +133,31 @@ public struct Destination {
     }
 
     /// Returns macosx sdk platform framework path.
-    public static func sdkPlatformFrameworkPath(environment: [String:String] = Process.env) -> AbsolutePath? {
+    public static func sdkPlatformFrameworkPaths(
+        environment: [String:String] = Process.env
+    ) -> (fwk: AbsolutePath, lib: AbsolutePath)? {
         if let path = _sdkPlatformFrameworkPath {
             return path
         }
         let platformPath = try? Process.checkNonZeroExit(
-            arguments: ["xcrun", "--sdk", "macosx", "--show-sdk-platform-path"], environment: environment).spm_chomp()
+            arguments: ["xcrun", "--sdk", "macosx", "--show-sdk-platform-path"],
+            environment: environment).spm_chomp()
 
         if let platformPath = platformPath, !platformPath.isEmpty {
-           _sdkPlatformFrameworkPath = AbsolutePath(platformPath).appending(
+            // For XCTest framework.
+            let fwk = AbsolutePath(platformPath).appending(
                 components: "Developer", "Library", "Frameworks")
+
+            // For XCTest Swift library.
+            let lib = AbsolutePath(platformPath).appending(
+                components: "Developer", "usr", "lib")
+
+            _sdkPlatformFrameworkPath = (fwk, lib)
         }
         return _sdkPlatformFrameworkPath
     }
     /// Cache storage for sdk platform path.
-    private static var _sdkPlatformFrameworkPath: AbsolutePath? = nil
+    private static var _sdkPlatformFrameworkPath: (fwk: AbsolutePath, lib: AbsolutePath)? = nil
 
     /// Target triple for the host system.
     private static let hostTargetTriple = Triple.hostTriple