Adjust to SE-0303: Add path, exclude, and sources parameters to plugin targets.

Author: abertelrud

Sources/PackageDescription/Target.swift

@@ -274,8 +274,6 @@ public final class Target {
         case .plugin:
             precondition(
                 url == nil &&
-                exclude.isEmpty &&
-                sources == nil &&
                 resources == nil &&
                 publicHeadersPath == nil &&
                 pkgConfig == nil &&
@@ -908,14 +906,17 @@ public final class Target {
     public static func plugin(
         name: String,
         capability: PluginCapability,
-        dependencies: [Dependency] = []
+        dependencies: [Dependency] = [],
+        path: String? = nil,
+        exclude: [String] = [],
+        sources: [String]? = nil
     ) -> Target {
       return Target(
           name: name,
           dependencies: dependencies,
-          path: nil,
-          exclude: [],
-          sources: nil,
+          path: path,
+          exclude: exclude,
+          sources: sources,
           publicHeadersPath: nil,
           type: .plugin,
           pluginCapability: capability)

Sources/PackageModel/Manifest/TargetDescription.swift

@@ -174,8 +174,6 @@ public struct TargetDescription: Equatable, Codable {
             if pluginUsages != nil { throw Error.disallowedPropertyInTarget(targetName: name, propertyName: "pluginUsages") }
         case .plugin:
             if url != nil { throw Error.disallowedPropertyInTarget(targetName: name, propertyName: "url") }
-            if !exclude.isEmpty { throw Error.disallowedPropertyInTarget(targetName: name, propertyName: "exclude") }
-            if sources != nil { throw Error.disallowedPropertyInTarget(targetName: name, propertyName: "sources") }
             if !resources.isEmpty { throw Error.disallowedPropertyInTarget(targetName: name, propertyName: "resources") }
             if publicHeadersPath != nil { throw Error.disallowedPropertyInTarget(targetName: name, propertyName: "publicHeadersPath") }
             if pkgConfig != nil { throw Error.disallowedPropertyInTarget(targetName: name, propertyName: "pkgConfig") }

Tests/PackageLoadingTests/PD5_5LoadingTests.swift

@@ -60,4 +60,31 @@ class PackageDescription5_5LoadingTests: PackageDescriptionLoadingTests {
             XCTAssertEqual(manifest.targets[0].pluginCapability, .buildTool)
         }
     }
+
+    func testPluginPluginTargetCustomization() throws {
+        let stream = BufferedOutputByteStream()
+        stream <<< """
+            import PackageDescription
+            let package = Package(
+               name: "Foo",
+               targets: [
+                   .plugin(
+                       name: "Foo",
+                       capability: .buildTool(),
+                       path: "Sources/Foo",
+                       exclude: ["IAmOut.swift"],
+                       sources: ["CountMeIn.swift"]
+                    )
+               ]
+            )
+            """
+
+        loadManifest(stream.bytes) { manifest in
+            XCTAssertEqual(manifest.targets[0].type, .plugin)
+            XCTAssertEqual(manifest.targets[0].pluginCapability, .buildTool)
+            XCTAssertEqual(manifest.targets[0].path, "Sources/Foo")
+            XCTAssertEqual(manifest.targets[0].exclude, ["IAmOut.swift"])
+            XCTAssertEqual(manifest.targets[0].sources, ["CountMeIn.swift"])
+        }
+    }
 }