Fix stats treating result list as a single result

server/games/game.py

@@ -710,7 +710,7 @@ def get_army_score(self, army):
             score = max(score, result[2])
         return score
 
-    def get_army_result(self, player):
+    def get_army_results(self, player):
         return self._results.get(self.get_player_option(player.id, 'Army'))
 
     def compute_rating(self, rating='global'):

server/stats/game_stats_service.py

@@ -42,8 +42,8 @@ async def process_game_stats(self, player: Player, game: Game, stats_json):
             self._logger.warn("Player %s reported stats of a game he was not part of", player.login)
             return
 
-        army_result = game.get_army_result(player)
-        if not army_result:
+        army_results = game.get_army_results(player)
+        if not army_results:
             self._logger.warn("No army result available for player %s", player.login)
             return
 
@@ -54,9 +54,9 @@ async def process_game_stats(self, player: Player, game: Game, stats_json):
         a_queue = []
         # Stores events to batch update
         e_queue = []
-        self._logger.debug('Army result for %s => %s ', player, army_result)
+        self._logger.debug('Army result for %s => %s ', player, army_results)
 
-        survived = army_result[1] == 'victory'
+        survived = any(result[1] == 'victory' for result in army_results)
         blueprint_stats = stats['blueprints']
         unit_stats = stats['units']
         scored_highest = highest_scorer == player.login

tests/unit_tests/test_game_stats_service.py

@@ -40,7 +40,7 @@ def player():
 def game(game_stats_service, player):
     game = Game(1, Mock(), game_stats_service)
     game._player_options[player.id] = {'Army': 1}
-    game._results = {1: ['', 'victory', '']}
+    game._results = {1: [('', 'victory', '')]}
     return game