velocity-python 0.1.99__tar.gz → 0.1.100__tar.gz

{velocity_python-0.1.99 → velocity_python-0.1.100}/PKG-INFO

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

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

{velocity_python-0.1.99 → velocity_python-0.1.100}/src/velocity/__init__.py

@@ -1,4 +1,4 @@
-__version__ = version = "0.1.99"
+__version__ = version = "0.1.100"
 
 import importlib as _importlib
 

{velocity_python-0.1.99 → velocity_python-0.1.100}/src/velocity/aws/handlers/mixins/data_service.py

@@ -442,6 +442,20 @@ class DataServiceMixin:
         if swallowed_error:
             tx._last_return_default_error = None
 
+        # A missing table under UNLOCKED auto-schema is a legitimate empty
+        # state, not an error: velocity creates tables on first write, so a
+        # table nothing has written to yet (user_data on a fresh site / for a
+        # new user) is simply empty — answer quietly instead of toasting a
+        # scary "relation does not exist" at the user. Under a LOCKED schema
+        # (production) tables exist only via ensure_schema, so a missing one
+        # is a real deploy bug and stays loud.
+        if (
+            swallowed_error
+            and swallowed_error.get("type") == "DbTableMissingError"
+            and not tx.engine.schema_locked
+        ):
+            swallowed_error = None
+
         _file_export_formats = {
             "excel",
             "csv",

{velocity_python-0.1.99 → velocity_python-0.1.100}/src/velocity_python.egg-info/PKG-INFO

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

@@ -239,6 +239,7 @@ tests/test_pdf.py
 tests/test_prepared_statements.py
 tests/test_psycopg3_upgrade.py
 tests/test_query_cache.py
+tests/test_query_missing_table_unlocked.py
 tests/test_restricted_direct_tables.py
 tests/test_retry_side_effect_guard.py
 tests/test_return_default_safety.py

velocity_python-0.1.100/tests/test_query_missing_table_unlocked.py

@@ -0,0 +1,98 @@
+"""A missing table under UNLOCKED auto-schema is empty, not an error.
+
+velocity creates tables on first write, so a table nothing has written to
+yet (``user_data`` on a fresh factory site / for a brand-new user) is
+semantically an empty table. The query path must answer with empty rows and
+NO error toast in that mode — the 2026-09-01 dance.10sp.us first sign-in
+toasted 'relation "user_data" does not exist' at the operator. Under a
+LOCKED schema (production), tables exist only via ensure_schema, so a
+missing one is a real deploy bug and must stay loud.
+"""
+
+import unittest
+from unittest.mock import MagicMock
+
+from velocity.aws.handlers.mixins.data_service import DataServiceMixin
+
+
+class _Service(DataServiceMixin):
+    pass
+
+
+def _harness(schema_locked):
+    service = _Service()
+    service._call_rwx_hook = MagicMock()
+
+    result = MagicMock()
+    result.all.return_value = []
+    result.sql = None
+    result.headers = []
+
+    table = MagicMock()
+    table.select.return_value = result
+
+    tx = MagicMock()
+    tx.table.return_value = table
+    tx.engine.schema_locked = schema_locked
+    # what @return_default records when the select swallowed 42P01
+    tx._last_return_default_error = None
+
+    def select_with_swallow(**params):
+        tx._last_return_default_error = {
+            "type": "DbTableMissingError",
+            "message": 'relation "user_data" does not exist',
+            "function": "velocity.db.core.table.Table.select",
+        }
+        return result
+
+    table.select.side_effect = select_with_swallow
+
+    context = MagicMock()
+    return service, tx, context
+
+
+class MissingTableUnlockedTest(unittest.TestCase):
+    def test_unlocked_missing_table_is_quiet_empty(self):
+        service, tx, context = _harness(schema_locked=False)
+        data = service.query_hook(
+            tx, "user_data", {"obj": "user_data", "params": {}}, context
+        )
+        self.assertEqual(data["rows"], [])
+        self.assertNotIn("error", data["config"])
+        context.response().toast.assert_not_called()
+
+    def test_locked_missing_table_stays_loud(self):
+        service, tx, context = _harness(schema_locked=True)
+        data = service.query_hook(
+            tx, "user_data", {"obj": "user_data", "params": {}}, context
+        )
+        self.assertEqual(data["rows"], [])
+        self.assertEqual(data["config"]["error"]["type"], "DbTableMissingError")
+        context.response().toast.assert_called()
+
+    def test_other_swallowed_errors_still_surface_when_unlocked(self):
+        service, tx, context = _harness(schema_locked=False)
+        tx.table.return_value.select.side_effect = None
+
+        def swallow_column_error(**params):
+            tx._last_return_default_error = {
+                "type": "DbColumnMissingError",
+                "message": 'column "bogus" does not exist',
+                "function": "velocity.db.core.table.Table.select",
+            }
+            result = MagicMock()
+            result.all.return_value = []
+            result.sql = None
+            result.headers = []
+            return result
+
+        tx.table.return_value.select.side_effect = swallow_column_error
+        data = service.query_hook(
+            tx, "user_data", {"obj": "user_data", "params": {}}, context
+        )
+        self.assertEqual(data["config"]["error"]["type"], "DbColumnMissingError")
+        context.response().toast.assert_called()
+
+
+if __name__ == "__main__":
+    unittest.main()