From b2bc5f07f7cc57b0953fbc4e1bcf38ce5d0d4e54 Mon Sep 17 00:00:00 2001
From: Claudio Ortolina <cloud8421@gmail.com>
Date: Thu, 3 Jul 2025 21:10:59 +0100
Subject: [PATCH] Remove unnecessary code

---
 lib/music_library/scrobble_rules/worker.ex    | 23 -------------------
 .../scrobble_rules/worker_test.exs            |  6 -----
 2 files changed, 29 deletions(-)

diff --git a/lib/music_library/scrobble_rules/worker.ex b/lib/music_library/scrobble_rules/worker.ex
index 3fa9390b..e60f749c 100644
--- a/lib/music_library/scrobble_rules/worker.ex
+++ b/lib/music_library/scrobble_rules/worker.ex
@@ -19,7 +19,6 @@ defmodule MusicLibrary.ScrobbleRules.Worker do
     case ScrobbleRules.apply_all_rules() do
       {:ok, results} ->
         log_results(results)
-        :ok
 
       {:error, reason} ->
         Logger.error("Failed to apply scrobble rules: #{inspect(reason)}")
@@ -64,26 +63,4 @@ defmodule MusicLibrary.ScrobbleRules.Worker do
       })
     end)
   end
-
-  @doc """
-  Schedules the worker to run periodically.
-
-  This should be called during application startup to ensure
-  the worker runs automatically.
-  """
-  def schedule_periodic do
-    # Schedule to run every 30 minutes
-    __MODULE__.new(%{})
-    |> Oban.insert(schedule_in: {30, :minute})
-  end
-
-  @doc """
-  Triggers the worker to run immediately.
-
-  Useful for manual rule application or testing.
-  """
-  def run_now do
-    __MODULE__.new(%{})
-    |> Oban.insert()
-  end
 end
diff --git a/test/music_library/scrobble_rules/worker_test.exs b/test/music_library/scrobble_rules/worker_test.exs
index f992cbf0..925219dd 100644
--- a/test/music_library/scrobble_rules/worker_test.exs
+++ b/test/music_library/scrobble_rules/worker_test.exs
@@ -86,10 +86,4 @@ defmodule MusicLibrary.ScrobbleRules.WorkerTest do
       assert :ok = Worker.perform(%Oban.Job{args: %{}})
     end
   end
-
-  describe "run_now/0" do
-    test "enqueues job for immediate execution" do
-      assert {:ok, %Oban.Job{}} = Worker.run_now()
-    end
-  end
 end