test: improve coverage (#843)

Author: ReenigneArcher

.readthedocs.yaml

@@ -12,6 +12,7 @@ build:
   tools:
     python: "3.14"
   apt_packages:
+    - doxygen  # required to generate source API docs
     - graphviz  # required to build diagrams
   jobs:
     post_build:

Jellyfin.Plugin.Themerr.Tests/TestThemerrController.cs

@@ -2,6 +2,7 @@
 using System.Collections;
 using System.Collections.Generic;
 using System.IO;
+using Jellyfin.Plugin.Themerr;
 using Jellyfin.Plugin.Themerr.Api;
 using Jellyfin.Plugin.Themerr.Storage;
 using MediaBrowser.Common.Configuration;
@@ -68,6 +69,18 @@ public void TestGetProgress()
         Assert.Empty(items);
     }
 
+    /// <summary>
+    /// Test TriggerUpdateRequest starts and completes an update.
+    /// </summary>
+    [Fact]
+    [Trait("Category", "Unit")]
+    public async Task TestTriggerUpdateRequest()
+    {
+        await _controller.TriggerUpdateRequest();
+
+        Assert.NotNull(_controller);
+    }
+
     /// <summary>
     /// Test GetProgress from API with items pre-populated in the database.
     /// </summary>
@@ -135,4 +148,71 @@ public async Task TestReplaceThemeNotFound()
         var result = await _controller.ReplaceTheme(Guid.NewGuid());
         Assert.IsType<NotFoundResult>(result);
     }
+
+    /// <summary>
+    /// Test ReplaceTheme returns 204 when the item theme is replaced.
+    /// </summary>
+    [Fact]
+    [Trait("Category", "Unit")]
+    public async Task TestReplaceThemeNoContent()
+    {
+        var tempPath = Path.Combine(Path.GetTempPath(), "ThemerrJellyfinTests", Guid.NewGuid().ToString("N"));
+        Directory.CreateDirectory(tempPath);
+        var movie = new Movie
+        {
+            Name = "Controller Replace",
+            Path = Path.Combine(tempPath, "Controller Replace (1970).mp4"),
+            ProductionYear = 1970,
+            ProviderIds = new Dictionary<string, string> { { MetadataProvider.Tmdb.ToString(), "controller-replace" } },
+        };
+        var themePath = ThemerrManager.GetThemePath(movie);
+        var audioStubPath = Path.Combine(Directory.GetCurrentDirectory(), "data", "audio_stub.mp3");
+        var repository = new ThemerrRepository(
+            Path.Combine(tempPath, "themerr.db"),
+            new Mock<ILogger>().Object);
+        repository.Save(movie, new ThemerrMediaItemSaveOptions
+        {
+            ThemePath = themePath,
+            ThemeProvider = ThemerrThemeProvider.User,
+            InThemerrDb = true,
+            InThemerrDbCheckedUtc = DateTime.UtcNow,
+            YoutubeThemeUrl = "https://www.youtube.com/watch?v=controller-replace",
+        });
+
+        var mockApplicationPaths = TestHelper.GetMockApplicationPaths(tempPath);
+        var mockLibraryManager = new Mock<ILibraryManager>();
+        mockLibraryManager
+            .Setup(x => x.GetItemById(movie.Id))
+            .Returns(movie);
+        var mockYoutubeClient = new Mock<IYoutubeClientWrapper>();
+        mockYoutubeClient
+            .Setup(x => x.DownloadAudioAsync(It.IsAny<string>(), It.IsAny<string>()))
+            .Returns<string, string>((_, destination) =>
+            {
+                File.Copy(audioStubPath, destination, true);
+                return Task.CompletedTask;
+            });
+        var manager = new ThemerrManager(
+            mockApplicationPaths.Object,
+            mockLibraryManager.Object,
+            new Mock<ILogger<ThemerrManager>>().Object,
+            mockYoutubeClient.Object,
+            repository);
+        var controller = new ThemerrController(manager, new Mock<ILogger<ThemerrController>>().Object);
+
+        try
+        {
+            var result = await controller.ReplaceTheme(movie.Id);
+
+            Assert.IsType<NoContentResult>(result);
+            Assert.True(File.Exists(themePath));
+        }
+        finally
+        {
+            if (File.Exists(themePath))
+            {
+                File.Delete(themePath);
+            }
+        }
+    }
 }