Merge pull request #458 from guarzo/guarzo/test_merge

refactor: add test coverage for api and operations
2025-12-13 03:06:15 +00:00 · 2025-07-13 02:53:34 +04:00
parent 90a04b517e 7b9e2c4fd9
commit a1f28cd245
60 changed files with 5652 additions and 5688 deletions
--- a/lib/wanderer_app_web/controllers/map_api_controller.ex
+++ b/lib/wanderer_app_web/controllers/map_api_controller.ex
@@ -552,7 +552,11 @@ defmodule WandererAppWeb.MapAPIController do

    with {:ok, map_id} <- APIUtils.fetch_map_id(normalized_params),
         {:ok, days} <- parse_days(params["days"]) do
-      raw_activity = WandererApp.Map.get_character_activity(map_id, days)
+      raw_activity =
+        case WandererApp.Map.get_character_activity(map_id, days) do
+          {:ok, activity} -> activity
+          {:error, _} -> []
+        end

      summarized_result =
        if raw_activity == [] do