fix(_check_for_duplicate_jobs): don't add current job if None

Caught by mypy 0.940 → 0.960.
freedomofpress · Jun 22, 2022 · ebb5336 · ebb5336
1 parent 8f2761b
commit ebb5336
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/securedrop_client/queue.py b/securedrop_client/queue.py
@@ -92,7 +92,8 @@ def _check_for_duplicate_jobs(self, job: QueueJob) -> bool:
         stored on self.current_job. We check that the job to be added is not among them.
         """
         in_progress_jobs = [in_progress_job for priority, in_progress_job in self.queue.queue]
-        in_progress_jobs.append(self.current_job)
+        if self.current_job is not None:
+            in_progress_jobs.append(self.current_job)
         if job in in_progress_jobs:
             logger.debug("Duplicate job {}, skipping".format(job))
             return True