<.tabbed_layout>
<:tab_append>
- <.button_dropdown text="Actions" class="justify-center w-full sm:w-50">
- <:option>
-
copied = false, 4000)
- "}
- >
- Copy RSS Feed
- <.icon name="hero-check" class="ml-2 h-4 w-4" />
-
-
- <:option>
-
-
- <:option>
- <.link
- href={~p"/sources/#{@source}"}
- method="delete"
- data-confirm="Are you sure you want to delete this source (leaving files in place)? This cannot be undone."
- >
- Delete Source
-
-
- <:option>
- <.link
- href={~p"/sources/#{@source}?delete_files=true"}
- method="delete"
- data-confirm="Are you sure you want to delete this source and it's files on disk? This cannot be undone."
- class="mt-5 md:mt-0"
- >
- Delete Source + Files
-
-
-
+ <.actions_dropdown source={@source} conn={@conn} />
<:tab title="Attributes">
diff --git a/lib/pinchflat_web/router.ex b/lib/pinchflat_web/router.ex
index e56a4f2..75f9ae0 100644
--- a/lib/pinchflat_web/router.ex
+++ b/lib/pinchflat_web/router.ex
@@ -32,6 +32,9 @@ defmodule PinchflatWeb.Router do
resources "/search", Searches.SearchController, only: [:show], singleton: true
resources "/sources", Sources.SourceController do
+ post "/force_download", Sources.SourceController, :force_download
+ post "/force_index", Sources.SourceController, :force_index
+
resources "/media", MediaItems.MediaItemController, only: [:show, :edit, :update, :delete]
end
end
diff --git a/test/pinchflat/slow_indexing/media_collection_indexing_worker_test.exs b/test/pinchflat/slow_indexing/media_collection_indexing_worker_test.exs
index 6037c54..6a3c140 100644
--- a/test/pinchflat/slow_indexing/media_collection_indexing_worker_test.exs
+++ b/test/pinchflat/slow_indexing/media_collection_indexing_worker_test.exs
@@ -14,6 +14,42 @@ defmodule Pinchflat.SlowIndexing.MediaCollectionIndexingWorkerTest do
setup :verify_on_exit!
+ describe "kickoff_with_task/3" do
+ setup do
+ source = source_fixture(index_frequency_minutes: 10)
+
+ {:ok, %{source: source}}
+ end
+
+ test "starts the worker", %{source: source} do
+ assert [] = all_enqueued(worker: MediaCollectionIndexingWorker)
+ assert {:ok, _} = MediaCollectionIndexingWorker.kickoff_with_task(source)
+ assert [_] = all_enqueued(worker: MediaCollectionIndexingWorker)
+ end
+
+ test "attaches a task", %{source: source} do
+ assert {:ok, task} = MediaCollectionIndexingWorker.kickoff_with_task(source)
+ assert task.source_id == source.id
+ end
+
+ test "can be called with additional job arguments", %{source: source} do
+ job_args = %{"force" => true}
+
+ assert {:ok, _} = MediaCollectionIndexingWorker.kickoff_with_task(source, job_args)
+
+ assert_enqueued(worker: MediaCollectionIndexingWorker, args: %{"id" => source.id, "force" => true})
+ end
+
+ test "can be called with additional job options", %{source: source} do
+ job_opts = [max_attempts: 5]
+
+ assert {:ok, _} = MediaCollectionIndexingWorker.kickoff_with_task(source, %{}, job_opts)
+
+ [job] = all_enqueued(worker: MediaCollectionIndexingWorker, args: %{"id" => source.id})
+ assert job.max_attempts == 5
+ end
+ end
+
describe "perform/1" do
test "it indexes the source if it should be indexed" do
expect(YtDlpRunnerMock, :run, fn _url, _opts, _ot, _addl_opts -> {:ok, ""} end)
@@ -31,6 +67,14 @@ defmodule Pinchflat.SlowIndexing.MediaCollectionIndexingWorkerTest do
perform_job(MediaCollectionIndexingWorker, %{id: source.id})
end
+ test "it indexes the source no matter what if the 'force' arg is passed" do
+ expect(YtDlpRunnerMock, :run, fn _url, _opts, _ot, _addl_opts -> {:ok, ""} end)
+
+ source = source_fixture(index_frequency_minutes: 0, last_indexed_at: DateTime.utc_now())
+
+ perform_job(MediaCollectionIndexingWorker, %{id: source.id, force: true})
+ end
+
test "it does not do any indexing if the source has been indexed and shouldn't be rescheduled" do
expect(YtDlpRunnerMock, :run, 0, fn _url, _opts, _ot, _addl_opts -> {:ok, ""} end)
diff --git a/test/pinchflat/slow_indexing/slow_indexing_helpers_test.exs b/test/pinchflat/slow_indexing/slow_indexing_helpers_test.exs
index f183a1e..80e7959 100644
--- a/test/pinchflat/slow_indexing/slow_indexing_helpers_test.exs
+++ b/test/pinchflat/slow_indexing/slow_indexing_helpers_test.exs
@@ -18,7 +18,7 @@ defmodule Pinchflat.SlowIndexing.SlowIndexingHelpersTest do
setup :verify_on_exit!
- describe "kickoff_indexing_task/1" do
+ describe "kickoff_indexing_task/3" do
test "it schedules a job" do
source = source_fixture(index_frequency_minutes: 1)
@@ -64,6 +64,25 @@ defmodule Pinchflat.SlowIndexing.SlowIndexingHelpersTest do
assert_raise Ecto.NoResultsError, fn -> Repo.reload!(task) end
end
+
+ test "can be called with additional job arguments" do
+ source = source_fixture(index_frequency_minutes: 1)
+ job_args = %{"force" => true}
+
+ assert {:ok, _} = SlowIndexingHelpers.kickoff_indexing_task(source, job_args)
+
+ assert_enqueued(worker: MediaCollectionIndexingWorker, args: %{"id" => source.id, "force" => true})
+ end
+
+ test "can be called with additional job options" do
+ source = source_fixture(index_frequency_minutes: 1)
+ job_opts = [max_attempts: 5]
+
+ assert {:ok, _} = SlowIndexingHelpers.kickoff_indexing_task(source, %{}, job_opts)
+
+ [job] = all_enqueued(worker: MediaCollectionIndexingWorker, args: %{"id" => source.id})
+ assert job.max_attempts == 5
+ end
end
describe "index_and_enqueue_download_for_media_items/1" do
diff --git a/test/pinchflat_web/controllers/source_controller_test.exs b/test/pinchflat_web/controllers/source_controller_test.exs
index 8282656..016a01a 100644
--- a/test/pinchflat_web/controllers/source_controller_test.exs
+++ b/test/pinchflat_web/controllers/source_controller_test.exs
@@ -8,6 +8,8 @@ defmodule PinchflatWeb.SourceControllerTest do
alias Pinchflat.Repo
alias Pinchflat.Settings
+ alias Pinchflat.Downloading.MediaDownloadWorker
+ alias Pinchflat.SlowIndexing.MediaCollectionIndexingWorker
setup do
media_profile = media_profile_fixture()
@@ -160,6 +162,59 @@ defmodule PinchflatWeb.SourceControllerTest do
end
end
+ describe "force_download" do
+ test "enqueues pending download tasks", %{conn: conn} do
+ source = source_fixture()
+ _media_item = media_item_fixture(%{source_id: source.id, media_filepath: nil})
+
+ assert [] = all_enqueued(worker: MediaDownloadWorker)
+ post(conn, ~p"/sources/#{source.id}/force_download")
+ assert [_] = all_enqueued(worker: MediaDownloadWorker)
+ end
+
+ test "redirects to the source page", %{conn: conn} do
+ source = source_fixture()
+
+ conn = post(conn, ~p"/sources/#{source.id}/force_download")
+ assert redirected_to(conn) == ~p"/sources/#{source.id}"
+ end
+ end
+
+ describe "force_index" do
+ test "forces an index", %{conn: conn} do
+ source = source_fixture()
+
+ assert [] = all_enqueued(worker: MediaCollectionIndexingWorker)
+ post(conn, ~p"/sources/#{source.id}/force_index")
+ assert [_] = all_enqueued(worker: MediaCollectionIndexingWorker)
+ end
+
+ test "forces an index even if one wouldn't normally run", %{conn: conn} do
+ source = source_fixture(index_frequency_minutes: 0, last_indexed_at: DateTime.utc_now())
+
+ post(conn, ~p"/sources/#{source.id}/force_index")
+ assert [job] = all_enqueued(worker: MediaCollectionIndexingWorker)
+ assert job.args == %{"id" => source.id, "force" => true}
+ end
+
+ test "deletes pending indexing tasks", %{conn: conn} do
+ source = source_fixture()
+ {:ok, task} = MediaCollectionIndexingWorker.kickoff_with_task(source)
+ job = Repo.preload(task, :job).job
+
+ assert job.state == "available"
+ post(conn, ~p"/sources/#{source.id}/force_index")
+ assert Repo.reload!(job).state == "cancelled"
+ end
+
+ test "redirects to the source page", %{conn: conn} do
+ source = source_fixture()
+
+ conn = post(conn, ~p"/sources/#{source.id}/force_index")
+ assert redirected_to(conn) == ~p"/sources/#{source.id}"
+ end
+ end
+
defp create_source(_) do
source = source_fixture()
media_item = media_item_with_attachments(%{source_id: source.id})
diff --git a/test/support/conn_case.ex b/test/support/conn_case.ex
index 1408762..ebf9814 100644
--- a/test/support/conn_case.ex
+++ b/test/support/conn_case.ex
@@ -21,8 +21,10 @@ defmodule PinchflatWeb.ConnCase do
quote do
# The default endpoint for testing
@endpoint PinchflatWeb.Endpoint
+ alias Pinchflat.Repo
use PinchflatWeb, :verified_routes
+ use Oban.Testing, repo: Repo
# Import conveniences for testing with connections
import Plug.Conn