Overhauled tests

test/db_tests/player_test.dart

@@ -31,7 +31,7 @@ void main() {
   });
 
   group('player tests', () {
-    test('all players get fetched correctly', () async {
+    test('Adding and fetching single player works correclty', () async {
       await database.playerDao.addPlayer(player: testPlayer);
       await database.playerDao.addPlayer(player: testPlayer2);
 
@@ -51,18 +51,50 @@ void main() {
       expect(fetchedPlayer2.createdAt, testPlayer2.createdAt);
     });
 
-    test('players get inserted correcly ', () async {
+    // TODO: Use upcoming addPlayers() method
+    test('Adding and fetching multiple players works correclty', () async {
       await database.playerDao.addPlayer(player: testPlayer);
-      final result = await database.playerDao.getPlayerById(
-        playerId: testPlayer.id,
-      );
+      await database.playerDao.addPlayer(player: testPlayer2);
 
-      expect(result.id, testPlayer.id);
-      expect(result.name, testPlayer.name);
-      expect(result.createdAt, testPlayer.createdAt);
+      final allPlayers = await database.playerDao.getAllPlayers();
+      expect(allPlayers.length, 2);
+
+      final fetchedPlayer1 = allPlayers.firstWhere(
+        (g) => g.id == testPlayer.id,
+      );
+      expect(fetchedPlayer1.name, testPlayer.name);
+      expect(fetchedPlayer1.createdAt, testPlayer.createdAt);
+
+      final fetchedPlayer2 = allPlayers.firstWhere(
+        (g) => g.id == testPlayer2.id,
+      );
+      expect(fetchedPlayer2.name, testPlayer2.name);
+      expect(fetchedPlayer2.createdAt, testPlayer2.createdAt);
     });
 
-    test('players get deleted correcly ', () async {
+    test('Adding the same player twice does not create duplicates', () async {
+      await database.playerDao.addPlayer(player: testPlayer);
+      await database.playerDao.addPlayer(player: testPlayer);
+
+      final allPlayers = await database.playerDao.getAllPlayers();
+      expect(allPlayers.length, 1);
+    });
+
+    test('Player existence check works correctly', () async {
+      var playerExists = await database.playerDao.playerExists(
+        playerId: testPlayer.id,
+      );
+      expect(playerExists, false);
+
+      await database.playerDao.addPlayer(player: testPlayer);
+
+      playerExists = await database.playerDao.playerExists(
+        playerId: testPlayer.id,
+      );
+      expect(playerExists, true);
+    });
+
+    test('Deleting a player works correclty', () async {
       await database.playerDao.addPlayer(player: testPlayer);
       final playerDeleted = await database.playerDao.deletePlayer(
         playerId: testPlayer.id,
@@ -75,7 +107,7 @@ void main() {
       expect(playerExists, false);
     });
 
-    test('player name gets updated correcly ', () async {
+    test('Updating a player name works correcly', () async {
       await database.playerDao.addPlayer(player: testPlayer);
 
       const newPlayerName = 'new player name';
@@ -91,22 +123,29 @@ void main() {
       expect(result.name, newPlayerName);
     });
 
-    test('get player count works correctly', () async {
-      final initialCount = await database.playerDao.getPlayerCount();
-      expect(initialCount, 0);
+    test('Getting the player count works correctly', () async {
+      var playerCount = await database.playerDao.getPlayerCount();
+      expect(playerCount, 0);
 
       await database.playerDao.addPlayer(player: testPlayer);
 
-      final playerAdded = await database.playerDao.getPlayerCount();
-      expect(playerAdded, 1);
+      playerCount = await database.playerDao.getPlayerCount();
+      expect(playerCount, 1);
 
-      final playerRemoved = await database.playerDao.deletePlayer(
-        playerId: testPlayer.id,
-      );
-      expect(playerRemoved, true);
+      await database.playerDao.addPlayer(player: testPlayer2);
 
-      final finalCount = await database.playerDao.getPlayerCount();
-      expect(finalCount, 0);
+      playerCount = await database.playerDao.getPlayerCount();
+      expect(playerCount, 2);
+
+      await database.playerDao.deletePlayer(playerId: testPlayer.id);
+
+      playerCount = await database.playerDao.getPlayerCount();
+      expect(playerCount, 1);
+
+      await database.playerDao.deletePlayer(playerId: testPlayer2.id);
+
+      playerCount = await database.playerDao.getPlayerCount();
+      expect(playerCount, 0);
     });
   });
 }