Add tests for ManagedFileSystem.MoveDirectory (#14065)

1 files changed, 110 insertions, 74 deletions

diff --git a/tests/Jellyfin.Server.Implementations.Tests/IO/ManagedFileSystemTests.cs b/tests/Jellyfin.Server.Implementations.Tests/IO/ManagedFileSystemTests.cs
index 95a5b8179..6997b51ac 100644
--- a/tests/Jellyfin.Server.Implementations.Tests/IO/ManagedFileSystemTests.cs
+++ b/tests/Jellyfin.Server.Implementations.Tests/IO/ManagedFileSystemTests.cs
@@ -5,83 +5,119 @@ using System.Runtime.InteropServices;
 using AutoFixture;
 using AutoFixture.AutoMoq;
 using Emby.Server.Implementations.IO;
+using Jellyfin.Extensions;
 using Xunit;
 
-namespace Jellyfin.Server.Implementations.Tests.IO
+namespace Jellyfin.Server.Implementations.Tests.IO;
+
+public class ManagedFileSystemTests
 {
-    public class ManagedFileSystemTests
+    private readonly IFixture _fixture;
+    private readonly ManagedFileSystem _sut;
+
+    public ManagedFileSystemTests()
+    {
+        _fixture = new Fixture().Customize(new AutoMoqCustomization { ConfigureMembers = true });
+        _sut = _fixture.Create<ManagedFileSystem>();
+    }
+
+    [Fact]
+    public void MoveDirectory_SameFileSystem_Correct()
+        => MoveDirectoryInternal();
+
+    [SkippableFact]
+    public void MoveDirectory_DifferentFileSystem_Correct()
+    {
+        const string DestinationParent = "/dev/shm";
+
+        Skip.IfNot(Directory.Exists(DestinationParent));
+
+        MoveDirectoryInternal(DestinationParent);
+    }
+
+    internal void MoveDirectoryInternal(string? destinationParent = null)
     {
-        private readonly IFixture _fixture;
-        private readonly ManagedFileSystem _sut;
-
-        public ManagedFileSystemTests()
-        {
-            _fixture = new Fixture().Customize(new AutoMoqCustomization { ConfigureMembers = true });
-            _sut = _fixture.Create<ManagedFileSystem>();
-        }
-
-        [SkippableTheory]
-        [InlineData("/Volumes/Library/Sample/Music/Playlists/", "../Beethoven/Misc/Moonlight Sonata.mp3", "/Volumes/Library/Sample/Music/Beethoven/Misc/Moonlight Sonata.mp3")]
-        [InlineData("/Volumes/Library/Sample/Music/Playlists/", "../../Beethoven/Misc/Moonlight Sonata.mp3", "/Volumes/Library/Sample/Beethoven/Misc/Moonlight Sonata.mp3")]
-        [InlineData("/Volumes/Library/Sample/Music/Playlists/", "Beethoven/Misc/Moonlight Sonata.mp3", "/Volumes/Library/Sample/Music/Playlists/Beethoven/Misc/Moonlight Sonata.mp3")]
-        [InlineData("/Volumes/Library/Sample/Music/Playlists/", "/mnt/Beethoven/Misc/Moonlight Sonata.mp3", "/mnt/Beethoven/Misc/Moonlight Sonata.mp3")]
-        public void MakeAbsolutePathCorrectlyHandlesRelativeFilePathsOnUnixLike(
-            string folderPath,
-            string filePath,
-            string expectedAbsolutePath)
-        {
-            Skip.If(OperatingSystem.IsWindows());
-
-            var generatedPath = _sut.MakeAbsolutePath(folderPath, filePath);
-            Assert.Equal(expectedAbsolutePath, generatedPath);
-        }
-
-        [SkippableTheory]
-        [InlineData(@"C:\\Volumes\Library\Sample\Music\Playlists\", @"..\Beethoven\Misc\Moonlight Sonata.mp3", @"C:\Volumes\Library\Sample\Music\Beethoven\Misc\Moonlight Sonata.mp3")]
-        [InlineData(@"C:\\Volumes\Library\Sample\Music\Playlists\", @"..\..\Beethoven\Misc\Moonlight Sonata.mp3", @"C:\Volumes\Library\Sample\Beethoven\Misc\Moonlight Sonata.mp3")]
-        [InlineData(@"C:\\Volumes\Library\Sample\Music\Playlists\", @"Beethoven\Misc\Moonlight Sonata.mp3", @"C:\Volumes\Library\Sample\Music\Playlists\Beethoven\Misc\Moonlight Sonata.mp3")]
-        [InlineData(@"C:\\Volumes\Library\Sample\Music\Playlists\", @"D:\\Beethoven\Misc\Moonlight Sonata.mp3", @"D:\\Beethoven\Misc\Moonlight Sonata.mp3")]
-        public void MakeAbsolutePathCorrectlyHandlesRelativeFilePathsOnWindows(
-            string folderPath,
-            string filePath,
-            string expectedAbsolutePath)
-        {
-            Skip.If(!OperatingSystem.IsWindows());
-
-            var generatedPath = _sut.MakeAbsolutePath(folderPath, filePath);
-
-            Assert.Equal(expectedAbsolutePath, generatedPath);
-        }
-
-        [Theory]
-        [InlineData("ValidFileName", "ValidFileName")]
-        [InlineData("AC/DC", "AC DC")]
-        [InlineData("Invalid\0", "Invalid ")]
-        [InlineData("AC/DC\0KD/A", "AC DC KD A")]
-        public void GetValidFilename_ReturnsValidFilename(string filename, string expectedFileName)
-        {
-            Assert.Equal(expectedFileName, _sut.GetValidFilename(filename));
-        }
-
-        [SkippableFact]
-        public void GetFileInfo_DanglingSymlink_ExistsFalse()
-        {
-            Skip.If(OperatingSystem.IsWindows());
-
-            string testFileDir = Path.Combine(Path.GetTempPath(), "jellyfin-test-data");
-            string testFileName = Path.Combine(testFileDir, Path.GetRandomFileName() + "-danglingsym.link");
-
-            Directory.CreateDirectory(testFileDir);
-            Assert.Equal(0, symlink("thispathdoesntexist", testFileName));
-            Assert.True(File.Exists(testFileName));
-
-            var metadata = _sut.GetFileInfo(testFileName);
-            Assert.False(metadata.Exists);
-        }
-
-        [SuppressMessage("Naming Rules", "SA1300:ElementMustBeginWithUpperCaseLetter", Justification = "Have to")]
-        [DllImport("libc", SetLastError = true, CharSet = CharSet.Ansi)]
-        [DefaultDllImportSearchPaths(DllImportSearchPath.UserDirectories)]
-        private static extern int symlink(string target, string linkpath);
+        const string TempFile0 = "tempfile0";
+        const string TempFile1 = "tempfile1";
+
+        destinationParent ??= Path.GetTempPath();
+
+        var sourceDir = Directory.CreateTempSubdirectory();
+        var destinationDir = Path.Join(destinationParent, Path.GetRandomFileName());
+        FileHelper.CreateEmpty(Path.Join(sourceDir.FullName, TempFile0));
+        FileHelper.CreateEmpty(Path.Join(sourceDir.FullName, TempFile1));
+
+        _sut.MoveDirectory(sourceDir.FullName, destinationDir);
+
+        Assert.True(Directory.Exists(destinationDir));
+        Assert.True(File.Exists(Path.Join(destinationDir, TempFile0)));
+        Assert.True(File.Exists(Path.Join(destinationDir, TempFile1)));
+        Assert.False(Directory.Exists(sourceDir.FullName));
+
+        Directory.Delete(destinationDir, true);
     }
+
+    [SkippableTheory]
+    [InlineData("/Volumes/Library/Sample/Music/Playlists/", "../Beethoven/Misc/Moonlight Sonata.mp3", "/Volumes/Library/Sample/Music/Beethoven/Misc/Moonlight Sonata.mp3")]
+    [InlineData("/Volumes/Library/Sample/Music/Playlists/", "../../Beethoven/Misc/Moonlight Sonata.mp3", "/Volumes/Library/Sample/Beethoven/Misc/Moonlight Sonata.mp3")]
+    [InlineData("/Volumes/Library/Sample/Music/Playlists/", "Beethoven/Misc/Moonlight Sonata.mp3", "/Volumes/Library/Sample/Music/Playlists/Beethoven/Misc/Moonlight Sonata.mp3")]
+    [InlineData("/Volumes/Library/Sample/Music/Playlists/", "/mnt/Beethoven/Misc/Moonlight Sonata.mp3", "/mnt/Beethoven/Misc/Moonlight Sonata.mp3")]
+    public void MakeAbsolutePathCorrectlyHandlesRelativeFilePathsOnUnixLike(
+        string folderPath,
+        string filePath,
+        string expectedAbsolutePath)
+    {
+        Skip.If(OperatingSystem.IsWindows());
+
+        var generatedPath = _sut.MakeAbsolutePath(folderPath, filePath);
+        Assert.Equal(expectedAbsolutePath, generatedPath);
+    }
+
+    [SkippableTheory]
+    [InlineData(@"C:\\Volumes\Library\Sample\Music\Playlists\", @"..\Beethoven\Misc\Moonlight Sonata.mp3", @"C:\Volumes\Library\Sample\Music\Beethoven\Misc\Moonlight Sonata.mp3")]
+    [InlineData(@"C:\\Volumes\Library\Sample\Music\Playlists\", @"..\..\Beethoven\Misc\Moonlight Sonata.mp3", @"C:\Volumes\Library\Sample\Beethoven\Misc\Moonlight Sonata.mp3")]
+    [InlineData(@"C:\\Volumes\Library\Sample\Music\Playlists\", @"Beethoven\Misc\Moonlight Sonata.mp3", @"C:\Volumes\Library\Sample\Music\Playlists\Beethoven\Misc\Moonlight Sonata.mp3")]
+    [InlineData(@"C:\\Volumes\Library\Sample\Music\Playlists\", @"D:\\Beethoven\Misc\Moonlight Sonata.mp3", @"D:\\Beethoven\Misc\Moonlight Sonata.mp3")]
+    public void MakeAbsolutePathCorrectlyHandlesRelativeFilePathsOnWindows(
+        string folderPath,
+        string filePath,
+        string expectedAbsolutePath)
+    {
+        Skip.IfNot(OperatingSystem.IsWindows());
+
+        var generatedPath = _sut.MakeAbsolutePath(folderPath, filePath);
+
+        Assert.Equal(expectedAbsolutePath, generatedPath);
+    }
+
+    [Theory]
+    [InlineData("ValidFileName", "ValidFileName")]
+    [InlineData("AC/DC", "AC DC")]
+    [InlineData("Invalid\0", "Invalid ")]
+    [InlineData("AC/DC\0KD/A", "AC DC KD A")]
+    public void GetValidFilename_ReturnsValidFilename(string filename, string expectedFileName)
+    {
+        Assert.Equal(expectedFileName, _sut.GetValidFilename(filename));
+    }
+
+    [SkippableFact]
+    public void GetFileInfo_DanglingSymlink_ExistsFalse()
+    {
+        Skip.If(OperatingSystem.IsWindows());
+
+        string testFileDir = Path.Combine(Path.GetTempPath(), "jellyfin-test-data");
+        string testFileName = Path.Combine(testFileDir, Path.GetRandomFileName() + "-danglingsym.link");
+
+        Directory.CreateDirectory(testFileDir);
+        Assert.Equal(0, symlink("thispathdoesntexist", testFileName));
+        Assert.True(File.Exists(testFileName));
+
+        var metadata = _sut.GetFileInfo(testFileName);
+        Assert.False(metadata.Exists);
+    }
+
+    [SuppressMessage("Naming Rules", "SA1300:ElementMustBeginWithUpperCaseLetter", Justification = "Have to")]
+    [DllImport("libc", SetLastError = true, CharSet = CharSet.Ansi)]
+    [DefaultDllImportSearchPaths(DllImportSearchPath.UserDirectories)]
+    private static extern int symlink(string target, string linkpath);
 }