Arkivum checksum bug: fixes to docstrings and comments

artefactual · Jul 20, 2017 · f0531cc · f0531cc
1 parent 27209c6
commit f0531cc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/storage_service/locations/models/arkivum.py b/storage_service/locations/models/arkivum.py
@@ -109,8 +109,7 @@ def post_move_from_storage_service(self, staging_path, destination_path, package
             url = 'https://' + self.host + '/api/2/files/release/' + relative_path
             headers = {'Content-Type': 'application/json'}
 
-            # Get size, checksum, and checksum algorithm from pointer file;
-            # infer compression algorithm from filename.
+            # Get size, checksum, and checksum algorithm from pointer file.
             root = etree.parse(package.full_pointer_file_path)
             fixity_elem = root.find('.//premis:fixity', namespaces=utils.NSMAP)
             algorithm = fixity_elem.findtext(

diff --git a/storage_service/locations/tests/test_arkivum.py b/storage_service/locations/tests/test_arkivum.py
@@ -13,6 +13,11 @@
 
 
 def get_pkg_uuid_path(package_uuid):
+    """Convert a UUID string to a string representation of a directory strucure
+    generated from it, e.g., 
+    input: ``'b9a07040-4f08-4a9c-b88a-5cc325e17e7e'``
+    output: ``'b9a0/7040/4f08/4a9c/b88a/5cc3/25e1/7e7e'``
+    """
     tmp = package_uuid.replace('-', '')
     return os.path.join(*[tmp[i:i + 4] for i in range(0, len(tmp), 4)])