assets/css/components/_board.scss

@@ -8,16 +8,13 @@
   width: 1.5%;
 }
 
-.board__container {
-  grid-area: board;
-}
-
 .board {
   background: $background-color;
   border-collapse: unset;
   border-radius: 2.8%;
   border-spacing: 1px;
   color: $foreground-color;
+  grid-area: board;
   height: var(--board-size);
   padding: calc(var(--board-size) / 20);
   position: relative;

lib/chess_web/controllers/game_controller.ex

@@ -15,9 +15,9 @@ defmodule ChessWeb.GameController do
       conn
       |> current_user()
       |> Game.for_user()
-      |> Game.ordered()
+      |> Game.ordered
       |> preload([:user, :opponent])
-      |> Repo.all()
+      |> Repo.all
 
     conn
     |> render("index.html", games: games, changeset: changeset)
@@ -42,7 +42,7 @@ defmodule ChessWeb.GameController do
           |> Repo.preload(:user)
           |> Repo.preload(:opponent)
         )
-        |> Mailer.deliver_later()
+        |> Mailer.deliver_later
 
         conn
         |> put_flash(:info, "Game created successfully.")
@@ -53,7 +53,7 @@ defmodule ChessWeb.GameController do
           conn
           |> current_user()
           |> User.opponents()
-          |> Repo.all()
+          |> Repo.all
 
         conn
         |> render("new.html", changeset: changeset, opponents: opponents)

lib/chess_web/templates/game/board.html.leex

@@ -37,7 +37,5 @@
     <% end %>
   </div>
 
-  <div class="game-state game-state--<%= @game.state %>">
+  <div class="game-state game-state--null"></div>
-    <%= states[@game.state] %>
-  </div>
 </div>

lib/chess_web/templates/password/edit.html.eex

@@ -1,7 +1,7 @@
 <div class="form">
   <h2><%= gettext "Password" %></h2>
 
-  <%= form_for @changeset, password_path(@conn, :update), [class: "update-password", novalidate: true], fn f -> %>
+  <%= form_for @changeset, password_path(@conn, :update), [class: "update-password"], fn f -> %>
     <%= if @changeset.action do %>
       <div class="alert alert-danger">
         <p>

lib/chess_web/templates/profile/edit.html.eex

@@ -1,7 +1,7 @@
 <div class="form">
   <h2><%= gettext "Profile" %></h2>
 
-  <%= form_for @changeset, profile_path(@conn, :update), [class: "update-profile", novalidate: true], fn f -> %>
+  <%= form_for @changeset, profile_path(@conn, :update), [class: "update-profile"], fn f -> %>
     <%= if @changeset.action do %>
       <div class="alert alert-danger">
         <p>

lib/chess_web/views/game_view.ex

@@ -7,8 +7,8 @@ defmodule ChessWeb.GameView do
 
   def won_lost(conn, game) do
     if game_over?(game) && game.state == "checkmate" do
-      (your_turn?(conn, game) &&
+      your_turn?(conn, game) &&
-         gettext("You lost")) ||
+        gettext("You lost") ||
         gettext("You won")
     end
   end
@@ -21,12 +21,9 @@ defmodule ChessWeb.GameView do
     cond do
       GameState.game_over?(game) ->
         states()[game.state]
-
       your_turn?(conn, game) ->
         gettext("Your turn")
-
+      true -> nil
-      true ->
-        nil
     end
   end
 
@@ -37,18 +34,16 @@ defmodule ChessWeb.GameView do
   end
 
   def your_turn?(conn, game) do
-    conn
+    player_colour(conn, game) == game.turn
-    |> current_user()
-    |> player_colour(game) == game.turn
   end
 
   def player_colour(user, game) do
-    (user.id == game.user_id && "white") || "black"
+    user.id == game.user_id && "white" || "black"
   end
 
   def files(conn, game) do
     ranks(conn, game)
-    |> Enum.reverse()
+    |> Enum.reverse
   end
 
   def ranks(conn, game) do
@@ -79,7 +74,7 @@ defmodule ChessWeb.GameView do
     %{
       "checkmate" => gettext("Checkmate!"),
       "stalemate" => gettext("Stalemate"),
-      "check" => gettext("Check")
+      "check" => gettext("Check"),
     }
   end
 end

lib/chess_web/views/live/board_live.ex

@@ -1,5 +1,5 @@
 defmodule ChessWeb.BoardLive do
-  use Phoenix.LiveView, container: {:div, class: "board__container"}
+  use Phoenix.LiveView
 
   alias Chess.Store.User
   alias Chess.Store.Game

test/chess_web/controllers/game_controller_test.exs

@@ -60,7 +60,7 @@ defmodule ChessWeb.GameControllerTest do
       |> login(user)
       |> get(game_path(conn, :show, game))
 
-    assert html_response(conn, 200) =~ "<div data-game-id=\"#{game.id}\">"
+    assert html_response(conn, 200) =~ "<div id=\"game\" data-game-id=\"#{game.id}\">"
   end
 
   test "does not show a game if the user is not a player", %{conn: conn} do
@@ -74,25 +74,25 @@ defmodule ChessWeb.GameControllerTest do
       conn
       |> login(other_user)
 
-    assert_error_sent(404, fn ->
+    assert_error_sent 404, fn ->
-      get(conn, game_path(conn, :show, game.id))
+      get conn, game_path(conn, :show, game.id)
-    end)
+    end
   end
 
   test "renders page not found when id is nonexistent", %{conn: conn} do
     user = insert(:user)
     conn = login(conn, user)
 
-    assert_error_sent(404, fn ->
+    assert_error_sent 404, fn ->
-      get(conn, game_path(conn, :show, -1))
+      get conn, game_path(conn, :show, -1)
-    end)
+    end
   end
 
   test "deletes game", %{conn: conn} do
-    game = Repo.insert!(%Game{})
+    game = Repo.insert! %Game{}
     user = insert(:user)
     conn = login(conn, user)
-    conn = delete(conn, game_path(conn, :delete, game))
+    conn = delete conn, game_path(conn, :delete, game)
     assert redirected_to(conn) == game_path(conn, :index)
     refute Repo.get(Game, game.id)
   end

test/features/profile_test.exs

@@ -7,12 +7,11 @@ defmodule Chess.Features.ProfileTest do
   import Chess.AuthenticationHelpers
 
   test "user can update their details", %{session: session} do
-    user =
+    user = insert(:user, %{
-      insert(:user, %{
+      name: "Link",
-        name: "Link",
+      email: "link@hyrule.com",
-        email: "link@hyrule.com",
+      password: "ilovezelda"
-        password: "ilovezelda"
+    })
-      })
 
     session
     |> login(user.email, "ilovezelda")
@@ -26,12 +25,11 @@ defmodule Chess.Features.ProfileTest do
   end
 
   test "name cannot be blank", %{session: session} do
-    user =
+    user = insert(:user, %{
-      insert(:user, %{
+      name: "Link",
-        name: "Link",
+      email: "link@hyrule.com",
-        email: "link@hyrule.com",
+      password: "ilovezelda"
-        password: "ilovezelda"
+    })
-      })
 
     session
     |> login(user.email, "ilovezelda")
@@ -42,16 +40,17 @@ defmodule Chess.Features.ProfileTest do
     |> click(button("Update Profile"))
 
     session
-    |> assert_has(css("[data-role='name-error']", text: "can't be blank"))
+    |> assert_has(
+      css("[data-role='name-error']", text: "can't be blank")
+    )
   end
 
   test "email cannot be blank", %{session: session} do
-    user =
+    user = insert(:user, %{
-      insert(:user, %{
+      name: "Link",
-        name: "Link",
+      email: "link@hyrule.com",
-        email: "link@hyrule.com",
+      password: "ilovezelda"
-        password: "ilovezelda"
+    })
-      })
 
     session
     |> login(user.email, "ilovezelda")
@@ -62,6 +61,8 @@ defmodule Chess.Features.ProfileTest do
     |> click(button("Update Profile"))
 
     session
-    |> assert_has(css("[data-role='email-error']", text: "can't be blank"))
+    |> assert_has(
+      css("[data-role='email-error']", text: "can't be blank")
+    )
   end
 end