From fb246df01aa4c9953ed36f8df9958aaea13c507f Mon Sep 17 00:00:00 2001 From: Michael Housh Date: Tue, 17 Dec 2024 15:44:51 -0500 Subject: [PATCH] fix: Fixes failing tests since we create backups now during configuration generation. --- Package.swift | 2 +- .../ConfigurationClientTests.swift | 14 +------------- 2 files changed, 2 insertions(+), 14 deletions(-) diff --git a/Package.swift b/Package.swift index 7bb79ca..b78ab66 100644 --- a/Package.swift +++ b/Package.swift @@ -114,8 +114,8 @@ let package = Package( .target( name: "PlaybookClient", dependencies: [ - "CodersClient", "CommandClient", + "CodersClient", "ConfigurationClient", "FileClient", .product(name: "Dependencies", package: "swift-dependencies"), diff --git a/Tests/ConfigurationClientTests/ConfigurationClientTests.swift b/Tests/ConfigurationClientTests/ConfigurationClientTests.swift index a23488b..c998e0e 100644 --- a/Tests/ConfigurationClientTests/ConfigurationClientTests.swift +++ b/Tests/ConfigurationClientTests/ConfigurationClientTests.swift @@ -35,18 +35,6 @@ struct ConfigurationClientTests: TestCase { #expect(FileManager.default.fileExists(atPath: tempFile.cleanFilePath)) #expect(fileClient.fileExists(tempFile)) #expect(output == tempFile.cleanFilePath) - - // Ensure that we do not overwrite files if they exist. - do { - _ = try await configuration.generate(.init( - force: false, - json: fileName.hasSuffix("json"), - path: .file(File(tempFile)!) - )) - #expect(Bool(false)) - } catch { - #expect(Bool(true)) - } } } } @@ -152,7 +140,7 @@ struct ConfigurationClientTests: TestCase { try await withGeneratedConfigFile(named: "config.toml", client: client) { configFile in @Dependency(\.fileClient) var fileClient - let backupUrl = configFile.url.appendingPathExtension(".back") + let backupUrl = configFile.url.appendingPathExtension("back") #expect(fileClient.fileExists(backupUrl) == false) let config = Configuration()