velocity-python 0.1.26__tar.gz → 0.1.27__tar.gz

{velocity_python-0.1.26/src/velocity_python.egg-info → velocity_python-0.1.27}/PKG-INFO

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

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

{velocity_python-0.1.26 → velocity_python-0.1.27}/src/velocity/__init__.py

@@ -1,4 +1,4 @@
-__version__ = version = "0.1.26"
+__version__ = version = "0.1.27"
 
 import importlib as _importlib
 

{velocity_python-0.1.26 → velocity_python-0.1.27}/src/velocity/aws/assets/indexing.py

@@ -76,7 +76,11 @@ def reconcile_form_builder_template_asset_usages(
 
 
 def load_asset_usages(tx, *, source_table: str, source_sys_id: str) -> list[AssetUsage]:
-    rows = tx.table("asset_usage_index").select(
+    usage_index = tx.table("asset_usage_index")
+    if not usage_index.exists():
+        return []
+
+    rows = usage_index.select(
         where={
             "source_table": source_table,
             "source_sys_id": source_sys_id,
@@ -87,6 +91,9 @@ def load_asset_usages(tx, *, source_table: str, source_sys_id: str) -> list[Asse
 
 def apply_asset_usage_reconciliation(tx, reconciliation: AssetUsageReconciliation) -> None:
     table = tx.table("asset_usage_index")
+    if not table.exists():
+        return
+
     for usage in reconciliation.to_create + reconciliation.to_refresh:
         row = _asset_usage_to_row(usage)
         table.upsert(
@@ -115,13 +122,16 @@ def resolve_asset_id_for_reference(tx, reference: AssetReference) -> str | None:
         return reference.asset_id
 
     images = tx.table("images")
+    if not images.exists():
+        return None
+
     for candidate_key in _candidate_storage_keys(reference.storage_key):
         row = images.find({"key": candidate_key})
         asset_id = _extract_sys_id(row)
         if asset_id:
             return asset_id
 
-    if reference.url:
+    if reference.url and images.column("url").exists():
         row = images.find({"url": reference.url})
         asset_id = _extract_sys_id(row)
         if asset_id:

{velocity_python-0.1.26 → velocity_python-0.1.27/src/velocity_python.egg-info}/PKG-INFO

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

@@ -145,6 +145,7 @@ src/velocity_python.egg-info/entry_points.txt
 src/velocity_python.egg-info/requires.txt
 src/velocity_python.egg-info/top_level.txt
 tests/test_amplify_build.py
+tests/test_asset_indexing.py
 tests/test_asset_references.py
 tests/test_assets_service.py
 tests/test_async_support.py

velocity_python-0.1.27/tests/test_asset_indexing.py

@@ -0,0 +1,103 @@
+from velocity.aws.assets.indexing import load_asset_usages, resolve_asset_id_for_reference
+from velocity.aws.assets.references import AssetReference
+
+
+class FakeRow:
+    def __init__(self, data):
+        self._data = dict(data)
+
+    def to_dict(self):
+        return dict(self._data)
+
+
+class FakeResult:
+    def __init__(self, rows):
+        self._rows = [dict(row) for row in rows]
+
+    def as_dict(self):
+        return self
+
+    def all(self):
+        return [dict(row) for row in self._rows]
+
+
+class FakeImagesTable:
+    def __init__(self):
+        self.calls = []
+        self._columns = {"key"}
+
+    def exists(self):
+        return True
+
+    def column(self, name):
+        table = self
+
+        class FakeColumn:
+            def exists(self_nonlocal):
+                return name in table._columns
+
+        return FakeColumn()
+
+    def find(self, where):
+        self.calls.append(dict(where))
+        if where == {"key": "folder/banner.png"}:
+            return FakeRow({"sys_id": "img-123"})
+        if where == {"url": "https://example.com/banner.png"}:
+            raise AssertionError("url lookup should be skipped when the column is absent")
+        return None
+
+
+class FakeUsageIndexTable:
+    def exists(self):
+        return False
+
+    def select(self, where=None):
+        raise AssertionError("select should be skipped when asset_usage_index is missing")
+
+
+class FakeTx:
+    def __init__(self, *, image_table_exists=True, image_columns=None, usage_index_exists=False):
+        self.images = FakeImagesTable()
+        self.images._exists = image_table_exists
+        self.images._columns = set(image_columns or {"key"})
+        self.usage_index = FakeUsageIndexTable()
+        self.usage_index._exists = usage_index_exists
+
+    def table(self, name):
+        if name == "images":
+            class ImagesTableProxy(FakeImagesTable):
+                pass
+
+            proxy = self.images
+            proxy.exists = lambda: proxy._exists
+            return proxy
+        if name == "asset_usage_index":
+            proxy = self.usage_index
+            proxy.exists = lambda: proxy._exists
+            return proxy
+        raise AssertionError(f"Unexpected table request: {name}")
+
+
+def test_load_asset_usages_returns_empty_when_usage_table_missing():
+    tx = FakeTx()
+
+    usages = load_asset_usages(tx, source_table="form_builder_template", source_sys_id="2680")
+
+    assert usages == []
+
+
+def test_resolve_asset_id_skips_url_lookup_when_images_url_column_missing():
+    tx = FakeTx(image_columns={"key"})
+    reference = AssetReference(
+        asset_id=None,
+        url="https://example.com/banner.png",
+        storage_key="folder/banner.png",
+        source_field="main_image_url",
+        source_field_type="image-widget",
+        raw_value="https://example.com/banner.png",
+    )
+
+    asset_id = resolve_asset_id_for_reference(tx, reference)
+
+    assert asset_id == "img-123"
+    assert tx.images.calls == [{"key": "folder/banner.png"}]