add reaper start_link in helper

testcontainers · Oct 15, 2023 · 8b93f71 · 8b93f71
1 parent 3bb461c
commit 8b93f71
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 16 deletions.
diff --git a/lib/reaper.ex b/lib/reaper.ex
@@ -15,8 +15,10 @@ defmodule TestcontainersElixir.Reaper do
   end
 
   def register(filter) do
-    :ok = ensure_reaper_is_alive()
-    GenServer.call(__MODULE__, {:register, filter}, 10000)
+    case GenServer.whereis(__MODULE__) do
+      nil -> IO.puts("Reaper is not configured, add it to test_helper.exs with TestcontainersElixir.Reaper.start_link()")
+      _pid -> GenServer.call(__MODULE__, {:register, filter}, 5000)
+    end
   end
 
   @impl true
@@ -79,18 +81,4 @@ defmodule TestcontainersElixir.Reaper do
       packet: :line
     ])
   end
-
-  defp ensure_reaper_is_alive() do
-    case __MODULE__.start_link() do
-      {:error, {:already_started, _}} ->
-        :ok
-
-      {:ok, _} ->
-        :ok
-
-      other ->
-        IO.puts("Ignoring unexpected result from starting reaper: #{inspect(other)}")
-        :ok
-    end
-  end
 end
diff --git a/test/test_helper.exs b/test/test_helper.exs
@@ -1,2 +1,3 @@
 ExUnit.configure(max_cases: System.schedulers_online() * 4)
+TestcontainersElixir.Reaper.start_link()
 ExUnit.start()