Fix bug in project deletion code

Author: laurentS

mapswipe_workers/mapswipe_workers/firebase_to_postgres/delete_project.py

@@ -22,6 +22,12 @@ def delete_project(project_ids: list) -> bool:
     Deletes project, groups, tasks and results from Firebase and Postgres.
     """
     for project_id in project_ids:
+        if project_id is None:
+            logger.info(
+                f"Will not delete Null project_id."
+            )
+            continue
+
         logger.info(
             f"Delete project, groups, tasks and results of project: {project_id}"
         )

mapswipe_workers/tests/integration/test_delete_project.py

@@ -121,6 +121,14 @@ def test_project_id_not_exists(self):
         self.verify_firebase_not_empty()
         self.verify_postgres_not_empty()
 
+    def test_project_id_empty(self):
+        """Test deletion of empty project_id list."""
+        delete_project.delete_project([])
+        time.sleep(5)  # Wait for Firebase Functions to complete
+
+        self.verify_firebase_not_empty()
+        self.verify_postgres_not_empty()
+
     def test_project_id_equals_none(self):
         """Test for project id which does not exists."""
         delete_project.delete_project([None])