Analysing target resources

Author: goergisn

ReferencePackages/UpdatedPackage/Package.swift

@@ -17,6 +17,7 @@ let package = Package(
         // Targets are the basic building blocks of a package, defining a module or a test suite.
         // Targets can depend on other targets in this package and products from dependencies.
         .target(
-            name: "ReferencePackage")
+            name: "ReferencePackage"
+        )
     ]
 )

Sources/PublicModules/PADPackageFileAnalyzer/SwiftPackageFileAnalyzer.swift

@@ -328,15 +328,22 @@ private extension SwiftPackageFileAnalyzer {
     ) throws -> [Change] {
         guard oldTarget != newTarget else { return [] }
 
+        // MARK: Target Resources
+        
+        let oldResourcePaths = Set(oldTarget.resources?.map(\.path) ?? [])
+        let newResourcePaths = Set(newTarget.resources?.map(\.path) ?? [])
+        
+        let addedResourcePaths = newResourcePaths.subtracting(oldResourcePaths)
+        let consistentResourcePaths = oldResourcePaths.intersection(newResourcePaths)
+        let removedResourcePaths = oldResourcePaths.subtracting(newResourcePaths)
+        
         // MARK: Target Dependencies
 
         let oldTargetDependencies = Set(oldTarget.targetDependencies ?? [])
         let newTargetDependencies = Set(newTarget.targetDependencies ?? [])
 
         let addedTargetDependencies = newTargetDependencies.subtracting(oldTargetDependencies)
         let removedTargetDependencies = oldTargetDependencies.subtracting(newTargetDependencies)
-
-        // TODO: Add analyzing of SwiftPackageDescription.Target.Resource
 
         // MARK: Product Dependencies
 
@@ -346,7 +353,29 @@ private extension SwiftPackageFileAnalyzer {
         let addedProductDependencies = newProductDependencies.subtracting(oldProductDependencies)
         let removedProductDependencies = oldProductDependencies.subtracting(newProductDependencies)
 
+        // MARK: Compiling list of changes
+        
         var listOfChanges = [String]()
+        
+        listOfChanges += addedResourcePaths.compactMap { path in
+            guard let resource = newTarget.resources?.first(where: { $0.path == path }) else { return nil }
+            return "Added resource \(resource.description)"
+        }
+        
+        listOfChanges += consistentResourcePaths.compactMap { path in
+            guard
+                let newResource = newTarget.resources?.first(where: { $0.path == path }),
+                let oldResource = oldTarget.resources?.first(where: { $0.path == path })
+            else { return nil }
+            
+            return "Changed resource from `\(oldResource.description)` to `\(newResource.description)`"
+        }
+        
+        listOfChanges += removedResourcePaths.compactMap { path in
+            guard let resource = oldTarget.resources?.first(where: { $0.path == path }) else { return nil }
+            return "Removed resource \(resource.description)"
+        }
+        
         listOfChanges += addedTargetDependencies.map { "Added dependency .target(name: \"\($0)\")" }
         listOfChanges += addedProductDependencies.map { "Added dependency .product(name: \"\($0)\", ...)" }
 

Sources/Shared/Package/SwiftPackageFileHelperModule/SwiftPackageDescription.swift

@@ -179,7 +179,7 @@ package extension SwiftPackageDescription {
         package let productDependencies: [String]?
         /// `.target(name: ...) dependency
         package let targetDependencies: [String]?
-        // TODO: check this in the SwiftPackageFileAnalyzer
+        /// The resources used by the Target
         package let resources: [Resource]?
 
         // Ignoring following properties for now as they are not handled in the `PackageAnalyzer`
@@ -266,6 +266,23 @@ package extension SwiftPackageDescription.Target {
     }
 }
 
+extension SwiftPackageDescription.Target.Resource: CustomStringConvertible {
+
+    package var description: String {
+        return switch rule {
+        case .copy: ".copy(\"\(path)\")"
+        case .embeddInCode: ".embeddInCode(\"\(path)\")"
+        case let .process(metadata):
+            if let localization = metadata["localization"] {
+                ".process(\"\(path)\", localization: \"\(localization)\")"
+            } else {
+                ".process(\"\(path)\")"
+            }
+        }
+    }
+}
+
+
 package extension SwiftPackageDescription.Target.Resource {
 
     enum Rule: Codable, Equatable {
@@ -319,19 +336,3 @@ package extension SwiftPackageDescription.Target.Resource {
         }
     }
 }
-
-extension SwiftPackageDescription.Target.Resource.Rule: CustomStringConvertible {
-
-    package var description: String {
-        return switch self {
-        case .copy: "copy"
-        case .embeddInCode: "embeddInCode"
-        case let .process(metadata):
-            if let localization = metadata["localization"] {
-                "process (localization: \(localization))"
-            } else {
-                "process"
-            }
-        }
-    }
-}

Sources/Shared/Public/PADCore/Change.swift

@@ -26,7 +26,7 @@ public struct Change: Equatable {
     ) {
         self.changeType = changeType
         self.parentPath = parentPath
-        self.listOfChanges = listOfChanges
+        self.listOfChanges = listOfChanges.sorted()
     }
 }
 

Tests/UnitTests/SwiftPackageFileAnalyzerTests.swift

@@ -84,7 +84,12 @@ class SwiftPackageFileAnalyzerTests: XCTestCase {
                             path: "some/new/path",
                             moduleType: .swiftTarget,
                             productDependencies: ["Some Product Dependency", "New Product Dependency"],
-                            targetDependencies: ["Some Target Dependency", "New Target Dependency"]
+                            targetDependencies: ["Some Target Dependency", "New Target Dependency"],
+                            resources: [
+                                .init(path: "copy-path", rule: .copy),
+                                .init(path: "process-path", rule: .process([:])),
+                                .init(path: "process-localization-path", rule: .process(["localization":"en_US"])),
+                            ]
                         )
                     ],
                     toolsVersion: "1.0"
@@ -106,7 +111,13 @@ class SwiftPackageFileAnalyzerTests: XCTestCase {
                             path: "some/old/path",
                             moduleType: .swiftTarget,
                             productDependencies: ["Some Product Dependency", "Old Product Dependency"],
-                            targetDependencies: ["Some Target Dependency", "Old Target Dependency"]
+                            targetDependencies: ["Some Target Dependency", "Old Target Dependency"],
+                            resources: [
+                                .init(path: "new-copy-path", rule: .copy),
+                                .init(path: "process-path", rule: .process(["localization":"en_US"])),
+                                .init(path: "process-localization-path", rule: .process([:])),
+                                .init(path: "embedd-in-code-path", rule: .embeddInCode),
+                            ]
                         )
                     ],
                     toolsVersion: "2.0"
@@ -204,13 +215,18 @@ class SwiftPackageFileAnalyzerTests: XCTestCase {
                 ),
                 parentPath: ".targets",
                 listOfChanges: [
+                    "Added resource .copy(\"copy-path\")",
+                    "Changed resource from `.process(\"process-path\", localization: \"en_US\")` to `.process(\"process-path\")`",
+                    "Changed resource from `.process(\"process-localization-path\")` to `.process(\"process-localization-path\", localization: \"en_US\")`",
+                    "Removed resource .copy(\"new-copy-path\")",
+                    "Removed resource .embeddInCode(\"embedd-in-code-path\")",
                     "Added dependency .target(name: \"New Target Dependency\")",
                     "Added dependency .product(name: \"New Product Dependency\", ...)",
                     "Changed path from \"some/old/path\" to \"some/new/path\"",
                     "Changed type from `.binaryTarget` to `.target`",
                     "Removed dependency .target(name: \"Old Target Dependency\")",
                     "Removed dependency .product(name: \"Old Product Dependency\", ...)"
-                ]
+                ].sorted()
             ),
             .init(
                 changeType: .removal(
@@ -220,7 +236,7 @@ class SwiftPackageFileAnalyzerTests: XCTestCase {
                 listOfChanges: []
             )
         ]
-
+        
         XCTAssertEqual(changes.changes, expectedChanges)
 
         waitForExpectations(timeout: 1)