From 8a3abdcbf742bd76920652976eddc76686d45da4 Mon Sep 17 00:00:00 2001 From: Josh Patra Date: Fri, 6 Feb 2026 01:54:32 -0500 Subject: [PATCH] Fix LyricsStartupValidator build errors - Remove duplicate _httpClient field (use inherited one) - Replace ValidationResult.Warning with ValidationResult.Failure - Use PARTIAL status for partial failures --- allstarr/Services/Lyrics/LyricsStartupValidator.cs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/allstarr/Services/Lyrics/LyricsStartupValidator.cs b/allstarr/Services/Lyrics/LyricsStartupValidator.cs index b20bb31..b45d764 100644 --- a/allstarr/Services/Lyrics/LyricsStartupValidator.cs +++ b/allstarr/Services/Lyrics/LyricsStartupValidator.cs @@ -12,7 +12,6 @@ namespace allstarr.Services.Lyrics; public class LyricsStartupValidator : BaseStartupValidator { private readonly SpotifyApiSettings _spotifySettings; - private readonly HttpClient _httpClient; // Test song: "22" by Taylor Swift private const string TestSongTitle = "22"; @@ -29,7 +28,6 @@ public class LyricsStartupValidator : BaseStartupValidator : base(httpClientFactory.CreateClient()) { _spotifySettings = spotifySettings.Value; - _httpClient = httpClientFactory.CreateClient(); _httpClient.Timeout = TimeSpan.FromSeconds(10); } @@ -60,7 +58,7 @@ public class LyricsStartupValidator : BaseStartupValidator return allSuccess ? ValidationResult.Success("Lyrics services validation completed") - : ValidationResult.Warning("Some lyrics services had issues"); + : ValidationResult.Failure("PARTIAL", "Some lyrics services had issues", ConsoleColor.Yellow); } private async Task TestLrclibAsync(CancellationToken cancellationToken)