Add separate "add test" manifest actions for XCTest and Swift Testing

Author: DougGregor

Sources/SourceKitLSP/Swift/CodeActions/PackageManifestEdits.swift

@@ -55,21 +55,36 @@ struct PackageManifestEdits: SyntaxCodeActionProvider {
     }
 
     do {
-      // Describe the target we are going to create.
-      let target = try TargetDescription(
-        name: "\(targetName)Tests",
-        dependencies: [.byName(name: targetName, condition: nil)],
-        type: .test
-      )
+      var actions: [CodeAction] = []
 
-      let edits = try AddTarget.addTarget(target, to: scope.file)
-      return [
-        CodeAction(
-          title: "Add test target",
-          kind: .refactor,
-          edit: edits.asWorkspaceEdit(snapshot: scope.snapshot)
-        )
+      let variants: [(AddTarget.TestHarness, String)] = [
+        (.swiftTesting, "Swift Testing"),
+        (.xctest, "XCTest")
       ]
+      for (testingLibrary, libraryName) in variants {
+        // Describe the target we are going to create.
+        let target = try TargetDescription(
+          name: "\(targetName)Tests",
+          dependencies: [.byName(name: targetName, condition: nil)],
+          type: .test
+        )
+
+        let edits = try AddTarget.addTarget(
+          target,
+          to: scope.file,
+          configuration: .init(testHarness: testingLibrary)
+        )
+
+        actions.append(
+          CodeAction(
+            title: "Add test target (\(libraryName))",
+            kind: .refactor,
+            edit: edits.asWorkspaceEdit(snapshot: scope.snapshot)
+          )
+        )
+      }
+
+      return actions
     } catch {
       return []
     }

Tests/SourceKitLSPTests/CodeActionTests.swift

@@ -522,7 +522,13 @@ final class CodeActionTests: XCTestCase {
     // Make sure we get the expected package manifest editing actions.
     XCTAssertTrue(
       codeActions.contains { action in
-        return action.title == "Add test target"
+        return action.title == "Add test target (Swift Testing)"
+      }
+    )
+
+    XCTAssertTrue(
+      codeActions.contains { action in
+        return action.title == "Add test target (XCTest)"
       }
     )