velocity-python 0.1.70__tar.gz → 0.1.72__tar.gz

{velocity_python-0.1.70 → velocity_python-0.1.72}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.1.70
+Version: 0.1.72
 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.70 → velocity_python-0.1.72}/pyproject.toml

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

{velocity_python-0.1.70 → velocity_python-0.1.72}/src/velocity/__init__.py

@@ -1,4 +1,4 @@
-__version__ = version = "0.1.70"
+__version__ = version = "0.1.72"
 
 import importlib as _importlib
 

{velocity_python-0.1.70 → velocity_python-0.1.72}/src/velocity/aws/handlers/mixins/web_handler.py

@@ -478,6 +478,26 @@ Request Details:
             }
         )
 
+    def OnActionStoreUserData(self, tx, context):
+        """Persist per-user UI settings posted by omnilib's user-settings store.
+
+        Generic counterpart to :meth:`OnActionGetUserData`. omnilib sends
+        ``{"userData": {keyPath: fragment, ...}}``; each entry is upserted into
+        the per-user settings table keyed by ``(userid, key)``. Business-agnostic
+        — every app shares the same store, so this lives here rather than being
+        re-implemented per app. App handlers may override to add side effects.
+        """
+        userdata = (context.payload() or {}).get("userData")
+        if not userdata:
+            return
+        email = (context.session() or {}).get("email_address")
+        table = getattr(context, "USER_DATA_TABLE", "user_data")
+        for key, val in userdata.items():
+            tx.table(table).upsert(
+                {"val": json.dumps(val), "sys_modified_by": email},
+                {"userid": email, "key": key},
+            )
+
     # ------------------------------------------------------------------
     # Enhanced lifecycle hooks
     # ------------------------------------------------------------------

{velocity_python-0.1.70 → velocity_python-0.1.72}/src/velocity/db/core/column.py

@@ -15,6 +15,17 @@ class Column:
         self.name = name
         self.table = table
 
+    def cursor(self):
+        """
+        Return the active cursor for this column's transaction.
+
+        Required by the ``@return_default`` decorator, which brackets the wrapped
+        call in a savepoint (``create``/``rollback``/``release``) on ``self.cursor()``.
+        Delegates to the owning table so the savepoint shares the table's cursor —
+        the same one ``self.tx.execute`` runs the statement on.
+        """
+        return self.table.cursor()
+
     def __str__(self):
         return (
             f"Table: {self.table.name}\n"
@@ -113,6 +124,7 @@ class Column:
         Returns the distinct values in this column, optionally ordered and/or limited in quantity.
         """
         sql, vals = self.sql.select(
+            self.tx,
             columns=f"distinct {self.name}",
             table=self.table.name,
             orderby=f"{self.name} {order}",
@@ -127,7 +139,7 @@ class Column:
         try:
             qcol = self.sql.quote_identifier(self.name)
             sql, vals = self.sql.select(
-                columns=f"max({qcol})", table=self.table.name, where=where
+                self.tx, columns=f"max({qcol})", table=self.table.name, where=where
             )
             return self.tx.execute(sql, vals).scalar()
         except (exceptions.DbTableMissingError, exceptions.DbColumnMissingError):

{velocity_python-0.1.70 → velocity_python-0.1.72}/src/velocity_python.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: velocity-python
-Version: 0.1.70
+Version: 0.1.72
 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.70 → velocity_python-0.1.72}/src/velocity_python.egg-info/SOURCES.txt

@@ -156,6 +156,7 @@ tests/test_asset_references.py
 tests/test_assets_service.py
 tests/test_async_support.py
 tests/test_batch_operations.py
+tests/test_column_tx_arg.py
 tests/test_concurrency_safety.py
 tests/test_connection_pool.py
 tests/test_connection_resilience.py
@@ -199,6 +200,7 @@ tests/test_spreadsheet_functions.py
 tests/test_sqlite_backend.py
 tests/test_sqs_per_record_transactions.py
 tests/test_ssm_config.py
+tests/test_store_user_data.py
 tests/test_sys_modified_count_postgres_demo.py
 tests/test_table_alter.py
 tests/test_transaction_class_wrapping.py

velocity_python-0.1.72/tests/test_column_tx_arg.py

@@ -0,0 +1,64 @@
+"""
+Regression tests for Column query methods staying in sync with the SQL builder.
+
+Two bugs these guard against:
+- ``Column`` lacked a ``cursor()`` method, so the ``@return_default`` decorator
+  (which brackets the call in a savepoint on ``self.cursor()``) raised
+  ``AttributeError: 'Column' object has no attribute 'cursor'``.
+- ``Column.distinct``/``Column.max`` called ``self.sql.select(columns=..., ...)``
+  without the leading ``tx`` positional the current ``SQL.select(tx, ...)``
+  signature requires, raising ``TypeError: ... missing 1 required positional
+  argument: 'tx'``.
+"""
+
+import unittest
+from unittest.mock import MagicMock
+
+
+def _make_column(name="class"):
+    from velocity.db.core.column import Column
+
+    tx = MagicMock()
+    tx.create_savepoint.return_value = "sp1"
+    result = MagicMock()
+    result.as_simple_list.return_value.all.return_value = ["a", "b"]
+    result.scalar.return_value = 5
+    tx.execute.return_value = result
+
+    sql = MagicMock()
+    sql.select.return_value = ("SELECT ...", [])
+    tx.engine.sql = sql
+
+    table = MagicMock()
+    table.tx = tx
+    table.name = "contact"
+    table.cursor.return_value = "CUR"
+
+    return Column(table, name), tx, sql, table
+
+
+class TestColumnCursorDelegation(unittest.TestCase):
+    def test_cursor_delegates_to_table(self):
+        col, _tx, _sql, table = _make_column()
+        self.assertEqual(col.cursor(), "CUR")
+        table.cursor.assert_called()
+
+
+class TestColumnSelectPassesTx(unittest.TestCase):
+    def test_distinct_passes_tx_first_and_returns_rows(self):
+        col, tx, sql, _table = _make_column()
+        result = col.distinct()
+        self.assertEqual(result, ["a", "b"])
+        args, _kwargs = sql.select.call_args
+        self.assertIs(args[0], tx, "distinct() must pass tx as the first positional arg")
+
+    def test_max_passes_tx_first_and_returns_scalar(self):
+        col, tx, sql, _table = _make_column()
+        result = col.max()
+        self.assertEqual(result, 5)
+        args, _kwargs = sql.select.call_args
+        self.assertIs(args[0], tx, "max() must pass tx as the first positional arg")
+
+
+if __name__ == "__main__":
+    unittest.main()

velocity_python-0.1.72/tests/test_store_user_data.py

@@ -0,0 +1,52 @@
+"""WebHandler.OnActionStoreUserData — centralized per-user settings persistence.
+
+Generic counterpart to OnActionGetUserData (already centralized). omnilib posts
+``{"userData": {keyPath: fragment, ...}}`` and each fragment is upserted into the
+USER_DATA_TABLE keyed by (userid, key). Lives in velocity so every app shares it
+instead of re-implementing it per handler.
+"""
+
+import json
+import unittest
+from unittest.mock import MagicMock
+
+from velocity.aws.handlers.mixins.web_handler import WebHandler
+
+
+def _ctx(payload, email="user@example.com"):
+    ctx = MagicMock()
+    ctx.payload.return_value = payload
+    ctx.session.return_value = {"email_address": email}
+    ctx.USER_DATA_TABLE = "user_data"
+    return ctx
+
+
+class StoreUserDataTest(unittest.TestCase):
+    def test_upserts_each_fragment_keyed_by_userid_and_key(self):
+        tx = MagicMock()
+        ctx = _ctx({"userData": {"grid.layout": {"w": 1}, "theme": "dark"}})
+
+        WebHandler.OnActionStoreUserData(MagicMock(), tx, ctx)
+
+        tx.table.assert_called_with("user_data")
+        calls = tx.table.return_value.upsert.call_args_list
+        self.assertEqual(len(calls), 2)
+
+        seen = {}
+        for data, pk in (c.args for c in calls):
+            seen[pk["key"]] = (
+                json.loads(data["val"]),
+                pk["userid"],
+                data["sys_modified_by"],
+            )
+        self.assertEqual(seen["grid.layout"], ({"w": 1}, "user@example.com", "user@example.com"))
+        self.assertEqual(seen["theme"], ("dark", "user@example.com", "user@example.com"))
+
+    def test_noop_when_no_user_data(self):
+        tx = MagicMock()
+        WebHandler.OnActionStoreUserData(MagicMock(), tx, _ctx({}))
+        tx.table.assert_not_called()
+
+
+if __name__ == "__main__":
+    unittest.main()