Merge "sync-tennis-stats: Bail out of number of players is too low"

jenkins-bot · Gerrit Code Review · commit cc8ec9bf84e6 · 2024-07-27T09:49:00.000Z
diff --git a/majavahbot/tasks/task_sync_tennis_stats.py b/majavahbot/tasks/task_sync_tennis_stats.py
@@ -102,12 +102,17 @@ def download_and_parse(
                     )
                 )
 
+        LOGGER.info("Got data for %s players on %s", len(players), update_date)
         return players, update_date
 
     def process_pdf(
         self, url: str, target_page: str, overrides: Dict[str, Dict[str, Any]]
     ):
         players, update_date = self.download_and_parse(url, overrides)
+
+        if len(players) < 100:
+            raise Exception("Got data for suspiciously few players!")
+
         LOGGER.info("Formatting rankings for the required on-wiki format")
 
         per_country: Dict[str, List[Dict[str, Any]]] = defaultdict(list)
