Minor change to joinGame logic (#40)

It is easier if the new player is not included in the game state list --------- Co-authored-by: sormys <szymonp1806@gmail.com>
poker-io · Apr 30, 2023 · 7e6547b · 7e6547b
1 parent 8afb04f
commit 7e6547b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/src/routes/joinGame.ts b/src/routes/joinGame.ts
@@ -73,7 +73,6 @@ router.get(
                 players: [],
                 gameMasterHash: sha256(result.rows[0].game_master).toString(),
               }
-              await client.query(createPlayerQuery, createPlayerValues)
               await client
                 .query(getGameInfoQuery, getGameInfoValues)
                 .then((result) => {
@@ -108,6 +107,7 @@ router.get(
                     }
                   })
                 })
+              await client.query(createPlayerQuery, createPlayerValues)
               res.send(gameInfo)
             }
           })

diff --git a/src/tests/joinGame.test.ts b/src/tests/joinGame.test.ts
@@ -58,10 +58,6 @@ test('Join game, correct arguments', async () => {
         nickname: gameMasterNick,
         playerHash: sha256(gameMasterToken).toString(),
       },
-      {
-        nickname: playerNick,
-        playerHash: sha256(playerToken).toString(),
-      },
     ],
     gameMasterHash: sha256(gameMasterToken).toString(),
   }