DiskProvider.MoveFile now overwrites existing file.

NzbDrone.Common.Test/DiskProviderTests.cs

@@ -34,6 +34,20 @@ namespace NzbDrone.Common.Test
             }
         }
 
+        [Test]
+        public void moveFile_should_overwrite_existing_file()
+        {
+            var diskProvider = new DiskProvider();
+            diskProvider.CopyDirectory(BinFolder.FullName, BinFolderCopy.FullName);
+
+            var targetPath = Path.Combine(BinFolderCopy.FullName, "file.move");
+
+            diskProvider.MoveFile(BinFolderCopy.GetFiles("*.dll", SearchOption.AllDirectories).First().FullName, targetPath);
+            diskProvider.MoveFile(BinFolderCopy.GetFiles("*.dll", SearchOption.AllDirectories).First().FullName, targetPath);
+
+            File.Exists(targetPath).Should().BeTrue();
+        }
+
         [Test]
         public void CopyFolder_should_copy_folder()
         {
@@ -45,6 +59,7 @@ namespace NzbDrone.Common.Test
             VerifyCopy();
         }
 
+
         [Test]
         public void CopyFolder_should_overright_existing_folder()
         {

NzbDrone.Common/DiskProvider.cs

@@ -98,24 +98,20 @@ namespace NzbDrone.Common
                 TransferDirectory(subDir.FullName, Path.Combine(target, subDir.Name), transferAction);
             }
 
-            foreach (var file in sourceFolder.GetFiles("*.*", SearchOption.TopDirectoryOnly))
+            foreach (var sourceFile in sourceFolder.GetFiles("*.*", SearchOption.TopDirectoryOnly))
             {
-                var destFile = Path.Combine(target, file.Name);
+                var destFile = Path.Combine(target, sourceFile.Name);
 
                 switch (transferAction)
                 {
                     case TransferAction.Copy:
                         {
-                            file.CopyTo(destFile, true);
+                            sourceFile.CopyTo(destFile, true);
                             break;
                         }
                     case TransferAction.Move:
                         {
-                            if (FileExists(destFile))
+                            MoveFile(sourceFile.FullName, destFile);
-                            {
-                                File.Delete(destFile);
-                            }
-                            file.MoveTo(destFile);
                             break;
                         }
                 }
@@ -130,6 +126,11 @@ namespace NzbDrone.Common
 
         public virtual void MoveFile(string sourcePath, string destinationPath)
         {
+            if (FileExists(destinationPath))
+            {
+                DeleteFile(destinationPath);
+            }
+
             File.Move(sourcePath, destinationPath);
         }