From bd96a70ad98db64b3a444a4e5dffa0716314fcf4 Mon Sep 17 00:00:00 2001
From: Logan Hunt <logan.hunt@usu.edu>
Date: Thu, 19 Jan 2023 13:51:38 -0700
Subject: [PATCH] Add github id in player structs for tests

---
 front/src/routes/home.jsx     | 4 ++--
 test/README.md                | 3 ---
 test/auth/password_test.exs   | 2 +-
 test/auth/pubkey_test.exs     | 2 +-
 test/schema/register_test.exs | 8 ++++----
 test/ssh/ssh_auth_test.exs    | 2 +-
 6 files changed, 9 insertions(+), 12 deletions(-)
 delete mode 100644 test/README.md

diff --git a/front/src/routes/home.jsx b/front/src/routes/home.jsx
index 874463d..baccb5f 100644
--- a/front/src/routes/home.jsx
+++ b/front/src/routes/home.jsx
@@ -55,8 +55,8 @@ export const Home = () => {
   }
 
   return (
-    <div>
+    <>
       <p>Looks like you're not signed in 👀. </p>
-    </div>
+    </>
   );
 };
diff --git a/test/README.md b/test/README.md
deleted file mode 100644
index 13d4389..0000000
--- a/test/README.md
+++ /dev/null
@@ -1,3 +0,0 @@
-TODO:
-- [ ] When password changed, remove all sessions
-- [ ] When session removed, ssh connection closed
diff --git a/test/auth/password_test.exs b/test/auth/password_test.exs
index 348032f..55d9867 100644
--- a/test/auth/password_test.exs
+++ b/test/auth/password_test.exs
@@ -2,7 +2,7 @@ defmodule Chessh.Auth.PasswordAuthenticatorTest do
   use ExUnit.Case
   alias Chessh.{Player, Repo}
 
-  @valid_user %{username: "logan", password: "password"}
+  @valid_user %{username: "logan", password: "password", github_id: 1}
 
   setup_all do
     Ecto.Adapters.SQL.Sandbox.checkout(Repo)
diff --git a/test/auth/pubkey_test.exs b/test/auth/pubkey_test.exs
index da2518b..52d326a 100644
--- a/test/auth/pubkey_test.exs
+++ b/test/auth/pubkey_test.exs
@@ -2,7 +2,7 @@ defmodule Chessh.Auth.PublicKeyAuthenticatorTest do
   use ExUnit.Case
   alias Chessh.{Key, Repo, Player}
 
-  @valid_user %{username: "logan", password: "password"}
+  @valid_user %{username: "logan", password: "password", github_id: 2}
   @valid_key %{
     name: "The Gamer Machine",
     key: "ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIJ/2LOJGGEd/dhFgRxJ5MMv0jJw4s4pA8qmMbZyulN44"
diff --git a/test/schema/register_test.exs b/test/schema/register_test.exs
index 0e9fdf1..00ac413 100644
--- a/test/schema/register_test.exs
+++ b/test/schema/register_test.exs
@@ -3,10 +3,10 @@ defmodule Chessh.Auth.UserRegistrationTest do
   use ExUnit.Case
   alias Chessh.{Player, Repo}
 
-  @valid_user %{username: "logan", password: "password"}
-  @invalid_username %{username: "a", password: "password"}
-  @invalid_password %{username: "aasdf", password: "pass"}
-  @repeated_username %{username: "LoGan", password: "password"}
+  @valid_user %{username: "logan", password: "password", github_id: 4}
+  @invalid_username %{username: "a", password: "password", github_id: 7}
+  @invalid_password %{username: "aasdf", password: "pass", github_id: 6}
+  @repeated_username %{username: "LoGan", password: "password", github_id: 5}
 
   test "Password must be at least 8 characters and username must be at least 2" do
     refute Player.registration_changeset(%Player{}, @invalid_password).valid?
diff --git a/test/ssh/ssh_auth_test.exs b/test/ssh/ssh_auth_test.exs
index 38777e8..54b0069 100644
--- a/test/ssh/ssh_auth_test.exs
+++ b/test/ssh/ssh_auth_test.exs
@@ -5,7 +5,7 @@ defmodule Chessh.SSH.AuthTest do
   @localhost '127.0.0.1'
   @localhost_inet {{127, 0, 0, 1}, 1}
   @key_name "The Gamer Machine"
-  @valid_user %{username: "logan", password: "password"}
+  @valid_user %{username: "logan", password: "password", github_id: 3}
   @client_test_keys_dir Path.join(Application.compile_env!(:chessh, :key_dir), "client_keys")
   @client_pub_key 'id_ed25519.pub'