velocity-python 0.1.35__tar.gz → 0.1.37__tar.gz

{velocity_python-0.1.35/src/velocity_python.egg-info → velocity_python-0.1.37}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.1.35
+Version: 0.1.37
 Summary: A rapid application development library for interfacing with data storage
 Author-email: Velocity Team <info@codeclubs.org>
 License-Expression: MIT

{velocity_python-0.1.35 → velocity_python-0.1.37}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "velocity-python"
-version = "0.1.35"
+version = "0.1.37"
 authors = [
   { name="Velocity Team", email="info@codeclubs.org" },
 ]

{velocity_python-0.1.35 → velocity_python-0.1.37}/src/velocity/__init__.py

@@ -1,4 +1,4 @@
-__version__ = version = "0.1.35"
+__version__ = version = "0.1.37"
 
 import importlib as _importlib
 

{velocity_python-0.1.35 → velocity_python-0.1.37}/src/velocity/payment/braintree_mirror.py

@@ -1,6 +1,6 @@
 import json
 from datetime import datetime, timezone
-from typing import Any, Dict, Optional
+from typing import Any, Dict, Optional, Set
 
 
 SUCCESS_STATUSES = {
@@ -19,19 +19,61 @@ def _get_value(data: Any, key: str, default: Any = None) -> Any:
     return getattr(data, key, default)
 
 
-def _to_jsonable(value: Any) -> Any:
+def _recursive_marker(value: Any) -> str:
+    return f"<recursive:{type(value).__name__}>"
+
+
+def _attribute_getter_fields(value: Any):
+    fields = getattr(value, "_setattrs", None)
+    if not isinstance(fields, (list, tuple, set)):
+        return None
+    return [str(field) for field in fields if not str(field).startswith("_")]
+
+
+def _to_jsonable(value: Any, _stack: Optional[Set[int]] = None) -> Any:
     if value is None or isinstance(value, (str, int, float, bool)):
         return value
+    if _stack is None:
+        _stack = set()
+
+    value_id = id(value)
+    if value_id in _stack:
+        return _recursive_marker(value)
+
     if isinstance(value, dict):
-        return {str(key): _to_jsonable(item) for key, item in value.items()}
+        _stack.add(value_id)
+        try:
+            return {
+                str(key): _to_jsonable(item, _stack) for key, item in value.items()
+            }
+        finally:
+            _stack.remove(value_id)
     if isinstance(value, (list, tuple, set)):
-        return [_to_jsonable(item) for item in value]
+        _stack.add(value_id)
+        try:
+            return [_to_jsonable(item, _stack) for item in value]
+        finally:
+            _stack.remove(value_id)
+    attribute_fields = _attribute_getter_fields(value)
+    if attribute_fields is not None:
+        _stack.add(value_id)
+        try:
+            return {
+                key: _to_jsonable(getattr(value, key, None), _stack)
+                for key in attribute_fields
+            }
+        finally:
+            _stack.remove(value_id)
     if hasattr(value, "__dict__"):
-        return {
-            str(key): _to_jsonable(item)
-            for key, item in vars(value).items()
-            if not str(key).startswith("_")
-        }
+        _stack.add(value_id)
+        try:
+            return {
+                str(key): _to_jsonable(item, _stack)
+                for key, item in vars(value).items()
+                if not str(key).startswith("_")
+            }
+        finally:
+            _stack.remove(value_id)
     return str(value)
 
 
@@ -56,10 +98,25 @@ def _table(tx, table_name: str):
     return table
 
 
+def _filtered_payload_for_table(table, payload: Dict[str, Any]) -> Dict[str, Any]:
+    try:
+        existing_columns = {
+            str(column).lower() for column in (table.sys_columns() or [])
+        }
+    except Exception:
+        return payload
+    if not existing_columns:
+        return payload
+    return {
+        key: value for key, value in payload.items() if str(key).lower() in existing_columns
+    }
+
+
 def _upsert(tx, table_name: str, payload: Dict[str, Any], key: Dict[str, Any]):
     table = _table(tx, table_name)
     if not table:
         return None
+    payload = _filtered_payload_for_table(table, payload)
     table.upsert(payload, key)
     return payload
 

{velocity_python-0.1.35 → velocity_python-0.1.37/src/velocity_python.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.1.35
+Version: 0.1.37
 Summary: A rapid application development library for interfacing with data storage
 Author-email: Velocity Team <info@codeclubs.org>
 License-Expression: MIT

{velocity_python-0.1.35 → velocity_python-0.1.37}/src/velocity_python.egg-info/SOURCES.txt

@@ -167,6 +167,7 @@ tests/test_n_plus_one.py
 tests/test_observability.py
 tests/test_payment_authorizenet_adapter.py
 tests/test_payment_braintree_adapter.py
+tests/test_payment_braintree_mirror.py
 tests/test_payment_profile_sorting.py
 tests/test_payment_stripe_adapter.py
 tests/test_pdf.py

velocity_python-0.1.37/tests/test_payment_braintree_mirror.py

@@ -0,0 +1,130 @@
+import json
+
+from velocity.payment.braintree_mirror import upsert_braintree_transaction_record
+
+
+class FakeTable:
+    def __init__(self):
+        self.upserts = []
+
+    def exists(self):
+        return True
+
+    def upsert(self, payload, key):
+        self.upserts.append((payload, key))
+
+
+class ColumnAwareTable(FakeTable):
+    def __init__(self, columns):
+        super().__init__()
+        self._columns = columns
+
+    def sys_columns(self):
+        return list(self._columns)
+
+
+class FakeTx:
+    def __init__(self):
+        self._table = FakeTable()
+
+    def table(self, table_name):
+        assert table_name == "braintree_transaction"
+        return self._table
+
+
+class ColumnAwareTx(FakeTx):
+    def __init__(self, columns):
+        self._table = ColumnAwareTable(columns)
+
+
+class RecursiveTransaction:
+    def __init__(self):
+        self.id = "txn_recursive"
+        self.status = "submitted_for_settlement"
+        self.amount = "10.50"
+        self.custom_fields = {"client_id": "client_1"}
+        self.credit_card_details = None
+        self.paypal_details = None
+        self.customer_details = None
+        self.billing_details = None
+        self.child = self
+
+
+class GatewayInternal:
+    def __init__(self):
+        self.configuration = {"private_key": "secret"}
+
+
+class AttributeGetterStyleTransaction:
+    def __init__(self):
+        self._setattrs = [
+            "id",
+            "status",
+            "amount",
+            "custom_fields",
+            "credit_card_details",
+            "paypal_details",
+            "customer_details",
+            "billing_details",
+            "child",
+        ]
+        self.id = "txn_gateway_trimmed"
+        self.status = "submitted_for_settlement"
+        self.amount = "42.00"
+        self.custom_fields = {"client_id": "client_2"}
+        self.credit_card_details = None
+        self.paypal_details = None
+        self.customer_details = None
+        self.billing_details = None
+        self.child = self
+        self.gateway = GatewayInternal()
+
+
+def test_upsert_braintree_transaction_record_handles_recursive_objects():
+    tx = FakeTx()
+    transaction = RecursiveTransaction()
+
+    payload = upsert_braintree_transaction_record(tx, transaction)
+
+    assert payload is not None
+    assert payload["braintree_id"] == "txn_recursive"
+    raw_response = json.loads(payload["raw_response"])
+    assert raw_response["id"] == "txn_recursive"
+    assert raw_response["child"] == "<recursive:RecursiveTransaction>"
+    assert tx._table.upserts
+
+
+def test_upsert_braintree_transaction_record_skips_gateway_internals():
+    tx = FakeTx()
+    transaction = AttributeGetterStyleTransaction()
+
+    payload = upsert_braintree_transaction_record(tx, transaction)
+
+    assert payload is not None
+    raw_response = json.loads(payload["raw_response"])
+    assert raw_response["id"] == "txn_gateway_trimmed"
+    assert raw_response["child"] == "<recursive:AttributeGetterStyleTransaction>"
+    assert "gateway" not in raw_response
+    assert tx._table.upserts
+
+
+def test_upsert_braintree_transaction_record_omits_missing_columns():
+    tx = ColumnAwareTx(
+        [
+            "braintree_id",
+            "status",
+            "amount",
+            "amount_attempted",
+            "authorized_amount",
+            "payment_ok",
+        ]
+    )
+    transaction = RecursiveTransaction()
+
+    payload = upsert_braintree_transaction_record(tx, transaction)
+
+    assert payload is not None
+    assert payload["braintree_id"] == "txn_recursive"
+    assert "raw_response" not in payload
+    assert "update_source" not in payload
+    assert tx._table.upserts