Add functional tests for binary with moved resources

Author: ffried

Fixtures/Resources/Moved/.gitignore

@@ -0,0 +1,5 @@
+.DS_Store
+/.build
+/Packages
+/*.xcodeproj
+xcuserdata/

Fixtures/Resources/Moved/Package.swift

@@ -0,0 +1,21 @@
+// swift-tools-version:5.3
+import PackageDescription
+
+let package = Package(
+    name: "Resources",
+    targets: [
+        .target(
+            name: "SwiftyResource",
+            resources: [
+                .copy("foo.txt"),
+            ]
+        ),
+
+        .target(
+            name: "SeaResource",
+            resources: [
+                .copy("foo.txt"),
+            ]
+        ),
+    ]
+)

Fixtures/Resources/Moved/Sources/SeaResource/foo.txt

@@ -0,0 +1 @@
+foo

Fixtures/Resources/Moved/Sources/SeaResource/main.m

@@ -0,0 +1,12 @@
+#import <Foundation/Foundation.h>
+
+int main(int argc, const char * argv[]) {
+    @autoreleasepool {
+        NSBundle *bundle = SWIFTPM_MODULE_BUNDLE;
+        NSString *foo = [bundle pathForResource:@"foo" ofType:@"txt"];
+        NSData *data = [NSFileManager.defaultManager contentsAtPath:foo];
+        NSString *contents = [[NSString alloc] initWithData:data encoding:NSUTF8StringEncoding];
+        printf("%s", contents.UTF8String);
+    }
+    return 0;
+}

Fixtures/Resources/Moved/Sources/SwiftyResource/foo.txt

@@ -0,0 +1 @@
+foo

Fixtures/Resources/Moved/Sources/SwiftyResource/main.swift

@@ -0,0 +1,7 @@
+import Foundation
+
+let bundle = Bundle.module
+
+let foo = bundle.path(forResource: "foo", ofType: "txt")!
+let contents = FileManager.default.contents(atPath: foo)!
+print(String(data: contents, encoding: .utf8)!, terminator: "")

Tests/FunctionalTests/ResourcesTests.swift

@@ -45,4 +45,48 @@ class ResourcesTests: XCTestCase {
                 """)
         }
     }
+
+    func testMovedBinaryResources() {
+        fixture(name: "Resources/Moved") { prefix in
+            var executables = ["SwiftyResource"]
+
+            // Objective-C module requires macOS
+            #if os(macOS)
+            executables.append("SeaResource")
+            #endif
+
+            // Both executables are built in one go since
+            // adding "--target execName" will not produce a binary
+            _ = try executeSwiftBuild(prefix, configuration: .Release)
+
+            let binPath = try AbsolutePath(
+                executeSwiftBuild(prefix, configuration: .Release, extraArgs: ["--show-bin-path"]).stdout
+                    .trimmingCharacters(in: .whitespacesAndNewlines)
+            )
+
+            for execName in executables {
+                try withTemporaryDirectory(prefix: execName) { tmpDirPath in
+                    defer {
+                        // Unblock and remove the tmp dir on deinit.
+                        try? localFileSystem.chmod(.userWritable, path: tmpDirPath, options: [.recursive])
+                        try? localFileSystem.removeFileTree(tmpDirPath)
+                    }
+
+                    let destBinPath = tmpDirPath.appending(component: execName)
+                    // Copy the binary
+                    try systemQuietly("cp", "-R", "-H",
+                                      binPath.appending(component: execName).pathString,
+                                      destBinPath.pathString)
+                    // Copy the resources
+                    try Process.checkNonZeroExit(args: "find", binPath.pathString, "(", "-name", "*.bundle", "-or", "-name", "*.resources", ")")
+                        .split(whereSeparator: { $0.isNewline })
+                        .filter { !$0.isEmpty }
+                        .forEach { try systemQuietly("cp", "-R", "-H", String($0), tmpDirPath.pathString) }
+                    // Run the binary
+                    let output = try Process.checkNonZeroExit(args: destBinPath.pathString)
+                    XCTAssertTrue(output.contains("foo"))
+                }
+            }
+        }
+    }
 }