Ensure feed generation on create/update

lib/changelog_web/controllers/admin/feed_controller.ex

@@ -32,6 +32,7 @@ defmodule ChangelogWeb.Admin.FeedController do
     case Repo.insert(changeset) do
       {:ok, feed} ->
         Repo.update(Feed.file_changeset(feed, feed_params))
+        FeedUpdater.queue(feed)
 
         conn
         |> put_flash(:result, "success")
@@ -54,6 +55,7 @@ defmodule ChangelogWeb.Admin.FeedController do
 
     case Repo.update(changeset) do
       {:ok, feed} ->
+      FeedUpdater.queue(feed)
         params = replace_next_edit_path(params, ~p"/admin/feeds/#{feed}/edit")
 
         conn

lib/changelog_web/controllers/home/feed_controller.ex

@@ -31,6 +31,7 @@ defmodule ChangelogWeb.Home.FeedController do
     case Repo.insert(changeset) do
       {:ok, feed} ->
         Repo.update(Feed.file_changeset(feed, feed_params))
+        FeedUpdater.queue(feed)
 
         conn
         |> put_flash(:success, "Your new feed has been created! 👏")

test/changelog_web/controllers/home/feed_controller_test.exs

@@ -2,11 +2,22 @@ defmodule ChangelogWeb.HomeFeedControllerTest do
   use ChangelogWeb.ConnCase
   use Changelog.EmailCase
 
-  alias Changelog.Feed
+  import Mock
+
+  alias Changelog.{Feed, ObanWorkers}
 
   @valid_attrs %{name: "My Custom Feed", starts_at: "2024-01-01"}
   @invalid_attrs %{name: ""}
 
+  setup_with_mocks(
+    [
+      {ObanWorkers.FeedUpdater, [], [queue: fn _ -> :ok end]},
+    ],
+    assigns
+  ) do
+    assigns
+  end
+
   @tag :as_inserted_member
   test "renders the index page with no feeds", %{conn: conn} do
     conn = get(conn, ~p"/~/feeds")
@@ -30,8 +41,9 @@ defmodule ChangelogWeb.HomeFeedControllerTest do
   test "creates feed and redirects", %{conn: conn} do
     conn = post(conn, ~p"/~/feeds", feed: @valid_attrs)
 
-    assert redirected_to(conn) == ~p"/~/feeds"
     assert count(Feed) == 1
+    assert called(ObanWorkers.FeedUpdater.queue(:_))
+    assert redirected_to(conn) == ~p"/~/feeds"
   end
 
   @tag :as_inserted_member
@@ -57,8 +69,19 @@ defmodule ChangelogWeb.HomeFeedControllerTest do
 
     conn = put(conn, ~p"/~/feeds/#{feed}", feed: @valid_attrs)
 
-    assert redirected_to(conn) == ~p"/~/feeds"
     assert count(Feed) == 1
+    assert called(ObanWorkers.FeedUpdater.queue(:_))
+    assert redirected_to(conn) == ~p"/~/feeds"
+  end
+
+  @tag :as_inserted_member
+  test "refreshes feed and redirects", %{conn: conn} do
+    feed = insert(:feed, owner: conn.assigns.current_user)
+
+    conn = post(conn, ~p"/~/feeds/#{feed}/refresh")
+
+    assert called(ObanWorkers.FeedUpdater.queue(:_))
+    assert redirected_to(conn) == ~p"/~/feeds"
   end
 
   @tag :as_inserted_member
@@ -67,6 +90,7 @@ defmodule ChangelogWeb.HomeFeedControllerTest do
 
     conn = put(conn, ~p"/~/feeds/#{feed}", feed: @invalid_attrs)
 
+    refute called(ObanWorkers.FeedUpdater.queue(:_))
     assert html_response(conn, 200) =~ ~r/error/
   end