fix: save test mode scores before broadcasting GameResult

Prevents race conditions creating errors in the TestClient and failing our CI.

Author: xeruf

server/src/sc/server/gaming/GameRoom.java

@@ -89,6 +89,11 @@ public synchronized void onGameOver(Map<Player, PlayerScore> results) throws Inv
     try {
       result = generateGameResult(results);
       logger.info("{} is over (regular={})", game, result.isRegular());
+      // save playerScore if test mode enabled
+      if (Boolean.parseBoolean(Configuration.get(Configuration.TEST_MODE))) {
+        List<Player> players = game.getPlayers();
+        gameRoomManager.addResultToScore(this.getResult(), players.get(0).getDisplayName(), players.get(1).getDisplayName());
+      }
       broadcast(result);
     } catch(Throwable t) {
       logger.error("Failed to generate GameResult from " + results, t);
@@ -98,12 +103,6 @@ public synchronized void onGameOver(Map<Player, PlayerScore> results) throws Inv
       saveReplay();
     }
 
-    // save playerScore if test mode enabled
-    if (Boolean.parseBoolean(Configuration.get(Configuration.TEST_MODE))) {
-      List<Player> players = game.getPlayers();
-      gameRoomManager.addResultToScore(this.getResult(), players.get(0).getDisplayName(), players.get(1).getDisplayName());
-    }
-
     kickAllClients();
     cancel();
   }