fix: storage or registry can be nil

Byzanteam · Jun 17, 2023 · fb6f9f6 · fb6f9f6
1 parent 0887b5c
commit fb6f9f6
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 8 deletions.
diff --git a/lib/workflow_metal/application/supervisor.ex b/lib/workflow_metal/application/supervisor.ex
@@ -25,12 +25,16 @@ defmodule WorkflowMetal.Application.Supervisor do
 
     config_child_spec = config_child_spec(application, config)
 
-    children = [
-      config_child_spec,
-      registry_child_spec,
-      storage_child_spec,
-      {WorkflowMetal.Application.WorkflowsSupervisor, application}
-    ]
+    children =
+      Enum.reject(
+        [
+          config_child_spec,
+          registry_child_spec,
+          storage_child_spec,
+          {WorkflowMetal.Application.WorkflowsSupervisor, application}
+        ],
+        &is_nil/1
+      )
 
     Supervisor.init(children, strategy: :one_for_one)
   end

diff --git a/lib/workflow_metal/registration/adapter.ex b/lib/workflow_metal/registration/adapter.ex
@@ -21,7 +21,7 @@ defmodule WorkflowMetal.Registration.Adapter do
   Return a supervisor spec for the registry
   """
   @callback child_spec(application, config) ::
-              {:ok, :supervisor.child_spec() | {module, term} | module, adapter_meta}
+              {:ok, :supervisor.child_spec() | {module, term} | module | nil, adapter_meta}
 
   @doc """
   Return a `:via` tuple to route a message to a process by its registered name

diff --git a/lib/workflow_metal/storage/adapter.ex b/lib/workflow_metal/storage/adapter.ex
@@ -13,7 +13,7 @@ defmodule WorkflowMetal.Storage.Adapter do
   Return a child spec for the storage
   """
   @callback child_spec(application, config) ::
-              {:ok, :supervisor.child_spec() | {module, term} | module, adapter_meta}
+              {:ok, :supervisor.child_spec() | {module, term} | module | nil, adapter_meta}
 
   # Workflow