Added new cleanup option: when not favorited

This is another way of solving #2077.

The root issue is that queued episodes are never auto-deleted
currently which means that if you automatically add episodes to the
queue you will eventually end up with AntennaPod refusing to auto
download more episodes because the cache is full and it can't make
space.

This option will only refuse to delete favorited items. Otherwise it
will simply delete the oldest episodes.

1 files changed, 89 insertions, 0 deletions

diff --git a/core/src/test/java/de/danoeh/antennapod/core/storage/ExceptFavoriteCleanupAlgorithmTest.java b/core/src/test/java/de/danoeh/antennapod/core/storage/ExceptFavoriteCleanupAlgorithmTest.java
new file mode 100644
index 000000000..8c02391ca
--- /dev/null
+++ b/core/src/test/java/de/danoeh/antennapod/core/storage/ExceptFavoriteCleanupAlgorithmTest.java
@@ -0,0 +1,89 @@
+package de.danoeh.antennapod.core.storage;
+
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.robolectric.RobolectricTestRunner;
+
+import java.io.File;
+import java.io.IOException;
+import java.util.ArrayList;
+import java.util.List;
+
+import de.danoeh.antennapod.core.feed.Feed;
+import de.danoeh.antennapod.core.feed.FeedItem;
+import de.danoeh.antennapod.core.preferences.UserPreferences;
+
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
+/**
+ * Tests that the APFavoriteCleanupAlgorithm is working correctly.
+ */
+@RunWith(RobolectricTestRunner.class)
+public class ExceptFavoriteCleanupAlgorithmTest extends DbCleanupTests {
+    private final int numberOfItems = EPISODE_CACHE_SIZE * 2;
+
+    public ExceptFavoriteCleanupAlgorithmTest() {
+        setCleanupAlgorithm(UserPreferences.EPISODE_CLEANUP_EXCEPT_FAVORITE);
+    }
+
+    @Test
+    public void testPerformAutoCleanupHandleUnplayed() throws IOException {
+        Feed feed = new Feed("url", null, "title");
+        List<FeedItem> items = new ArrayList<>();
+        feed.setItems(items);
+        List<File> files = new ArrayList<>();
+        populateItems(numberOfItems, feed, items, files, FeedItem.UNPLAYED, false, false);
+
+        DBTasks.performAutoCleanup(context);
+        for (int i = 0; i < files.size(); i++) {
+            if (i < EPISODE_CACHE_SIZE) {
+                assertTrue("Only enough items should be deleted", files.get(i).exists());
+            } else {
+                assertFalse("Expected episode to be deleted", files.get(i).exists());
+            }
+        }
+    }
+
+    @Test
+    public void testPerformAutoCleanupDeletesQueued() throws IOException {
+        Feed feed = new Feed("url", null, "title");
+        List<FeedItem> items = new ArrayList<>();
+        feed.setItems(items);
+        List<File> files = new ArrayList<>();
+        populateItems(numberOfItems, feed, items, files, FeedItem.UNPLAYED, true, false);
+
+        DBTasks.performAutoCleanup(context);
+        for (int i = 0; i < files.size(); i++) {
+            if (i < EPISODE_CACHE_SIZE) {
+                assertTrue("Only enough items should be deleted", files.get(i).exists());
+            } else {
+                assertFalse("Queued episodes should be deleted", files.get(i).exists());
+            }
+        }
+    }
+
+    @Test
+    public void testPerformAutoCleanupSavesFavorited() throws IOException {
+        Feed feed = new Feed("url", null, "title");
+        List<FeedItem> items = new ArrayList<>();
+        feed.setItems(items);
+        List<File> files = new ArrayList<>();
+        populateItems(numberOfItems, feed, items, files, FeedItem.UNPLAYED, false, true);
+
+        DBTasks.performAutoCleanup(context);
+        for (int i = 0; i < files.size(); i++) {
+            assertTrue("Favorite episodes should should not be deleted", files.get(i).exists());
+        }
+    }
+
+    @Override
+    public void testPerformAutoCleanupShouldNotDeleteBecauseInQueue() throws IOException {
+        // Yes it should
+    }
+
+    @Override
+    public void testPerformAutoCleanupShouldNotDeleteBecauseInQueue_withFeedsWithNoMedia() throws IOException {
+        // Yes it should
+    }
+}