mirror of
https://github.com/jellyfin/jellyfin.git
synced 2026-06-03 06:18:28 +01:00
Replace == null with is null
This commit is contained in:
@@ -472,7 +472,7 @@ namespace Jellyfin.Model.Tests
|
||||
}
|
||||
}
|
||||
}
|
||||
else if (playMethod == null)
|
||||
else if (playMethod is null)
|
||||
{
|
||||
Assert.Null(val.SubProtocol);
|
||||
Assert.Equal("stream", uri.Filename);
|
||||
|
||||
@@ -61,7 +61,7 @@ namespace Jellyfin.Naming.Tests.Video
|
||||
|
||||
Assert.Equal(is3D, result.Is3D);
|
||||
|
||||
if (format3D == null)
|
||||
if (format3D is null)
|
||||
{
|
||||
Assert.Null(result?.Format3D);
|
||||
}
|
||||
|
||||
@@ -25,7 +25,7 @@ namespace Jellyfin.Naming.Tests.Video
|
||||
files.Select(i => VideoResolver.Resolve(i, false, _namingOptions)).OfType<VideoFileInfo>().ToList(),
|
||||
_namingOptions).ToList();
|
||||
|
||||
Assert.Single(result.Where(v => v.ExtraType == null));
|
||||
Assert.Single(result.Where(v => v.ExtraType is null));
|
||||
Assert.Single(result.Where(v => v.ExtraType != null));
|
||||
}
|
||||
|
||||
@@ -44,7 +44,7 @@ namespace Jellyfin.Naming.Tests.Video
|
||||
files.Select(i => VideoResolver.Resolve(i, false, _namingOptions)).OfType<VideoFileInfo>().ToList(),
|
||||
_namingOptions).ToList();
|
||||
|
||||
Assert.Single(result.Where(v => v.ExtraType == null));
|
||||
Assert.Single(result.Where(v => v.ExtraType is null));
|
||||
Assert.Single(result.Where(v => v.ExtraType != null));
|
||||
Assert.Equal(2, result[0].AlternateVersions.Count);
|
||||
}
|
||||
|
||||
@@ -328,11 +328,11 @@ namespace Jellyfin.Providers.Tests.Manager
|
||||
People = oldValue
|
||||
};
|
||||
|
||||
var lockedFields = lockField == null ? Array.Empty<MetadataField>() : new[] { (MetadataField)lockField };
|
||||
var lockedFields = lockField is null ? Array.Empty<MetadataField>() : new[] { (MetadataField)lockField };
|
||||
MetadataService<Movie, MovieInfo>.MergeBaseItemData(source, target, lockedFields, replaceData, false);
|
||||
|
||||
actualValue = target.People;
|
||||
return newValue?.Equals(actualValue) ?? actualValue == null;
|
||||
return newValue?.Equals(actualValue) ?? actualValue is null;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
@@ -367,12 +367,12 @@ namespace Jellyfin.Providers.Tests.Manager
|
||||
};
|
||||
property.SetValue(target.Item, oldValue);
|
||||
|
||||
var lockedFields = lockField == null ? Array.Empty<MetadataField>() : new[] { (MetadataField)lockField };
|
||||
var lockedFields = lockField is null ? Array.Empty<MetadataField>() : new[] { (MetadataField)lockField };
|
||||
// generic type doesn't actually matter to call the static method, just has to be filled in
|
||||
MetadataService<TItemType, TIdType>.MergeBaseItemData(source, target, lockedFields, replaceData, false);
|
||||
|
||||
actualValue = property.GetValue(target.Item);
|
||||
return newValue?.Equals(actualValue) ?? actualValue == null;
|
||||
return newValue?.Equals(actualValue) ?? actualValue is null;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -430,7 +430,7 @@ namespace Jellyfin.Providers.Tests.Manager
|
||||
.Returns((int)order);
|
||||
}
|
||||
|
||||
var provider = hasOrder == null
|
||||
var provider = hasOrder is null
|
||||
? new Mock<TProviderType>(MockBehavior.Strict)
|
||||
: hasOrder.As<TProviderType>();
|
||||
provider.Setup(p => p.Name)
|
||||
@@ -473,12 +473,12 @@ namespace Jellyfin.Providers.Tests.Manager
|
||||
Mock<IHasOrder>? hasOrder = null;
|
||||
if (order != null)
|
||||
{
|
||||
hasOrder = forcedProvider == null ? new Mock<IHasOrder>() : forcedProvider.As<IHasOrder>();
|
||||
hasOrder = forcedProvider is null ? new Mock<IHasOrder>() : forcedProvider.As<IHasOrder>();
|
||||
hasOrder.Setup(i => i.Order)
|
||||
.Returns((int)order);
|
||||
}
|
||||
|
||||
var provider = hasOrder == null
|
||||
var provider = hasOrder is null
|
||||
? new Mock<TProviderType>(MockBehavior.Strict)
|
||||
: hasOrder.As<TProviderType>();
|
||||
provider.Setup(p => p.Name)
|
||||
|
||||
@@ -80,7 +80,7 @@ namespace Jellyfin.Providers.Tests.MediaInfo
|
||||
|
||||
var actual = await embeddedImageProvider.GetImage(input, type, CancellationToken.None);
|
||||
Assert.NotNull(actual);
|
||||
if (expectedFormat == null)
|
||||
if (expectedFormat is null)
|
||||
{
|
||||
Assert.False(actual.HasImage);
|
||||
}
|
||||
@@ -131,7 +131,7 @@ namespace Jellyfin.Providers.Tests.MediaInfo
|
||||
|
||||
var actual = await embeddedImageProvider.GetImage(input, type, CancellationToken.None);
|
||||
Assert.NotNull(actual);
|
||||
if (expectedFormat == null)
|
||||
if (expectedFormat is null)
|
||||
{
|
||||
Assert.False(actual.HasImage);
|
||||
}
|
||||
|
||||
@@ -50,7 +50,7 @@ namespace Jellyfin.Server.Implementations.Tests.TypedBaseItem
|
||||
|
||||
private static bool IsProjectAssemblyName(string? name)
|
||||
{
|
||||
if (name == null)
|
||||
if (name is null)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user