clear timers between test cases

test/example_app/test/send_notification_test.exs

@@ -1,5 +1,5 @@
 defmodule ExampleAppWeb.SendNotificationTest do
-  use ExUnit.Case, async: true
+  use BoomNotifier.Case, async: true
   use Wallaby.Feature
 
   def select_email(session, item), do: visit(session, Wallaby.Element.attr(item, "href"))

test/support/case.ex

@@ -0,0 +1,9 @@
+defmodule BoomNotifier.Case do
+  use ExUnit.CaseTemplate
+
+  setup_all do
+    on_exit(fn ->
+      TestUtils.cancel_notification_sender_timers()
+    end)
+  end
+end

test/support/test_utils.ex

@@ -11,6 +11,12 @@ defmodule TestUtils do
     Agent.update(:boom_notifier, fn _ -> %{} end)
   end
 
+  def cancel_notification_sender_timers() do
+    :sys.get_state(Process.whereis(BoomNotifier.NotificationSender))
+    |> Map.values()
+    |> Enum.each(&Process.cancel_timer/1)
+  end
+
   def flush_messages(timeout \\ 10) do
     receive do
       _message -> flush_messages()

test/unit/error_info_test.exs

@@ -1,5 +1,5 @@
 defmodule ErrorInfoTest do
-  use ExUnit.Case
+  use BoomNotifier.Case
 
   import Plug.Conn
   import Phoenix.ConnTest

test/unit/error_storage_test.exs

@@ -1,5 +1,5 @@
 defmodule ErrorStorageTest do
-  use ExUnit.Case
+  use BoomNotifier.Case
 
   alias BoomNotifier.ErrorInfo
   alias BoomNotifier.ErrorStorage

test/unit/mailer_notifier_test.exs

@@ -1,5 +1,5 @@
 defmodule MailerNotifierTest do
-  use ExUnit.Case
+  use BoomNotifier.Case
   use Plug.Test
 
   import TestUtils

test/unit/notification_sender_test.exs

@@ -1,5 +1,5 @@
 defmodule BoomNotifier.NotificationSenderTest do
-  use ExUnit.Case
+  use BoomNotifier.Case
 
   import TestUtils
 
@@ -48,18 +48,12 @@ defmodule BoomNotifier.NotificationSenderTest do
     end
   end
 
-  def cancel_timers() do
-    :sys.get_state(Process.whereis(NotificationSender))
-    |> Map.values()
-    |> Enum.each(&Process.cancel_timer/1)
-  end
-
   setup do
     self() |> Process.register(@pid_name)
     clear_error_storage()
 
     on_exit(&flush_messages/0)
-    on_exit(&cancel_timers/0)
+    on_exit(&cancel_notification_sender_timers/0)
 
     :ok
   end

test/unit/notifier_test.exs

@@ -1,5 +1,5 @@
 defmodule NotifierTest do
-  use ExUnit.Case
+  use BoomNotifier.Case
   use Plug.Test
 
   import ExUnit.CaptureLog

test/unit/use_test.exs

@@ -1,5 +1,5 @@
 defmodule BoomNotifier.UseTest do
-  use ExUnit.Case
+  use BoomNotifier.Case
 
   @already_sent {:plug_conn, :sent}
 

test/unit/webhook_notifier_test.exs

@@ -1,5 +1,5 @@
 defmodule WebhookNotifierTest do
-  use ExUnit.Case, async: false
+  use BoomNotifier.Case
   use Plug.Test
 
   doctest BoomNotifier