fix: remove the :ok tuple when calling get_current_status_message (#1039)

Author: Arkenan

lib/beacon_api/helpers.ex

@@ -48,7 +48,7 @@ defmodule BeaconApi.Helpers do
   @spec block_root_by_block_id(block_id()) ::
           {:ok, root_info()} | {:error, String.t()} | :not_found | :empty_slot | :invalid_id
   def block_root_by_block_id(:head) do
-    with {:ok, current_status} <- BeaconChain.get_current_status_message() do
+    with current_status <- BeaconChain.get_current_status_message() do
       # TODO compute is_optimistic_or_invalid
       execution_optimistic = true
       {:ok, {current_status.head_root, execution_optimistic, false}}

lib/lambda_ethereum_consensus/beacon/beacon_chain.ex

@@ -89,7 +89,7 @@ defmodule LambdaEthereumConsensus.Beacon.BeaconChain do
   @spec get_fork_version() :: Types.version()
   def get_fork_version(), do: GenServer.call(__MODULE__, :get_fork_version)
 
-  @spec get_current_status_message() :: {:ok, Types.StatusMessage.t()} | {:error, any}
+  @spec get_current_status_message() :: Types.StatusMessage.t()
   def get_current_status_message(), do: GenServer.call(__MODULE__, :get_current_status_message)
 
   ##########################

test/unit/beacon_api/beacon_api_v1_test.exs

@@ -33,7 +33,7 @@ defmodule Unit.BeaconApiTest.V1 do
 
     start_link_supervised!({Db, dir: tmp_dir})
 
-    patch(BeaconChain, :get_current_status_message, {:ok, status_message})
+    patch(BeaconChain, :get_current_status_message, status_message)
     patch(BeaconChain, :get_genesis_time, 42)
 
     :ok