Added new test case for FeedItemUtil.java to test getIds() method (#5182)

1 files changed, 51 insertions, 8 deletions

diff --git a/core/src/test/java/de/danoeh/antennapod/core/util/FeedItemUtilTest.java b/core/src/test/java/de/danoeh/antennapod/core/util/FeedItemUtilTest.java
index ffb788c25..6bc3c09a2 100644
--- a/core/src/test/java/de/danoeh/antennapod/core/util/FeedItemUtilTest.java
+++ b/core/src/test/java/de/danoeh/antennapod/core/util/FeedItemUtilTest.java
@@ -5,9 +5,11 @@ import org.junit.runner.RunWith;
 import org.junit.runners.Parameterized;
 import org.junit.runners.Parameterized.Parameters;
 
+import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
+import java.util.List;
 
 import de.danoeh.antennapod.model.feed.Feed;
 import de.danoeh.antennapod.model.feed.FeedItem;
@@ -26,14 +28,14 @@ public class FeedItemUtilTest {
 
     @Parameters
     public static Collection<Object[]> data() {
-        return Arrays.asList(new Object[][] {
-                { "average", FEED_LINK, ITEM_LINK, ITEM_LINK },
-                { "null item link - fallback to feed", FEED_LINK, null, FEED_LINK},
-                { "empty item link - same as null", FEED_LINK, "", FEED_LINK},
-                { "blank item link - same as null", FEED_LINK, "  ", FEED_LINK},
-                { "fallback, but feed link is null too", null, null, null },
-                { "fallback - but empty feed link - same as null", "", null, null},
-                { "fallback - but blank feed link - same as null", "  ", null, null}
+        return Arrays.asList(new Object[][]{
+                {"average", FEED_LINK, ITEM_LINK, ITEM_LINK},
+                {"null item link - fallback to feed", FEED_LINK, null, FEED_LINK},
+                {"empty item link - same as null", FEED_LINK, "", FEED_LINK},
+                {"blank item link - same as null", FEED_LINK, "  ", FEED_LINK},
+                {"fallback, but feed link is null too", null, null, null},
+                {"fallback - but empty feed link - same as null", "", null, null},
+                {"fallback - but blank feed link - same as null", "  ", null, null}
         });
     }
 
@@ -44,6 +46,47 @@ public class FeedItemUtilTest {
         this.expected = expected;
     }
 
+
+    // Test the getIds() method
+    @Test
+    public void testGetIds() {
+        List<FeedItem> feedItemsList = new ArrayList<FeedItem>(5);
+        List<Integer> idList = new ArrayList<Integer>();
+
+        idList.add(980);
+        idList.add(324);
+        idList.add(226);
+        idList.add(164);
+        idList.add(854);
+
+        for (int i = 0; i < 5; i++) {
+            FeedItem item = createFeedItem(feedLink, itemLink);
+            item.setId(idList.get(i));
+            feedItemsList.add(item);
+        }
+
+        long[] actual = FeedItemUtil.getIds(feedItemsList);
+
+        // covers edge case for getIds() method
+        List<FeedItem> emptyList = new ArrayList<FeedItem>();
+        long[] testEmptyList = FeedItemUtil.getIds(emptyList);
+        assertEquals(msg, 0, testEmptyList.length);
+        assertEquals(msg, 980, actual[0]);
+        assertEquals(msg, 324, actual[1]);
+        assertEquals(msg, 226, actual[2]);
+        assertEquals(msg, 164, actual[3]);
+        assertEquals(msg, 854, actual[4]);
+
+    }
+
+    // Tests the Null value for getLinkWithFallback() method
+    @Test
+    public void testLinkWithFallbackNullValue() {
+        String actual = FeedItemUtil.getLinkWithFallback(null);
+        assertEquals(msg, null, actual);
+    }
+
+
     @Test
     public void testLinkWithFallback() {
         String actual = FeedItemUtil.getLinkWithFallback(createFeedItem(feedLink, itemLink));