Use generated hash instead of hard coded one.

Author: jverswijver

datajoint/external.py

@@ -347,9 +347,8 @@ def delete(self, *, delete_external_files=None, limit=None, display_progress=Tru
                     try:
                         self._remove_external_file(external_path)
                     except Exception as error:
-                        if errors_as_string:
-                            error = str(error)
-                        error_list.append((uuid, external_path, error))
+                        error_list.append((uuid, external_path,
+                                           str(error) if errors_as_string else error))
             return error_list
 
 

tests/test_s3.py

@@ -7,6 +7,8 @@
 from .schema_external import schema, SimpleRemote
 from datajoint.errors import DataJointError
 import os
+from datajoint.hash import uuid_from_buffer
+from datajoint.blob import pack
 
 
 class TestS3:
@@ -126,12 +128,12 @@ def test_remove_object_exception():
                                                      errors_as_string=False)
 
         # Teardown
-        print(error_list[0][0])
         os.system('mc admin policy remove myminio test')
         os.system('mc admin user remove myminio jeffjeff')
         schema.external['share'].s3.client = old_client
         schema.external['share'].delete(delete_external_files=True)
         os.remove("/tmp/policy.json")
+
         # Raise the error we want if the error matches the expected uuid
-        if str(error_list[0][0]) == '4c16e9e1-5b64-474e-4b56-ba22bb17faae':
+        if str(error_list[0][0]) == str(uuid_from_buffer(pack(test[1]))):
             raise error_list[0][2]