Define child_spec for Postgrex.Notifications, closes #548

elixir-ecto · Jul 27, 2021 · 194488f · 194488f
1 parent e6052cb
commit 194488f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/lib/postgrex/notifications.ex b/lib/postgrex/notifications.ex
@@ -84,6 +84,14 @@ defmodule Postgrex.Notifications do
 
   @type server :: GenServer.server()
 
+  @doc false
+  def child_spec(opts) do
+    %{
+      id: __MODULE__,
+      start: {__MODULE__, :start_link, [opts]}
+    }
+  end
+
   @doc """
   Start the notification connection process and connect to postgres.
 

diff --git a/test/notification_test.exs b/test/notification_test.exs
@@ -12,6 +12,10 @@ defmodule NotificationTest do
     {:ok, [pid: pid, pid_ps: pid_ps]}
   end
 
+  test "defines child spec" do
+    start_supervised!({PN, @opts})
+  end
+
   test "fails on sync connection by default" do
     Process.flag(:trap_exit, true)
     assert {:error, _} = PN.start_link(database: "nobody_knows_it")