fix Renderable.ensure() not always returning correct type

breunigs · Jul 22, 2024 · 533ad55 · 533ad55
1 parent 9ccc01a
commit 533ad55
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/basemap/renderable.ex b/lib/basemap/renderable.ex
@@ -26,14 +26,15 @@ defmodule Basemap.Renderable do
       def path(:cache, extra), do: Path.join("data/cache/basemap/#{name()}", extra)
       def path(:container, extra), do: Path.join("/workdir/basemap/#{name()}", extra)
 
+      @spec ensure() :: :ok | {:error, reason :: binary()}
       def ensure() do
         ident = "basemap #{name()}"
         {stale, reason} = Benchmark.measure("#{ident} staleness check", &staleness/0)
 
         if stale do
           Logger.info("#{ident} is stale because #{reason}")
           Benchmark.measure("#{ident} rendering", &render/0)
-        end
+        end || :ok
       end
 
       @impl Basemap.Renderable