added a couple of hashing tests

Author: tclose

pydra/utils/hash.py

@@ -72,7 +72,6 @@ def hash_object(obj: object) -> Hash:
     try:
         return hash_single(obj, Cache({}))
     except Exception as e:
-        hash_single(obj, Cache({}))
         raise UnhashableError(f"Cannot hash object {obj!r}") from e
 
 
@@ -227,9 +226,8 @@ def bytes_repr_dict(obj: dict, cache: Cache) -> Iterator[bytes]:
 
 @register_serializer(_SpecialForm)
 @register_serializer(type)
-def bytes_repr_type(obj: type, cache: Cache) -> Iterator[bytes]:
-    cls = type(obj)
-    yield f"{cls.__module__}.{cls.__name__}".encode()
+def bytes_repr_type(klass: type, cache: Cache) -> Iterator[bytes]:
+    yield f"type:({klass.__module__}.{klass.__name__})".encode()
 
 
 @register_serializer(list)

pydra/utils/tests/test_hash.py

@@ -2,6 +2,7 @@
 from hashlib import blake2b
 from pathlib import Path
 
+import attrs
 import pytest
 
 from ..hash import Cache, UnhashableError, bytes_repr, hash_object, register_serializer
@@ -133,6 +134,20 @@ def __init__(self, x):
     assert re.match(rb".*\.MyClass:{str:1:x=.{16}}", obj_repr)
 
 
+def test_bytes_repr_attrs_slots():
+    @attrs.define
+    class MyClass:
+        x: int
+
+    obj_repr = join_bytes_repr(MyClass(1))
+    assert re.match(rb".*\.MyClass:{str:1:x=.{16}}", obj_repr)
+
+
+def test_bytes_repr_type():
+    obj_repr = join_bytes_repr(Path)
+    assert obj_repr == b"type:(pathlib.Path)"
+
+
 def test_recursive_object():
     a = []
     b = [a]