Fixed some dialyzer issues removing unused functions

Author: rodrigo-o

lib/lambda_ethereum_consensus/fork_choice/fork_choice.ex

@@ -259,7 +259,6 @@ defmodule LambdaEthereumConsensus.ForkChoice do
     %{slot: slot, body: body} = head_block
 
     OperationsCollector.notify_new_block(head_block)
-
     ExecutionChain.notify_new_block(slot, body.eth1_data, body.execution_payload)
 
     update_fork_choice_data(
@@ -271,7 +270,7 @@ defmodule LambdaEthereumConsensus.ForkChoice do
 
     Libp2pPort.notify_new_block({slot, head_root})
 
-    Logger.info("[Fork choice] Updated fork choice cache", slot: slot)
+    Logger.debug("[Fork choice] Updated fork choice cache", slot: slot)
 
     :ok
   end

lib/lambda_ethereum_consensus/p2p/gossip/beacon_block.ex

@@ -38,19 +38,20 @@ defmodule LambdaEthereumConsensus.P2P.Gossip.BeaconBlock do
     :ok
   end
 
-  @spec subscribe_to_topic() :: :ok | :error
-  def subscribe_to_topic() do
-    topic()
-    |> Libp2pPort.subscribe_to_topic(__MODULE__)
-    |> case do
-      :ok ->
-        :ok
+  # # TODO: Is anyone using this function?
+  # @spec subscribe_to_topic() :: :ok | :error
+  # def subscribe_to_topic() do
+  #   topic()
+  #   |> Libp2pPort.subscribe_to_topic(__MODULE__)
+  #   |> case do
+  #     :ok ->
+  #       :ok
 
-      {:error, reason} ->
-        Logger.error("[Gossip] Subscription failed: '#{reason}'")
-        :error
-    end
-  end
+  #     {:error, reason} ->
+  #       Logger.error("[Gossip] Subscription failed: '#{reason}'")
+  #       :error
+  #   end
+  # end
 
   def topic() do
     fork_context = ForkChoice.get_fork_digest() |> Base.encode16(case: :lower)

lib/lambda_ethereum_consensus/p2p/gossip/blob_sidecar.ex

@@ -26,19 +26,20 @@ defmodule LambdaEthereumConsensus.P2P.Gossip.BlobSideCar do
     end
   end
 
-  @spec subscribe_to_topics() :: :ok | {:error, String.t()}
-  def subscribe_to_topics() do
-    Enum.each(topics(), fn topic ->
-      case Libp2pPort.subscribe_to_topic(topic, __MODULE__) do
-        :ok ->
-          :ok
+  # TODO: Is anyone using this function?
+  # @spec subscribe_to_topics() :: :ok | {:error, String.t()}
+  # def subscribe_to_topics() do
+  #   Enum.each(topics(), fn topic ->
+  #     case Libp2pPort.subscribe_to_topic(topic, __MODULE__) do
+  #       :ok ->
+  #         :ok
 
-        {:error, reason} ->
-          Logger.error("[Gossip] Subscription failed: '#{reason}'")
-          {:error, reason}
-      end
-    end)
-  end
+  #       {:error, reason} ->
+  #         Logger.error("[Gossip] Subscription failed: '#{reason}'")
+  #         {:error, reason}
+  #     end
+  #   end)
+  # end
 
   def topics() do
     # TODO: this doesn't take into account fork digest changes

lib/lambda_ethereum_consensus/p2p/gossip/operations_collector.ex

@@ -40,17 +40,18 @@ defmodule LambdaEthereumConsensus.P2P.Gossip.OperationsCollector do
     "bls_to_execution_change"
   ]
 
-  def subscribe_to_topics() do
-    Enum.reduce_while(topics(), :ok, fn topic, _acc ->
-      case Libp2pPort.subscribe_to_topic(topic, __MODULE__) do
-        :ok ->
-          {:cont, :ok}
-
-        {:error, reason} ->
-          {:halt, {:error, "[OperationsCollector] Subscription failed: '#{reason}'"}}
-      end
-    end)
-  end
+  # TODO: Is anyone using this function?
+  # def subscribe_to_topics() do
+  #   Enum.reduce_while(topics(), :ok, fn topic, _acc ->
+  #     case Libp2pPort.subscribe_to_topic(topic, __MODULE__) do
+  #       :ok ->
+  #         {:cont, :ok}
+
+  #       {:error, reason} ->
+  #         {:halt, {:error, "[OperationsCollector] Subscription failed: '#{reason}'"}}
+  #     end
+  #   end)
+  # end
 
   @spec get_bls_to_execution_changes(non_neg_integer()) :: list(SignedBLSToExecutionChange.t())
   def get_bls_to_execution_changes(count) do