Remove walkthrough_notifiers from use BoomNotifier

Make accumulated_occurrences testing more clear
wyeworks · Aug 29, 2024 · 98ff12e · 98ff12e
1 parent 9047686
commit 98ff12e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/lib/boom_notifier.ex b/lib/boom_notifier.ex
@@ -28,10 +28,6 @@ defmodule BoomNotifier do
         fn notifier, options -> BoomNotifier.Api.validate_notifiers(notifier, options) end
       )
 
-      def walkthrough_notifiers(callback) do
-        unquote(config) |> BoomNotifier.Api.walkthrough_notifiers(callback)
-      end
-
       def notify_error(conn, error) do
         BoomNotifier.Api.notify_error(unquote(config), conn, error)
       end

diff --git a/test/unit/notification_sender_test.exs b/test/unit/notification_sender_test.exs
@@ -86,6 +86,8 @@ defmodule BoomNotifier.NotificationSenderTest do
         NotificationSender.trigger_notify(@settings_groupping, error_info)
       end
 
+      assert_receive({:notify_called, %{occurrences: %{accumulated_occurrences: 1}}})
+      assert_receive({:notify_called, %{occurrences: %{accumulated_occurrences: 2}}})
       assert_receive({:notify_called, %{occurrences: %{accumulated_occurrences: 4}}})
     end
   end