joshpatra/allstarr
1
0
Fork 0
mirror of https://github.com/SoPat712/allstarr.git synced 2026-02-09 23:55:10 -05:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Fix nullable reference warnings

Browse Source
This commit is contained in:
Josh Patra
2026-01-30 11:36:59 -05:00
parent ba0fe35e72
commit e23e22a736
5 changed files with 13 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
allstarr.Tests/DeezerDownloadServiceTests.cs
View File

@@ -85,7 +85,7 @@ public class DeezerDownloadServiceTests : IDisposable
var serviceProviderMock = new Mock<IServiceProvider>(); var serviceProviderMock = new Mock<IServiceProvider>();
serviceProviderMock.Setup(sp => sp.GetService(typeof(allstarr.Services.Subsonic.PlaylistSyncService))) serviceProviderMock.Setup(sp => sp.GetService(typeof(allstarr.Services.Subsonic.PlaylistSyncService)))
.Returns(null); .Returns(null!);
return new DeezerDownloadService( return new DeezerDownloadService(
_httpClientFactoryMock.Object, _httpClientFactoryMock.Object,

4
allstarr.Tests/JellyfinProxyServiceTests.cs
View File

@@ -107,8 +107,8 @@ public class JellyfinProxyServiceTests
Assert.True(captured!.Headers.Contains("Authorization")); Assert.True(captured!.Headers.Contains("Authorization"));
var authHeader = captured.Headers.GetValues("Authorization").First(); var authHeader = captured.Headers.GetValues("Authorization").First();
Assert.Contains("MediaBrowser", authHeader); Assert.Contains("MediaBrowser", authHeader);
Assert.Contains(_settings.ApiKey, authHeader); Assert.Contains(_settings.ApiKey!, authHeader);
Assert.Contains(_settings.ClientName, authHeader); Assert.Contains(_settings.ClientName!, authHeader);
} }
[Fact] [Fact]

2
allstarr.Tests/QobuzDownloadServiceTests.cs
View File

@@ -88,7 +88,7 @@ public class QobuzDownloadServiceTests : IDisposable
var serviceProviderMock = new Mock<IServiceProvider>(); var serviceProviderMock = new Mock<IServiceProvider>();
serviceProviderMock.Setup(sp => sp.GetService(typeof(allstarr.Services.Subsonic.PlaylistSyncService))) serviceProviderMock.Setup(sp => sp.GetService(typeof(allstarr.Services.Subsonic.PlaylistSyncService)))
.Returns(null); .Returns(null!);
return new QobuzDownloadService( return new QobuzDownloadService(
_httpClientFactoryMock.Object, _httpClientFactoryMock.Object,

4
allstarr/Services/Jellyfin/JellyfinStartupValidator.cs
View File

@@ -140,7 +140,7 @@ public class JellyfinStartupValidator : BaseStartupValidator
$"Token=\"{settings.ApiKey}\""; $"Token=\"{settings.ApiKey}\"";
using var request = new HttpRequestMessage(HttpMethod.Get, using var request = new HttpRequestMessage(HttpMethod.Get,
$"{settings.Url.TrimEnd('/')}/System/Info"); $"{settings.Url?.TrimEnd('/')}/System/Info");
request.Headers.Add("Authorization", authHeader); request.Headers.Add("Authorization", authHeader);
var response = await _httpClient.SendAsync(request, cancellationToken); var response = await _httpClient.SendAsync(request, cancellationToken);
@@ -180,7 +180,7 @@ public class JellyfinStartupValidator : BaseStartupValidator
{ {
try try
{ {
var url = $"{settings.Url.TrimEnd('/')}/Library/MediaFolders"; var url = $"{settings.Url?.TrimEnd('/')}/Library/MediaFolders";
if (!string.IsNullOrWhiteSpace(settings.UserId)) if (!string.IsNullOrWhiteSpace(settings.UserId))
{ {
url += $"?userId={settings.UserId}"; url += $"?userId={settings.UserId}";

9
allstarr/Services/SquidWTF/SquidWTFMetadataService.cs
View File

@@ -481,11 +481,11 @@ public class SquidWTFMetadataService : IMusicMetadataService
var songs = new List<Song>(); var songs = new List<Song>();
// Get playlist name for album field // Get playlist name for album field
var playlistName = playlist.Value.TryGetProperty("title", out var titleEl) var playlistName = playlist?.TryGetProperty("title", out var titleEl) == true
? titleEl.GetString() ?? "Unknown Playlist" ? titleEl.GetString() ?? "Unknown Playlist"
: "Unknown Playlist"; : "Unknown Playlist";
if (tracks != null) if (tracks.HasValue)
{ {
int trackIndex = 1; int trackIndex = 1;
foreach (var entry in tracks.Value.EnumerateArray()) foreach (var entry in tracks.Value.EnumerateArray())
@@ -785,6 +785,11 @@ public class SquidWTFMetadataService : IMusicMetadataService
tracks = tracksEl; tracks = tracksEl;
} }
if (!playlist.HasValue)
{
throw new InvalidOperationException("Playlist data is missing");
}
var externalId = playlist.Value.GetProperty("uuid").GetString()!; var externalId = playlist.Value.GetProperty("uuid").GetString()!;
// Get curator/creator name // Get curator/creator name