Media item lifecycle improvements (#18)

* Added jobs for enqueuing/dequeuing media based on source's status

* adds lifecycle columns to source and media_item tables

test/pinchflat/tasks/source_tasks_test.exs

@@ -103,14 +103,23 @@ defmodule Pinchflat.Tasks.SourceTasksTest do
       assert Enum.all?(first_run, fn %MediaItem{} -> true end)
       assert Enum.all?(duplicate_run, fn %Ecto.Changeset{} -> true end)
     end
+
+    test "it updates the source's last_indexed_at field", %{source: source} do
+      assert source.last_indexed_at == nil
+
+      SourceTasks.index_media_items(source)
+      source = Repo.reload!(source)
+
+      assert DateTime.diff(DateTime.utc_now(), source.last_indexed_at) < 1
+    end
   end
 
-  describe "enqueue_pending_media_downloads/1" do
+  describe "enqueue_pending_media_tasks/1" do
     test "it enqueues a job for each pending media item" do
       source = source_fixture()
       media_item = media_item_fixture(source_id: source.id, media_filepath: nil)
 
-      assert :ok = SourceTasks.enqueue_pending_media_downloads(source)
+      assert :ok = SourceTasks.enqueue_pending_media_tasks(source)
 
       assert_enqueued(worker: VideoDownloadWorker, args: %{"id" => media_item.id})
     end
@@ -119,7 +128,7 @@ defmodule Pinchflat.Tasks.SourceTasksTest do
       source = source_fixture()
       _media_item = media_item_fixture(source_id: source.id, media_filepath: "some/filepath.mp4")
 
-      assert :ok = SourceTasks.enqueue_pending_media_downloads(source)
+      assert :ok = SourceTasks.enqueue_pending_media_tasks(source)
 
       refute_enqueued(worker: VideoDownloadWorker)
     end
@@ -130,7 +139,7 @@ defmodule Pinchflat.Tasks.SourceTasksTest do
 
       assert [] = Tasks.list_tasks_for(:media_item_id, media_item.id)
 
-      assert :ok = SourceTasks.enqueue_pending_media_downloads(source)
+      assert :ok = SourceTasks.enqueue_pending_media_tasks(source)
 
       assert [_] = Tasks.list_tasks_for(:media_item_id, media_item.id)
     end
@@ -138,7 +147,7 @@ defmodule Pinchflat.Tasks.SourceTasksTest do
     test "it does not create a job if the source is set to not download" do
       source = source_fixture(download_media: false)
 
-      assert :ok = SourceTasks.enqueue_pending_media_downloads(source)
+      assert :ok = SourceTasks.enqueue_pending_media_tasks(source)
 
       refute_enqueued(worker: VideoDownloadWorker)
     end
@@ -147,7 +156,22 @@ defmodule Pinchflat.Tasks.SourceTasksTest do
       source = source_fixture(download_media: false)
       media_item = media_item_fixture(source_id: source.id, media_filepath: nil)
 
-      assert :ok = SourceTasks.enqueue_pending_media_downloads(source)
+      assert :ok = SourceTasks.enqueue_pending_media_tasks(source)
+      assert [] = Tasks.list_tasks_for(:media_item_id, media_item.id)
+    end
+  end
+
+  describe "dequeue_pending_media_tasks/1" do
+    test "it deletes all pending tasks for a source's media items" do
+      source = source_fixture()
+      media_item = media_item_fixture(source_id: source.id, media_filepath: nil)
+
+      SourceTasks.enqueue_pending_media_tasks(source)
+      assert_enqueued(worker: VideoDownloadWorker, args: %{"id" => media_item.id})
+
+      assert :ok = SourceTasks.dequeue_pending_media_tasks(source)
+
+      refute_enqueued(worker: VideoDownloadWorker)
       assert [] = Tasks.list_tasks_for(:media_item_id, media_item.id)
     end
   end