add logs to delete_file paths

bloomberg · Oct 17, 2024 · 23912ab · 23912ab
1 parent f220e42
commit 23912ab
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 1 deletion.
diff --git a/pybossa/repositories/task_repository.py b/pybossa/repositories/task_repository.py
@@ -497,6 +497,7 @@ def _delete_zip_files_from_store(self, project):
         json_taskruns_filename = json_exporter.download_name(project, 'task_run')
         csv_taskruns_filename = csv_exporter.download_name(project, 'task_run')
         container = "user_%s" % project.owner_id
+        current_app.logger.info("delete_zip_files_from_store. project %d container %s. delete files %s, %s, %s, %s", project.id, container, json_tasks_filename, csv_tasks_filename, json_taskruns_filename, csv_taskruns_filename)
         uploader.delete_file(json_tasks_filename, container)
         uploader.delete_file(csv_tasks_filename, container)
         uploader.delete_file(json_taskruns_filename, container)

diff --git a/pybossa/uploader/cloud_store.py b/pybossa/uploader/cloud_store.py
@@ -55,7 +55,9 @@ def _upload_file(self, file, container): # pragma: no cover
 
     def delete_file(self, name, container):  # pragma: no cover
         try:
+            from flask import current_app
             key = self.key_name(container, name)
+            current_app.logger.info("delete_file cloud_store delete. container %s, name %s, key %s", container, name, key)
             self.bucket.delete_key(key)
             return True
         except Exception:

diff --git a/pybossa/uploader/local.py b/pybossa/uploader/local.py
@@ -69,7 +69,9 @@ def _upload_file(self, file, container):
     def delete_file(self, filename, container):
         """Delete file from filesystem."""
         try:
+            from flask import current_app
             path = self.get_file_path(container, filename)
+            current_app.logger.info("delete_file local delete. container %s, filename %s, path %s", container, filename, path)
             os.remove(path)
             return True
         except Exception:

diff --git a/test/test_uploader/test_local_uploader.py b/test/test_uploader/test_local_uploader.py
@@ -19,7 +19,7 @@
 
 import os
 import tempfile
-from test import Test
+from test import Test, with_context
 from pybossa.uploader.local import LocalUploader
 from unittest.mock import patch
 from werkzeug.datastructures import FileStorage
@@ -130,6 +130,7 @@ def test_local_folder_is_created(self, mock):
         err_msg = "This local path should exist: %s" % path
         assert os.path.isdir(path) is True, err_msg
 
+    @with_context
     @patch('os.remove', return_value=None)
     def test_local_folder_delete(self, mock):
         """Test LOCAL UPLOADER delete works."""