Cosmetic fixes to make the tests pass

openqabot/loader/repohash.py

@@ -21,7 +21,7 @@ def get_max_revision(
 ) -> int:
     max_rev = 0
 
-    url_base = f"http://download.suse.de/ibs/{project.replace(':',':/')}"
+    url_base = f"http://download.suse.de/ibs/{project.replace(':', ':/')}"
 
     for repo in repos:
         # openSUSE and SLE incidents have different handling of architecture

openqabot/types/incident.py

@@ -158,12 +158,9 @@ def has_failures(self, token) -> bool:
         if failed_jobs:
             log.info("Found %s failed jobs for incident %s:", len(failed_jobs), self.id)
             if log.isEnabledFor(LOG_DEBUG_LEVEL):
-                log_debug = lambda job_id: log.debug(
-                    "Job %s is not marked as acceptable for incident %s",
-                    job_id,
-                    self.id,
+                list(
+                    map(lambda job: self.log_debug_incident(job["job_id"]), failed_jobs)
                 )
-                [log_debug(res["job_id"]) for res in failed_jobs]
 
         if not results:
             raise NoResultsError(
@@ -172,3 +169,10 @@ def has_failures(self, token) -> bool:
 
         # we only need to check if there are any failed jobs
         return any(failed_jobs)
+
+    def log_debug_incident(self, job_id):
+        log.debug(
+            "Job %s is not marked as acceptable for incident %s",
+            job_id,
+            self.id,
+        )

openqabot/types/incidents.py

@@ -76,6 +76,7 @@ def _is_scheduled_job(
 
         return False
 
+    # pylint: disable=too-many-return-statements
     def _handle_incident(
         self,
         inc: Incident,