Move Xcode templates dependency to initializer (#592)

Author: tinder-cfuller

Sources/NodesXcodeTemplatesGenerator/XcodeTemplates.swift

@@ -6,27 +6,19 @@ import Foundation
 
 public final class XcodeTemplates {
 
-    private let templates: [XcodeTemplate]
+    private let config: Config
+    private let fileSystem: FileSystem
 
-    public init(config: Config) {
-        var templates: [XcodeTemplate] = UIFramework.Kind
-            .allCases
-            .compactMap { try? NodeXcodeTemplate(for: $0, config: config) }
-        if config.isViewInjectedTemplateEnabled {
-            templates.append(NodeViewInjectedXcodeTemplate(config: config))
-        }
-        templates += [
-            PluginListNodeXcodeTemplate(config: config),
-            PluginNodeXcodeTemplate(config: config),
-            PluginXcodeTemplate(config: config),
-            WorkerXcodeTemplate(config: config)
-        ]
-        self.templates = templates
+    public init(
+        config: Config,
+        fileSystem: FileSystem = FileManager.default
+    ) {
+        self.config = config
+        self.fileSystem = fileSystem
     }
 
     public func generate(
-        identifier: String,
-        using fileSystem: FileSystem = FileManager.default
+        identifier: String
     ) throws {
         let url: URL = fileSystem.libraryURL
             .appendingPathComponent("Developer")
@@ -35,13 +27,24 @@ public final class XcodeTemplates {
             .appendingPathComponent("File Templates")
             .appendingPathComponent("Nodes Architecture Framework (\(identifier))")
         try? fileSystem.removeItem(at: url)
-        try generate(at: url, using: fileSystem)
+        try generate(at: url)
     }
 
     public func generate(
-        at url: URL,
-        using fileSystem: FileSystem = FileManager.default
+        at url: URL
     ) throws {
+        var templates: [XcodeTemplate] = UIFramework.Kind
+            .allCases
+            .compactMap { try? NodeXcodeTemplate(for: $0, config: config) }
+        if config.isViewInjectedTemplateEnabled {
+            templates.append(NodeViewInjectedXcodeTemplate(config: config))
+        }
+        templates += [
+            PluginListNodeXcodeTemplate(config: config),
+            PluginNodeXcodeTemplate(config: config),
+            PluginXcodeTemplate(config: config),
+            WorkerXcodeTemplate(config: config)
+        ]
         let generator: XcodeTemplateGenerator = .init(fileSystem: fileSystem)
         try templates.forEach { try generator.generate(template: $0, into: url) }
     }

Tests/NodesXcodeTemplatesGeneratorTests/XcodeTemplatesTests.swift

@@ -9,7 +9,7 @@ final class XcodeTemplatesTests: XCTestCase {
 
     func testGenerateWithIdentifier() throws {
         let fileSystem: FileSystemMock = .init()
-        try XcodeTemplates(config: givenConfig()).generate(identifier: "identifier", using: fileSystem)
+        try XcodeTemplates(config: givenConfig(), fileSystem: fileSystem).generate(identifier: "identifier")
         // swiftlint:disable:next large_tuple
         let writes: [(contents: String, path: String, atomically: Bool)] = fileSystem.writes
         writes.forEach { assertSnapshot(of: $0.contents, as: .lines, named: "Contents.\(name(from: $0.path))") }
@@ -22,7 +22,7 @@ final class XcodeTemplatesTests: XCTestCase {
     func testGenerateWithURL() throws {
         let fileSystem: FileSystemMock = .init()
         let url: URL = .init(fileURLWithPath: "/")
-        try XcodeTemplates(config: givenConfig()).generate(at: url, using: fileSystem)
+        try XcodeTemplates(config: givenConfig(), fileSystem: fileSystem).generate(at: url)
         // swiftlint:disable:next large_tuple
         let writes: [(contents: String, path: String, atomically: Bool)] = fileSystem.writes
         writes.forEach { assertSnapshot(of: $0.contents, as: .lines, named: "Contents.\(name(from: $0.path))") }