Fix tests

Author: vtm9

apps/codebattle/lib/codebattle/tournament/context.ex

@@ -292,6 +292,8 @@ defmodule Codebattle.Tournament.Context do
 
   @spec update(Tournament.t(), map()) :: {:ok, Tournament.t()} | {:error, Ecto.Changeset.t()}
   def update(tournament, params) do
+    params = Map.put_new(params, "creator_id", tournament.creator_id)
+
     tournament
     |> Tournament.changeset(prepare_tournament_params(params))
     |> Repo.update()
@@ -451,7 +453,13 @@ defmodule Codebattle.Tournament.Context do
   end
 
   defp normalize_moderator_ids(%{moderator_ids: moderator_ids} = params) do
-    creator_id = get_in(params, [:creator, :id])
+    creator_id =
+      params
+      |> Map.get(:creator)
+      |> case do
+        nil -> Map.get(params, :creator_id)
+        creator -> Map.get(creator, :id) || Map.get(params, :creator_id)
+      end
 
     normalized_moderator_ids =
       moderator_ids

apps/codebattle/lib/codebattle_web/controllers/admin/group_tournament_controller.ex

@@ -173,11 +173,6 @@ defmodule CodebattleWeb.Admin.GroupTournamentController do
         |> put_flash(:error, "Group tournament checker failed.")
         |> redirect(to: Routes.admin_group_tournament_path(conn, :show, group_tournament))
 
-      {:run_error, result} ->
-        conn
-        |> put_flash(:error, "Group tournament checker failed: #{format_run_error(result)}")
-        |> redirect(to: Routes.admin_group_tournament_path(conn, :show, group_tournament))
-
       {:error, %Ecto.Changeset{} = changeset} ->
         conn
         |> put_flash(:error, "Group tournament checker failed: #{inspect(changeset.errors)}")
@@ -246,11 +241,6 @@ defmodule CodebattleWeb.Admin.GroupTournamentController do
     GroupTaskContext.list_latest_solutions(group_tournament.group_task_id, player_ids)
   end
 
-  defp format_run_error(%{"body" => %{"error" => error}}) when is_binary(error), do: error
-  defp format_run_error(%{"error" => error}) when is_binary(error), do: error
-  defp format_run_error(%{errors: errors}), do: inspect(errors)
-  defp format_run_error(result), do: inspect(result)
-
   defp parse_user_id(user_id) when is_integer(user_id) and user_id > 0, do: {:ok, user_id}
 
   defp parse_user_id(user_id) when is_binary(user_id) do

apps/codebattle/lib/codebattle_web/controllers/api/v1/group_task_solution_controller.ex

@@ -1,7 +1,7 @@
 defmodule CodebattleWeb.Api.V1.GroupTaskSolutionController do
   use CodebattleWeb, :controller
 
-  alias Codebattle.GroupTournament.Context
+  alias Codebattle.GroupTask.Context
 
   def create(conn, params) do
     if FunWithFlags.enabled?(:group_tasks_api) do

apps/codebattle/test/codebattle/user/user_test.exs

@@ -1,4 +1,4 @@
-defmodule Codebattle.UserTest do
+defmodule Codebattle.User.ModelTest do
   use CodebattleWeb.ConnCase, async: false
 
   alias Codebattle.User

apps/codebattle/test/codebattle_web/controllers/api/v1/tournament_controller_test.exs

@@ -1,5 +1,5 @@
 defmodule CodebattleWeb.Api.V1.TournamentControllerTest do
-  use CodebattleWeb.ConnCase, async: true
+  use CodebattleWeb.ConnCase, async: false
 
   alias Codebattle.Tournament.Context