localstack-snapshot 0.3.2__tar.gz → 0.3.3__tar.gz

{localstack_snapshot-0.3.2 → localstack_snapshot-0.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: localstack-snapshot
-Version: 0.3.2
+Version: 0.3.3
 Summary: Extracted snapshot testing lib for LocalStack
 Author-email: LocalStack Contributors <info@localstack.cloud>
 License: 

localstack_snapshot-0.3.3/localstack_snapshot/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.3.3"

{localstack_snapshot-0.3.2 → localstack_snapshot-0.3.3}/localstack_snapshot/snapshots/prototype.py

@@ -183,14 +183,21 @@ class SnapshotSession:
                 return obj_.value
             elif hasattr(obj_, "__dict__"):
                 # This is an object - let's try to convert it to a dictionary
-                # A  naive approach would be to use the '__dict__' object directly, but that only lists the attributes
+                # A naive approach would be to use the '__dict__' object directly, but that only lists the attributes.
                 # In order to also serialize the properties, we use the __dir__() method
                 # Filtering by everything that is not a method gives us both attributes and properties
-                # We also (still) skip private attributes/properties, so everything that starts with an underscore
+                # We also (still) skip private attributes/properties, so everything that starts with an underscore.
                 return {
                     k: _convert_object_to_dict(getattr(obj_, k))
                     for k in obj_.__dir__()
-                    if not k.startswith("_") and type(getattr(obj_, k, "")).__name__ != "method"
+                    if (
+                        # Skip private attributes
+                        not k.startswith("_")
+                        # Skip everything that's not a method
+                        and type(getattr(obj_, k, "")).__name__ != "method"
+                        # Skip everything that refers to itself (identity functions), as that leads to recursion
+                        and getattr(obj_, k) != obj_
+                    )
                 }
             return obj_
 

{localstack_snapshot-0.3.2 → localstack_snapshot-0.3.3}/localstack_snapshot.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: localstack-snapshot
-Version: 0.3.2
+Version: 0.3.3
 Summary: Extracted snapshot testing lib for LocalStack
 Author-email: LocalStack Contributors <info@localstack.cloud>
 License: 

{localstack_snapshot-0.3.2 → localstack_snapshot-0.3.3}/pyproject.toml

@@ -7,7 +7,7 @@ name = "localstack-snapshot"
 authors = [
     { name = "LocalStack Contributors", email = "info@localstack.cloud" }
 ]
-version = "0.3.2"
+version = "0.3.3"
 description = "Extracted snapshot testing lib for LocalStack"
 dependencies = [
     "jsonpath-ng>1.6",

{localstack_snapshot-0.3.2 → localstack_snapshot-0.3.3}/tests/test_snapshots.py

@@ -137,6 +137,21 @@ class TestSnapshotManager:
         sm.match_object("key_a", CustomObject(name="myname"))
         sm._assert_all()
 
+    def test_match_object_with_identity_function(self):
+        class CustomObject:
+            def __init__(self, name):
+                self.name = name
+
+            @property
+            def me_myself_and_i(self):
+                # This would lead to a RecursionError, so we cannot snapshot this method
+                return self
+
+        sm = SnapshotSession(scope_key="A", verify=True, base_file_path="", update=False)
+        sm.recorded_state = {"key_a": {"name": "myname"}}
+        sm.match_object("key_a", CustomObject(name="myname"))
+        sm._assert_all()
+
     def test_match_object_change(self):
         class CustomObject:
             def __init__(self, name):

localstack_snapshot-0.3.2/localstack_snapshot/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.3.2"