fix: Path.GetTempPath should end with directory separator (#1172)

`Path.GetTempPath` should end with directory separator
TestableIO · Dec 3, 2024 · d439c7f · d439c7f
1 parent 3e4b4e2
commit d439c7f
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/TestableIO.System.IO.Abstractions.TestingHelpers/MockFileSystem.cs b/src/TestableIO.System.IO.Abstractions.TestingHelpers/MockFileSystem.cs
@@ -14,7 +14,7 @@ namespace System.IO.Abstractions.TestingHelpers
     public class MockFileSystem : FileSystemBase, IMockFileDataAccessor
     {
         private const string DEFAULT_CURRENT_DIRECTORY = @"C:\";
-        private const string TEMP_DIRECTORY = @"C:\temp";
+        private const string TEMP_DIRECTORY = @"C:\temp\";
 
         private readonly IDictionary<string, FileSystemEntry> files;
         private readonly IDictionary<string, MockDriveData> drives;

diff --git a/tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockPathTests.cs b/tests/TestableIO.System.IO.Abstractions.TestingHelpers.Tests/MockPathTests.cs
@@ -382,6 +382,20 @@ public void GetTempPath_Called_ReturnsStringLengthGreaterThanZero()
             Assert.That(result.Length > 0, Is.True);
         }
 
+        [Test]
+        public void GetTempPath_ShouldEndWithDirectorySeparator()
+        {
+            //Arrange
+            var mockPath = new MockFileSystem().Path;
+            var directorySeparator = mockPath.DirectorySeparatorChar.ToString();
+
+            //Act
+            var result = mockPath.GetTempPath();
+
+            //Assert
+            Assert.That(result, Does.EndWith(directorySeparator));
+        }
+
         [Test]
         [TestCase(null)]
         [TestCase("")]