fix: handle case-insensitive file overwrite on Windows

test: add tests for Windows file copy behavior

Author: ana1250

src/TestableIO.System.IO.Abstractions.TestingHelpers/MockFile.cs

@@ -48,7 +48,7 @@ public override void AppendAllBytes(string path, ReadOnlySpan<byte> bytes)
             AppendAllBytes(path, bytes.ToArray());
         }
 #endif
-        
+
     /// <inheritdoc />
     public override void AppendAllLines(string path, IEnumerable<string> contents)
     {
@@ -163,6 +163,11 @@ public override void Copy(string sourceFileName, string destFileName, bool overw
                 throw CommonExceptions.FileAlreadyExists(destFileName);
             }
 
+            if (string.Equals(sourceFileName, destFileName, StringComparison.OrdinalIgnoreCase) && XFS.IsWindowsPlatform())
+            {
+                throw CommonExceptions.ProcessCannotAccessFileInUse(destFileName);
+            }
+
             mockFileDataAccessor.RemoveFile(destFileName);
         }
 
@@ -522,10 +527,29 @@ public override void Move(string sourceFileName, string destFileName)
         mockFileDataAccessor.PathVerifier.IsLegalAbsoluteOrRelative(sourceFileName, nameof(sourceFileName));
         mockFileDataAccessor.PathVerifier.IsLegalAbsoluteOrRelative(destFileName, nameof(destFileName));
 
+        var sourceFile = mockFileDataAccessor.GetFile(sourceFileName);
+
+        if (sourceFile == null)
+        {
+            throw CommonExceptions.FileNotFound(sourceFileName);
+        }
+
+        if (!sourceFile.AllowedFileShare.HasFlag(FileShare.Delete))
+        {
+            throw CommonExceptions.ProcessCannotAccessFileInUse();
+        }
+
+        VerifyDirectoryExists(destFileName);
+
         if (mockFileDataAccessor.GetFile(destFileName) != null)
         {
             if (mockFileDataAccessor.StringOperations.Equals(destFileName, sourceFileName))
             {
+                if (XFS.IsWindowsPlatform())
+                {
+                    mockFileDataAccessor.RemoveFile(sourceFileName);
+                    mockFileDataAccessor.AddFile(destFileName, mockFileDataAccessor.AdjustTimes(new MockFileData(sourceFile), TimeAdjustments.LastAccessTime), false);
+                }
                 return;
             }
             else
@@ -534,16 +558,6 @@ public override void Move(string sourceFileName, string destFileName)
             }
         }
 
-        var sourceFile = mockFileDataAccessor.GetFile(sourceFileName);
-
-        if (sourceFile == null)
-        {
-            throw CommonExceptions.FileNotFound(sourceFileName);
-        }
-        if (!sourceFile.AllowedFileShare.HasFlag(FileShare.Delete))
-        {
-            throw CommonExceptions.ProcessCannotAccessFileInUse();
-        }
         VerifyDirectoryExists(destFileName);
 
         mockFileDataAccessor.RemoveFile(sourceFileName, false);
@@ -822,6 +836,11 @@ public override void Replace(string sourceFileName, string destinationFileName,
             throw CommonExceptions.FileNotFound(destinationFileName);
         }
 
+        if (mockFileDataAccessor.StringOperations.Equals(sourceFileName, destinationFileName) && XFS.IsWindowsPlatform())
+        {
+            throw CommonExceptions.ProcessCannotAccessFileInUse();
+        }
+
         if (destinationBackupFileName != null)
         {
             Copy(destinationFileName, destinationBackupFileName, overwrite: true);
@@ -1066,7 +1085,7 @@ public override void WriteAllBytes(string path, byte[] bytes)
 
         mockFileDataAccessor.AddFile(path, mockFileDataAccessor.AdjustTimes(new MockFileData(bytes.ToArray()), TimeAdjustments.All));
     }
-        
+
 #if FEATURE_FILE_SPAN
         /// <inheritdoc cref="IFile.WriteAllBytes(string,ReadOnlySpan{byte})"/>
         public override void WriteAllBytes(string path, ReadOnlySpan<byte> bytes)

tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockFileCopyTests.cs

@@ -246,6 +246,19 @@ public async Task MockFile_Copy_ShouldThrowNotSupportedExceptionWhenSourcePathCo
         await That(action).Throws<NotSupportedException>();
     }
 
+    [Test]
+    [WindowsOnly(WindowsSpecifics.Drives)]
+    public async Task Copy_Should_ThrowIOException_When_OverwritingWithSameNameDifferentCase()
+    {
+        var fileSystem = new MockFileSystem();
+        string path = @"C:\Temp\file.txt";
+        string pathUpper = @"C:\Temp\FILE.TXT";
+
+        fileSystem.File.WriteAllText(path, "Hello");
+
+        await That(() => fileSystem.File.Copy(path, pathUpper, true)).Throws<IOException>().HasMessage($"The process cannot access the file '{pathUpper}' because it is being used by another process.");
+    }
+
     [Test]
     [WindowsOnly(WindowsSpecifics.Drives)]
     public async Task MockFile_Copy_ShouldThrowNotSupportedExceptionWhenSourcePathContainsInvalidDriveLetter()

tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockFileMoveTests.cs

@@ -249,6 +249,26 @@ public async Task MockFile_Move_ShouldThrowNotSupportedExceptionWhenDestinationP
         await That(action).Throws<NotSupportedException>();
     }
 
+    [Test]
+    [WindowsOnly(WindowsSpecifics.Drives)]
+    public async Task MockFile_Move_CaseOnlyRename_ShouldChangeCase()
+    {
+        string sourceFilePath = @"c:\something\demo.txt";
+        string destFilePath = @"c:\something\DEMO.TXT";
+        string sourceFileContent = "content";
+
+        var fileSystem = new MockFileSystem(new Dictionary<string, MockFileData>
+        {
+        {
+                sourceFilePath, new MockFileData(sourceFileContent)}
+        });
+
+        fileSystem.File.Move(sourceFilePath, destFilePath);
+
+        await That(fileSystem.FileExists(destFilePath)).IsTrue();
+        await That(fileSystem.GetFile(destFilePath).TextContents).IsEqualTo(sourceFileContent);
+    }
+
     [Test]
     public async Task MockFile_Move_ShouldThrowArgumentExceptionWhenSourceIsEmpty_Message()
     {

tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockFileTests.cs

@@ -547,7 +547,7 @@ public async Task MockFile_AppendText_CreatesNewFileForAppendToNonExistingFile()
         await That(file.TextContents).IsEqualTo("New too!");
         await That(filesystem.FileExists(filepath)).IsTrue();
     }
-        
+
 #if !NET9_0_OR_GREATER
     [Test]
     public void Serializable_works()
@@ -567,7 +567,7 @@ public void Serializable_works()
         Assert.Pass();
     }
 #endif
-        
+
 #if !NET9_0_OR_GREATER
     [Test]
     public async Task Serializable_can_deserialize()
@@ -670,7 +670,7 @@ public async Task MockFile_Replace_ShouldCreateBackup()
         fileSystem.File.Replace(path1, path2, path3);
 
         await That(fileSystem.File.ReadAllText(path3)).IsEqualTo("2");
-    } 
+    }
 
     [Test]
     public async Task MockFile_Replace_ShouldThrowIfDirectoryOfBackupPathDoesNotExist()
@@ -730,4 +730,20 @@ public async Task MockFile_OpenRead_ShouldReturnReadOnlyStream()
         await That(stream.CanWrite).IsFalse();
         await That(() => stream.WriteByte(0)).Throws<NotSupportedException>();
     }
+
+    [Test]
+    [WindowsOnly(WindowsSpecifics.Drives)]
+    public async Task MockFile_Replace_SameSourceAndDestination_ShouldThrowIOException()
+    {
+        string sourceFilePath = @"c:\something\demo.txt";
+        string destFilePath = @"c:\something\Demo.txt";
+        string fileContent = "content";
+
+        var fileSystem = new MockFileSystem(new Dictionary<string, MockFileData>
+        {
+            { sourceFilePath, new MockFileData(fileContent) }
+        });
+
+        await That(() => fileSystem.File.Replace(sourceFilePath, destFilePath, null, true)).Throws<IOException>().HasMessage("The process cannot access the file because it is being used by another process.");
+    }
 }