Remove spurious colon on hash strings

Author: coldino

src/obelisk/filetypes/binary.py

@@ -16,7 +16,7 @@ def get_metadata_from_binary(file_path: Path) -> ManifestEntry | None:
     file_size = file_path.stat().st_size
 
     # Construct the manifest entry
-    hash_str = f'md5:{hasher.hexdigest()}:{file_size}:'
+    hash_str = f'md5:{hasher.hexdigest()}:{file_size}'
     return ManifestEntry(
         filename=file_path.name,
         hash=hash_str,

tests/unit/filetypes/test_binary.py

@@ -13,7 +13,7 @@
 def _expected_hash(data: bytes) -> str:
     h = hashlib.md5()  # noqa: S324 - not using for cryptography in any way
     h.update(data)
-    return f'md5:{h.hexdigest()}:{len(data)}:'
+    return f'md5:{h.hexdigest()}:{len(data)}'
 
 
 def test_get_metadata_from_binary_empty_file(tmp_path: Path) -> None:

tests/unit/test_scanner.py

@@ -16,7 +16,7 @@
 def _expected_hash(data: bytes) -> str:
     h = hashlib.md5()  # noqa: S324 - not for cryptography
     h.update(data)
-    return f'md5:{h.hexdigest()}:{len(data)}:'
+    return f'md5:{h.hexdigest()}:{len(data)}'
 
 
 def test_create_manifest_from_folder_basic_and_skips(tmp_path: Path) -> None: