formatting

Author: John Bute

Sources/PackageLoading/ManifestLoader+Validation.swift

@@ -310,7 +310,9 @@ extension Basics.Diagnostic {
     }
 
     static func invalidDependencyOnTestTarget(dependency: String, targetName: String) -> Self {
-        return .error("Invalid dependency: '\(targetName)' cannot depend on test target dependency '\(dependency)'. Only test targets can depend on other test targets")
+        .error(
+            "Invalid dependency: '\(targetName)' cannot depend on test target dependency '\(dependency)'. Only test targets can depend on other test targets"
+        )
     }
 
     static func invalidBinaryLocation(targetName: String) -> Self {

Sources/PackageLoading/PackageBuilder.swift

@@ -948,17 +948,20 @@ public final class PackageBuilder {
                         ))
                 }
         }
-        
+
         // Ensure non-test targets do not depend on test targets.
         // Only test targets are allowed to have dependencies on other test targets.
         if !potentialModule.isTest {
             for dependency in dependencies {
                 if let depTarget = dependency.module, depTarget.type == .test {
-                    self.observabilityScope.emit(.invalidDependencyOnTestTarget(dependency: dependency.name, targetName: potentialModule.name))
+                    self.observabilityScope.emit(.invalidDependencyOnTestTarget(
+                        dependency: dependency.name,
+                        targetName: potentialModule.name
+                    ))
                 }
             }
         }
-        
+
         // Create the build setting assignment table for this target.
         let buildSettings = try self.buildSettings(
             for: manifestTarget,

Tests/PackageGraphTests/ModulesGraphTests.swift

@@ -409,17 +409,16 @@ final class ModulesGraphTests: XCTestCase {
             result.check(modules: "Bar", "Baz", "Foo")
         }
     }
-    
+
     func testLibraryInvalidDependencyOnTestTarget() throws {
-        
         let fs = InMemoryFileSystem(
             emptyFiles:
             "/Foo/Sources/Foo/Foo.swift",
             "/Foo/Tests/FooTest/FooTest.swift"
         )
-        
+
         let observability = ObservabilitySystem.makeForTesting()
-        
+
         let _ = try loadModulesGraph(
             fileSystem: fs,
             manifests: [
@@ -432,30 +431,30 @@ final class ModulesGraphTests: XCTestCase {
                     ],
                     targets: [
                         TargetDescription(name: "Foo", dependencies: ["FooTest"]),
-                        TargetDescription(name: "FooTest", type: .test)
+                        TargetDescription(name: "FooTest", type: .test),
                     ]
                 ),
             ],
-            observabilityScope: observability.topScope)
-        
+            observabilityScope: observability.topScope
+        )
+
         testDiagnostics(observability.diagnostics) { result in
             result.check(
                 diagnostic: "Invalid dependency: 'Foo' cannot depend on test target dependency 'FooTest'. Only test targets can depend on other test targets",
                 severity: .error
             )
         }
     }
-    
+
     func testValidDependencyOnTestTarget() throws {
-        
         let fs = InMemoryFileSystem(
             emptyFiles:
             "/Foo/Tests/Foo/Foo.swift",
             "/Foo/Tests/FooTest/FooTest.swift"
         )
-        
+
         let observability = ObservabilitySystem.makeForTesting()
-        
+
         let _ = try loadModulesGraph(
             fileSystem: fs,
             manifests: [
@@ -467,15 +466,16 @@ final class ModulesGraphTests: XCTestCase {
                     ],
                     targets: [
                         TargetDescription(name: "Foo", dependencies: ["FooTest"], type: .test),
-                        TargetDescription(name: "FooTest", type: .test)
+                        TargetDescription(name: "FooTest", type: .test),
                     ]
                 ),
             ],
-            observabilityScope: observability.topScope)
-        
+            observabilityScope: observability.topScope
+        )
+
         XCTAssertNoDiagnostics(observability.diagnostics)
     }
-    
+
     // Make sure there is no error when we reference Test targets in a package and then
     // use it as a dependency to another package. SR-2353
     func testTestTargetDeclInExternalPackage() throws {