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

{velocity_python-0.1.36/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.36
+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.36 → velocity_python-0.1.37}/pyproject.toml

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

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

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

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

@@ -23,6 +23,13 @@ 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
@@ -47,6 +54,16 @@ def _to_jsonable(value: Any, _stack: Optional[Set[int]] = None) -> Any:
             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__"):
         _stack.add(value_id)
         try:
@@ -81,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.36 → 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.36
+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.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

velocity_python-0.1.36/tests/test_payment_braintree_mirror.py

@@ -1,50 +0,0 @@
-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 FakeTx:
-    def __init__(self):
-        self._table = FakeTable()
-
-    def table(self, table_name):
-        assert table_name == "braintree_transaction"
-        return self._table
-
-
-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
-
-
-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