From 8933574ce9f0bd0af7f6bf5d6a99d0011839eeb3 Mon Sep 17 00:00:00 2001 From: Adam Date: Tue, 9 Jul 2024 20:22:06 -0400 Subject: [PATCH] Correct PremiereDateComparerTests Correct PremiereDateComparerTests - Adjust expected values, add comments, add extra test --- .../Sorting/PremiereDateComparerTests.cs | 20 ++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/tests/Jellyfin.Server.Implementations.Tests/Sorting/PremiereDateComparerTests.cs b/tests/Jellyfin.Server.Implementations.Tests/Sorting/PremiereDateComparerTests.cs index 153debdfc0..605f68d782 100644 --- a/tests/Jellyfin.Server.Implementations.Tests/Sorting/PremiereDateComparerTests.cs +++ b/tests/Jellyfin.Server.Implementations.Tests/Sorting/PremiereDateComparerTests.cs @@ -23,22 +23,32 @@ namespace Jellyfin.Server.Implementations.Tests.Sorting public PremiereDateTestData() { // Both have premier date + // Expected: x listed first Add( new Movie { PremiereDate = new DateTime(2021, 1, 1) }, new Movie { PremiereDate = new DateTime(2021, 1, 3) }, - 0); + -1); - // Only x has premiere date + // Only x has premiere date, with earlier year than y + // Expected: x listed first Add( new Movie { PremiereDate = new DateTime(2021, 1, 1) }, new Movie { ProductionYear = 2022 }, - 1); + -1); // Only x has premiere date, with same year as y + // Expected: y listed first Add( - new Movie { PremiereDate = new DateTime(2021, 3, 1) }, + new Movie { PremiereDate = new DateTime(2021, 1, 2) }, new Movie { ProductionYear = 2021 }, - 2); + 1); + + // Only x has a premiere date, with later year than y + // Expected: y listed first + Add( + new Movie { PremiereDate = new DateTime(2021, 3, 1) }, + new Movie { ProductionYear = 2020 }, + 1); } } }