Ensure testable variants of targets depend on the primary variant's resource bundle (#9486)

This is required in order to ensure that package resource accessors are
correctly generated for the testable variants

Closes #9459
Depends on swiftlang/swift-build#967

Author: owenv

Fixtures/Miscellaneous/TestableExeWithResources/Package.swift

@@ -0,0 +1,18 @@
+// swift-tools-version: 6.0
+import PackageDescription
+
+let package = Package(
+    name: "TestableExe",
+    targets: [
+        .executableTarget(
+            name: "TestableExe",
+            resources: [.embedInCode("foo.txt")]
+        ),
+        .testTarget(
+            name: "TestableExeTests",
+            dependencies: [
+                "TestableExe",
+            ]
+        ),
+    ]
+)

Fixtures/Miscellaneous/TestableExeWithResources/Sources/TestableExe/foo.txt

@@ -0,0 +1 @@
+bar

Fixtures/Miscellaneous/TestableExeWithResources/Sources/TestableExe/main.swift

@@ -0,0 +1,5 @@
+public func GetGreeting1() -> String {
+    return String(decoding: PackageResources.foo_txt, as: UTF8.self)
+}
+
+print("\(GetGreeting1())!")

Fixtures/Miscellaneous/TestableExeWithResources/Tests/TestableExeTests/TestableExeTests.swift

@@ -0,0 +1,8 @@
+import XCTest
+@testable import TestableExe
+
+final class TestableExeTests: XCTestCase {
+    func testExample() throws {
+        XCTAssertEqual(GetGreeting1(), "bar\n")
+    }
+}

Sources/SwiftBuildSupport/PackagePIFProjectBuilder+Modules.swift

@@ -317,6 +317,14 @@ extension PackagePIFProjectBuilder {
             // `inputResourceBundleName`.
             shouldGenerateBundleAccessor = true
             shouldGenerateEmbedInCodeAccessor = true
+            if resourceBundleName != nil {
+                let resourceTargetID = pifTargetIdForResourceBundle(sourceModule.name)
+                self.project[keyPath: sourceModuleTargetKeyPath].common.addDependency(
+                    on: resourceTargetID,
+                    platformFilters: [],
+                    linkProduct: false
+                )
+            }
         }
 
         // Find the PIF target for the resource bundle, if any. Otherwise fall back to the module.

Tests/CommandsTests/TestCommandTests.swift

@@ -1251,4 +1251,29 @@ struct TestCommandTests {
         }
     }
 
+    @Test(
+            .IssueWindowsLongPath,
+            .tags(
+                .Feature.TargetType.Executable,
+            ),
+            arguments: SupportedBuildSystemOnAllPlatforms, BuildConfiguration.allCases,
+        )
+        func testableExecutableWithEmbeddedResources(
+            buildSystem: BuildSystemProvider.Kind,
+            configuration: BuildConfiguration,
+        ) async throws {
+            try await withKnownIssue(isIntermittent: true) {
+                try await fixture(name: "Miscellaneous/TestableExeWithResources") { fixturePath in
+                    let result = try await execute(
+                        ["--vv"],
+                        packagePath: fixturePath,
+                        configuration: configuration,
+                        buildSystem: buildSystem,
+                    )
+                }
+            } when: {
+                .windows == ProcessInfo.hostOperatingSystem
+            }
+         }
+
 }