Updated tests

w3stling · Aug 14, 2023 · b897581 · b897581
1 parent 87a56cc
commit b897581
Showing 1 changed file with 31 additions and 18 deletions.
diff --git a/src/test/java/com/apptasticsoftware/rssreader/util/ItemComparatorTest.java b/src/test/java/com/apptasticsoftware/rssreader/util/ItemComparatorTest.java
@@ -17,23 +17,36 @@ public class ItemComparatorTest {
     @Test
     void testSortNewestItem() throws IOException {
         var items = new RssReader().read("https://www.theverge.com/rss/reviews/index.xml")
-                .sorted(ItemComparator.newestItemFirst())
-                .map(i -> i.getPubDateZonedDateTime().orElse(null))
-                .filter(Objects::nonNull)
-                .map(ZonedDateTime::toEpochSecond)
-                .collect(Collectors.toList());
+                                   .sorted(ItemComparator.newestItemFirst())
+                                   .map(i -> i.getPubDateZonedDateTime().orElse(null))
+                                   .filter(Objects::nonNull)
+                                   .map(ZonedDateTime::toEpochSecond)
+                                   .collect(Collectors.toList());
+
+        assertTrue(isDescendingSortOrder(items));
+    }
+
+    @Test
+    void testSortNewestItemWithCustomDateTimeParser() throws IOException {
+        var items = new RssReader().setDateTimeParser(new DateTime())
+                                   .read("https://www.theverge.com/rss/reviews/index.xml")
+                                   .sorted(ItemComparator.newestItemFirst())
+                                   .map(i -> i.getPubDateZonedDateTime().orElse(null))
+                                   .filter(Objects::nonNull)
+                                   .map(ZonedDateTime::toEpochSecond)
+                                   .collect(Collectors.toList());
 
         assertTrue(isDescendingSortOrder(items));
     }
 
     @Test
     void testSortNewestItemWithDateTimeParser() throws IOException {
         var items = new RssReader().read("https://www.theverge.com/rss/reviews/index.xml")
-                .sorted(ItemComparator.newestItemFirst(new DateTime()))
-                .map(i -> i.getPubDateZonedDateTime().orElse(null))
-                .filter(Objects::nonNull)
-                .map(ZonedDateTime::toEpochSecond)
-                .collect(Collectors.toList());
+                                   .sorted(ItemComparator.newestItemFirst(new DateTime()))
+                                   .map(i -> i.getPubDateZonedDateTime().orElse(null))
+                                   .filter(Objects::nonNull)
+                                   .map(ZonedDateTime::toEpochSecond)
+                                   .collect(Collectors.toList());
 
         assertTrue(isDescendingSortOrder(items));
     }
@@ -52,10 +65,10 @@ void testSortOldestItemFirst() throws IOException {
     @Test
     void testSortOldestItemFirstWithDateTimeParser() throws IOException {
         var items = new RssReader().read("https://www.theverge.com/rss/reviews/index.xml")
-                .sorted(ItemComparator.oldestItemFirst(new DateTime()))
-                .map(i -> i.getPubDateZonedDateTime().orElse(null))
-                .filter(Objects::nonNull)
-                .collect(Collectors.toList());
+                                   .sorted(ItemComparator.oldestItemFirst(new DateTime()))
+                                   .map(i -> i.getPubDateZonedDateTime().orElse(null))
+                                   .filter(Objects::nonNull)
+                                   .collect(Collectors.toList());
 
         assertTrue(isAscendingSortOrder(items));
     }
@@ -64,10 +77,10 @@ void testSortOldestItemFirstWithDateTimeParser() throws IOException {
     void testSortChannelTitle() {
         var urlList = List.of("https://www.theverge.com/rss/reviews/index.xml", "https://feeds.macrumors.com/MacRumors-All");
         var items = new RssReader().read(urlList)
-                .sorted(ItemComparator.channelTitle())
-                .map(i -> i.getChannel().getTitle())
-                .filter(Objects::nonNull)
-                .collect(Collectors.toList());
+                                   .sorted(ItemComparator.channelTitle())
+                                   .map(i -> i.getChannel().getTitle())
+                                   .filter(Objects::nonNull)
+                                   .collect(Collectors.toList());
 
         assertTrue(isAscendingSortOrder(items));
     }