Adapt

config/config.exs

@@ -8,7 +8,7 @@ config :archethic_fas,
     uco: 6887,
     matic: 3890,
     bnb: 1839,
-    bitcoin: 1,
+    btc: 1,
     eth: 1027
   }
 

lib/archethic_fas/quotes/cache.ex

@@ -20,28 +20,40 @@ defmodule ArchethicFAS.Quotes.Cache do
   @spec get_latest() :: {:ok, %{Currency.t() => float()}} | {:error, String.t()}
   def get_latest() do
     case :ets.lookup(@table, :latest) do
-      [{:latest, value}] -> value
-      [] -> {:error, "Value not cached yet"}
+      [{:latest, value}] ->
+        value
+
+      [] ->
+        case hydrate() do
+          {:ok, value} ->
+            value
+
+          {:error, reason} ->
+            Logger.warning("Hydrating failed: #{inspect(reason)}")
+            {:error, "Value not cached yet"}
+        end
     end
   end
 
+  def hydrate do
+    GenServer.call(__MODULE__, :hydrate)
+  end
+
   def init([]) do
     :ets.new(@table, [:named_table, :set, :public])
 
     {:ok, :no_state}
   end
 
-  def handle_info(:hydrate, state) do
+  def handle_call(:hydrate, _from, state) do
     case Quotes.fetch_latest(Currency.list()) do
       {:ok, result} ->
         :ets.insert(@table, {:latest, {:ok, result}})
+        {:reply, {:ok, result}, state}
 
-      {:error, reason} ->
-        Logger.warning("Hydrating failed: #{inspect(reason)}")
-
-        :ets.insert(@table, {:latest, {:error, reason}})
+      {:error, _reason} = e ->
+        :ets.delete(@table, :latest)
+        {:reply, e, state}
     end
-
-    {:noreply, state}
   end
 end

lib/archethic_fas/quotes/currency.ex

@@ -5,7 +5,7 @@ defmodule ArchethicFAS.Quotes.Currency do
 
   @ucids Application.compile_env!(:archethic_fas, :ucids)
 
-  @type t :: :eth | :bnb | :matic | :bitcoin | :uco
+  @type t :: :eth | :bnb | :matic | :btc | :uco
 
   @doc """
   Return all handled currencies
@@ -22,7 +22,7 @@ defmodule ArchethicFAS.Quotes.Currency do
   def cast("eth"), do: {:ok, :eth}
   def cast("bnb"), do: {:ok, :bnb}
   def cast("matic"), do: {:ok, :matic}
-  def cast("bitcoin"), do: {:ok, :bitcoin}
+  def cast("btc"), do: {:ok, :btc}
   def cast("uco"), do: {:ok, :uco}
   def cast(_), do: :error
 

lib/archethic_fas/quotes/scheduler.ex

@@ -1,6 +1,6 @@
 defmodule ArchethicFAS.Quotes.Scheduler do
   @moduledoc """
-  Responsible of calling MFA every N ms
+  Responsible of hydrating the cache each minute
   """
 
   alias ArchethicFAS.Quotes.Cache
@@ -18,7 +18,7 @@ defmodule ArchethicFAS.Quotes.Scheduler do
   end
 
   def handle_info(:tick, state) do
-    send(Cache, :hydrate)
+    Cache.hydrate()
 
     Process.send_after(self(), :tick, 60_000)
     {:noreply, state}

test/archethic_fas/quotes/currency_test.exs

@@ -9,7 +9,7 @@ defmodule ArchethicFAS.Quotes.CurrencyTest do
       assert {:ok, :eth} = Currency.cast("eth")
       assert {:ok, :bnb} = Currency.cast("bnb")
       assert {:ok, :matic} = Currency.cast("matic")
-      assert {:ok, :bitcoin} = Currency.cast("bitcoin")
+      assert {:ok, :btc} = Currency.cast("btc")
       assert {:ok, :uco} = Currency.cast("uco")
     end
 

test/archethic_fas/quotes/provider/coin_market_cap_test.exs

@@ -20,16 +20,16 @@ defmodule ArchethicFAS.Quotes.Provider.CoinMarketCapTest do
       assert {:ok, map} = CoinMarketCap.fetch_latest([:matic])
       assert [:matic] = Map.keys(map)
       assert Enum.all?(Map.values(map), &is_float/1)
-      assert {:ok, map} = CoinMarketCap.fetch_latest([:bitcoin])
-      assert [:bitcoin] = Map.keys(map)
+      assert {:ok, map} = CoinMarketCap.fetch_latest([:btc])
+      assert [:btc] = Map.keys(map)
       assert Enum.all?(Map.values(map), &is_float/1)
       assert {:ok, map} = CoinMarketCap.fetch_latest([:bnb])
       assert [:bnb] = Map.keys(map)
       assert Enum.all?(Map.values(map), &is_float/1)
       assert {:ok, map} = CoinMarketCap.fetch_latest([:uco])
       assert [:uco] = Map.keys(map)
       assert Enum.all?(Map.values(map), &is_float/1)
-      assert {:ok, map} = CoinMarketCap.fetch_latest([:eth, :matic, :bitcoin, :bnb, :uco])
+      assert {:ok, map} = CoinMarketCap.fetch_latest([:eth, :matic, :btc, :bnb, :uco])
       assert 5 = map_size(map)
       assert Enum.all?(Map.values(map), &is_float/1)
     end