added to __repr__ in pure py version the module name, consistently wi…

…th C extension

frozendict/core.py

@@ -60,7 +60,12 @@ def __repr__(self, *args, **kwargs):
         """
 
         body = super().__repr__(*args, **kwargs)
-        name = self.__class__.__name__
+        klass = self.__class__
+
+        if klass == frozendict or klass == coold:
+            name = f"frozendict.{klass.__name__}"
+        else:
+            name = klass.__name__
 
         return f"{name}({body})"
 

test/common.py

@@ -60,6 +60,18 @@ def fd_items(fd_dict):
 def fd_empty():
     return frozendict_class()
 
+@pytest.fixture
+def module_prefix():
+    try:
+        sc = subclass
+    except NameError:
+        sc = False
+
+    if sc:
+        return ""
+
+    return "frozendict."
+
 ##############################################################################
 # main tests
 
@@ -151,19 +163,22 @@ def test_items(fd, fd_dict):
 def test_fromkeys(fd, fd_giulia):
     assert frozendict_class.fromkeys(["Marco", "Giulia"], "Sulla") == fd_giulia
 
-def test_repr(fd, fd_dict):
+def test_repr(fd, fd_dict, module_prefix):
     classname = frozendict_class.__name__
     repr_fd = repr(fd)
-    expected_repr = f"{classname}({repr(fd_dict)})"
+    prefix = module_prefix
+    expected_repr = f"{prefix}{classname}({repr(fd_dict)})"
     assert repr_fd == expected_repr
 
-def test_str(fd, fd_dict):
+def test_str(fd, fd_dict, module_prefix):
     classname = frozendict_class.__name__
-    assert str(fd) == f"{classname}({repr(fd_dict)})"
+    prefix = module_prefix
+    assert str(fd) == f"{prefix}{classname}({repr(fd_dict)})"
 
-def test_format(fd, fd_dict):
+def test_format(fd, fd_dict, module_prefix):
     classname = frozendict_class.__name__
-    assert format(fd) == f"{classname}({repr(fd_dict)})"
+    prefix = module_prefix
+    assert format(fd) == f"{prefix}{classname}({repr(fd_dict)})"
 
 def test_iter(fd):
     items = []

test/subclass_only.py

@@ -1,3 +1,5 @@
+subclass = True
+
 def test_empty_sub(fd_empty):
     assert fd_empty == frozendict_class({}) == frozendict_class([]) == frozendict_class({}, **{})
 

test/test_coold_subclass.py

@@ -12,14 +12,14 @@ def __new__(cls, *args, **kwargs):
 curr_path = Path(__file__)
 curr_dir = curr_path.parent
 
+with open(curr_dir / "subclass_only.py") as f:
+    subclass_only_code = f.read()
+
+exec(subclass_only_code)
+
 common_path = curr_dir / "common.py"
 
 with open(common_path) as f:
     common_code = f.read()
 
 exec(common_code)
-
-with open(curr_dir / "subclass_only.py") as f:
-    subclass_only_code = f.read()
-
-exec(subclass_only_code)

test/test_frozendict_c_subclass.py

@@ -12,14 +12,14 @@ def __new__(cls, *args, **kwargs):
 curr_path = Path(__file__)
 curr_dir = curr_path.parent
 
+with open(curr_dir / "subclass_only.py") as f:
+    subclass_only_code = f.read()
+
+exec(subclass_only_code)
+
 common_path = curr_dir / "common.py"
 
 with open(common_path) as f:
     common_code = f.read()
 
 exec(common_code)
-
-with open(curr_dir / "subclass_only.py") as f:
-    subclass_only_code = f.read()
-
-exec(subclass_only_code)

test/test_frozendict_subclass.py

@@ -14,12 +14,12 @@ def __new__(cls, *args, **kwargs):
 
 common_path = curr_dir / "common.py"
 
-with open(common_path) as f:
-    common_code = f.read()
-
-exec(common_code)
-
 with open(curr_dir / "subclass_only.py") as f:
     subclass_only_code = f.read()
 
 exec(subclass_only_code)
+
+with open(common_path) as f:
+    common_code = f.read()
+
+exec(common_code)