From d24dc4726ba3658c8a5b145460a2b125529f3ecc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=81lvaro?= <alvaro.girona@gmail.com>
Date: Sat, 15 Jun 2024 21:24:08 +0200
Subject: [PATCH] Log errors on fact processors

---
 .../facts/champion_picked_item/fact_processor.ex              | 1 +
 .../facts/champion_picked_summoner_spell/fact_processor.ex    | 4 +++-
 .../facts/champion_played_game/fact_processor.ex              | 4 +++-
 3 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/apps/lol_analytics/lib/lol_analytics/facts/champion_picked_item/fact_processor.ex b/apps/lol_analytics/lib/lol_analytics/facts/champion_picked_item/fact_processor.ex
index 08c4a76..cdf915d 100644
--- a/apps/lol_analytics/lib/lol_analytics/facts/champion_picked_item/fact_processor.ex
+++ b/apps/lol_analytics/lib/lol_analytics/facts/champion_picked_item/fact_processor.ex
@@ -14,6 +14,7 @@ defmodule LolAnalytics.Facts.ChampionPickedItem.FactProcessor do
          {:ok, decoded_match} <- Poison.decode(body, as: %LoLAPI.Model.MatchResponse{}) do
       process_game_data(decoded_match)
     else
+      Logger.error("Could not process data from #{url} for ChampionPickedItem")
       _ -> {:error, "Could not process data from #{url}"}
     end
   end
diff --git a/apps/lol_analytics/lib/lol_analytics/facts/champion_picked_summoner_spell/fact_processor.ex b/apps/lol_analytics/lib/lol_analytics/facts/champion_picked_summoner_spell/fact_processor.ex
index ea13979..6446448 100644
--- a/apps/lol_analytics/lib/lol_analytics/facts/champion_picked_summoner_spell/fact_processor.ex
+++ b/apps/lol_analytics/lib/lol_analytics/facts/champion_picked_summoner_spell/fact_processor.ex
@@ -13,7 +13,9 @@ defmodule LolAnalytics.Facts.ChampionPickedSummonerSpell.FactProcessor do
          {:ok, decoded_match} <- Poison.decode(body, as: %LoLAPI.Model.MatchResponse{}) do
       process_game_data(decoded_match)
     else
-      _ -> {:error, "Could not process data from #{url}"}
+      _ ->
+        Logger.error("Could not process data from #{url} for ChampionPickedSummonerSpell")
+        {:error, "Could not process data from #{url}"}
     end
   end
 
diff --git a/apps/lol_analytics/lib/lol_analytics/facts/champion_played_game/fact_processor.ex b/apps/lol_analytics/lib/lol_analytics/facts/champion_played_game/fact_processor.ex
index 12efafb..4f9a7b6 100644
--- a/apps/lol_analytics/lib/lol_analytics/facts/champion_played_game/fact_processor.ex
+++ b/apps/lol_analytics/lib/lol_analytics/facts/champion_played_game/fact_processor.ex
@@ -11,7 +11,9 @@ defmodule LolAnalytics.Facts.ChampionPlayedGame.FactProcessor do
          {:ok, decoded_match} <- Poison.decode(body, as: %LoLAPI.Model.MatchResponse{}) do
       process_game_data(decoded_match)
     else
-      _ -> {:error, "Could not process data from #{url}"}
+      _ ->
+        Logger.error("Could not process data from #{url} for ChampionPlayedGame")
+        {:error, "Could not process data from #{url}"}
     end
   end