Only deleteItem if file exists at URL.

Implemented here partly to ease merge with proposed changes in other pull requests.
Antondomashnev · Apr 7, 2018 · 43c639d · 43c639d
1 parent 7424aee
commit 43c639d
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/FBSnapshotsViewer/Extensions/FileManager+Move.swift b/FBSnapshotsViewer/Extensions/FileManager+Move.swift
@@ -10,7 +10,13 @@ import Foundation
 
 extension FileManager {
     func moveItem(at fromURL: URL, to toURL: URL) throws {
-        try self.removeItem(at: toURL)
+        try deleteItem(at: toURL)
         try self.copyItem(at: fromURL, to: toURL)
     }
+
+    func deleteItem(at url: URL) throws {
+        if fileExists(atPath: url.path) {
+            try self.removeItem(at: url)
+        }
+    }
 }
diff --git a/FBSnapshotsViewer/Managers/SnapshotTestResultSwapper.swift b/FBSnapshotsViewer/Managers/SnapshotTestResultSwapper.swift
@@ -92,9 +92,9 @@ class SnapshotTestResultSwapper {
         let failedImageURL = URL(fileURLWithPath: failedImagePath, isDirectory: false)
 
         do {
-            try fileManager.removeItem(at: referenceImageURL)
-            try fileManager.removeItem(at: diffImageURL)
-            try fileManager.removeItem(at: failedImageURL)
+            try fileManager.deleteItem(at: referenceImageURL)
+            try fileManager.deleteItem(at: diffImageURL)
+            try fileManager.deleteItem(at: failedImageURL)
         }
         catch let error {
             throw SnapshotTestResultSwapperError.canNotPerformFileManagerOperation(testResult: testResult, underlyingError: error)