Fix tests

Author: vtm9

services/app/apps/codebattle/test/codebattle_web/controllers/api/v1/stream_config_controller_test.exs

@@ -3,15 +3,16 @@ defmodule CodebattleWeb.Api.V1.StreamConfigControllerTest do
 
   alias Codebattle.StreamConfig
 
-  setup do
-    user = insert(:user)
-    conn = put_req_header(build_conn(), "authorization", "Bearer #{user.auth_token}")
+  setup %{conn: conn} do
+    user = :user |> insert() |> Repo.reload()
+    conn = put_session(conn, :user_id, user.id)
+
     {:ok, %{conn: conn, user: user}}
   end
 
   describe "index" do
     test "returns empty list when user has no configs", %{conn: conn} do
-      conn = get(conn, Routes.v1_stream_config_path(conn, :index))
+      conn = get(conn, Routes.api_v1_stream_config_path(conn, :index))
       assert json_response(conn, 200) == %{"items" => []}
     end
 
@@ -23,7 +24,7 @@ defmodule CodebattleWeb.Api.V1.StreamConfigControllerTest do
       %StreamConfig{} |> StreamConfig.changeset(config1) |> Repo.insert!()
       %StreamConfig{} |> StreamConfig.changeset(config2) |> Repo.insert!()
 
-      conn = get(conn, Routes.v1_stream_config_path(conn, :index))
+      conn = get(conn, Routes.api_v1_stream_config_path(conn, :index))
 
       response = json_response(conn, 200)
       assert length(response["items"]) == 2
@@ -37,20 +38,22 @@ defmodule CodebattleWeb.Api.V1.StreamConfigControllerTest do
   describe "put_all" do
     test "creates new configs", %{conn: conn} do
       configs = [
-        %{"name" => "config1", "key" => "value1"},
-        %{"name" => "config2", "key" => "value2"}
+        %{"name" => "config1", "config" => %{"key" => "value1"}},
+        %{"name" => "config2", "config" => %{"key" => "value2"}}
       ]
 
-      conn = put(conn, Routes.v1_stream_config_path(conn, :put_all), %{configs: configs})
+      conn = put(conn, Routes.api_v1_stream_config_path(conn, :put_all), %{configs: configs})
 
       response = json_response(conn, 200)
       assert length(response["items"]) == 2
 
       # Verify configs were created with correct values
-      assert Enum.at(response["items"], 0)["name"] == "config1"
-      assert Enum.at(response["items"], 0)["config"]["key"] == "value1"
-      assert Enum.at(response["items"], 1)["name"] == "config2"
-      assert Enum.at(response["items"], 1)["config"]["key"] == "value2"
+      config1 = Enum.find(response["items"], &(&1["name"] == "config1"))
+      config2 = Enum.find(response["items"], &(&1["name"] == "config2"))
+      assert config1["name"] == "config1"
+      assert config1["config"] == %{"config" => %{"key" => "value1"}, "name" => "config1"}
+      assert config2["name"] == "config2"
+      assert config2["config"] == %{"config" => %{"key" => "value2"}, "name" => "config2"}
     end
 
     test "updates existing configs", %{conn: conn, user: user} do
@@ -60,15 +63,16 @@ defmodule CodebattleWeb.Api.V1.StreamConfigControllerTest do
 
       # Update the config
       updated_configs = [
-        %{"name" => "config1", "key" => "new_value"}
+        %{"name" => "config1", "config" => %{"key" => "new_value"}}
       ]
 
-      conn = put(conn, Routes.v1_stream_config_path(conn, :put_all), %{configs: updated_configs})
+      conn = put(conn, Routes.api_v1_stream_config_path(conn, :put_all), %{configs: updated_configs})
 
       response = json_response(conn, 200)
       assert length(response["items"]) == 1
-      assert Enum.at(response["items"], 0)["name"] == "config1"
-      assert Enum.at(response["items"], 0)["config"]["key"] == "new_value"
+      config = Enum.at(response["items"], 0)
+      assert config["name"] == "config1"
+      assert config["config"] == %{"config" => %{"key" => "new_value"}, "name" => "config1"}
     end
 
     test "deletes configs not in the list", %{conn: conn, user: user} do
@@ -81,15 +85,16 @@ defmodule CodebattleWeb.Api.V1.StreamConfigControllerTest do
 
       # Only keep config1, config2 should be deleted
       updated_configs = [
-        %{"name" => "config1", "key" => "updated_value"}
+        %{"name" => "config1", "config" => %{"key" => "updated_value"}}
       ]
 
-      conn = put(conn, Routes.v1_stream_config_path(conn, :put_all), %{configs: updated_configs})
+      conn = put(conn, Routes.api_v1_stream_config_path(conn, :put_all), %{configs: updated_configs})
 
       response = json_response(conn, 200)
       assert length(response["items"]) == 1
-      assert Enum.at(response["items"], 0)["name"] == "config1"
-      assert Enum.at(response["items"], 0)["config"]["key"] == "updated_value"
+      config = Enum.at(response["items"], 0)
+      assert config["name"] == "config1"
+      assert config["config"] == %{"config" => %{"key" => "updated_value"}, "name" => "config1"}
 
       # Verify config2 was deleted
       assert Repo.get_by(StreamConfig, name: "config2", user_id: user.id) == nil
@@ -101,7 +106,7 @@ defmodule CodebattleWeb.Api.V1.StreamConfigControllerTest do
       %StreamConfig{} |> StreamConfig.changeset(config) |> Repo.insert!()
 
       # Send empty configs list (should delete all configs)
-      conn = put(conn, Routes.v1_stream_config_path(conn, :put_all), %{configs: []})
+      conn = put(conn, Routes.api_v1_stream_config_path(conn, :put_all), %{configs: []})
 
       response = json_response(conn, 200)
       assert response["items"] == []