altcodepro-polydb-python 2.6.0__tar.gz → 2.6.1__tar.gz

{altcodepro_polydb_python-2.6.0/src/altcodepro_polydb_python.egg-info → altcodepro_polydb_python-2.6.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: altcodepro-polydb-python
-Version: 2.6.0
+Version: 2.6.1
 Summary: Production-ready multi-cloud database abstraction layer with connection pooling, retry logic, and thread safety
 Author: AltCodePro
 Project-URL: Homepage, https://github.com/altcodepro/polydb-python

{altcodepro_polydb_python-2.6.0 → altcodepro_polydb_python-2.6.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "altcodepro-polydb-python"
-version = "2.6.0"
+version = "2.6.1"
 description = "Production-ready multi-cloud database abstraction layer with connection pooling, retry logic, and thread safety"
 readme = { file = "README.md", content-type = "text/markdown" }
 requires-python = ">=3.11"

{altcodepro_polydb_python-2.6.0 → altcodepro_polydb_python-2.6.1/src/altcodepro_polydb_python.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: altcodepro-polydb-python
-Version: 2.6.0
+Version: 2.6.1
 Summary: Production-ready multi-cloud database abstraction layer with connection pooling, retry logic, and thread safety
 Author: AltCodePro
 Project-URL: Homepage, https://github.com/altcodepro/polydb-python

{altcodepro_polydb_python-2.6.0 → altcodepro_polydb_python-2.6.1}/src/altcodepro_polydb_python.egg-info/SOURCES.txt

@@ -100,6 +100,7 @@ tests/test_azure_table_id_property_persisted.py
 tests/test_azure_table_id_return_value_sanitization_mismatch.py
 tests/test_azure_table_key_field_filter.py
 tests/test_azure_table_large_record_patch_roundtrip.py
+tests/test_azure_table_list_filter.py
 tests/test_azure_table_self_heal.py
 tests/test_azure_table_unpack_entity.py
 tests/test_blockchain.py

{altcodepro_polydb_python-2.6.0 → altcodepro_polydb_python-2.6.1}/src/polydb/adapters/AzureTableStorageAdapter.py

@@ -785,22 +785,34 @@ class AzureTableStorageAdapter(NoSQLKVAdapter):
                         self._sanitize_prop_name(orig_k) if orig_k != _MODEL_FIELD else _MODEL_FIELD
                     )
 
-                ev = self._encode_value(orig_v)
-
-                if ev is None:
-                    parts.append(f"{sk} eq null")
-                elif isinstance(ev, bool):
-                    parts.append(f"{sk} eq {str(ev).lower()}")
-                elif isinstance(ev, (int, float)):
-                    parts.append(f"{sk} eq {ev}")
-                elif isinstance(ev, datetime):
-                    iso = ev.isoformat()
-                    if not iso.endswith("Z"):
-                        iso = iso + "Z"
-                    parts.append(f"{sk} eq datetime'{iso}'")
-                else:
-                    sval = str(ev).replace("'", "''")
-                    parts.append(f"{sk} eq '{sval}'")
+                def _eq(prop: str, raw: Any) -> str:
+                    ev = self._encode_value(raw)
+                    if ev is None:
+                        return f"{prop} eq null"
+                    if isinstance(ev, bool):
+                        return f"{prop} eq {str(ev).lower()}"
+                    if isinstance(ev, (int, float)):
+                        return f"{prop} eq {ev}"
+                    if isinstance(ev, datetime):
+                        iso = ev.isoformat()
+                        if not iso.endswith("Z"):
+                            iso = iso + "Z"
+                        return f"{prop} eq datetime'{iso}'"
+                    return f"{prop} eq '{str(ev).replace(chr(39), chr(39) * 2)}'"
+
+                if isinstance(orig_v, (list, tuple, set, frozenset)) and all(
+                    isinstance(x, (str, int, float, bool)) for x in orig_v
+                ):
+                    # A list of plain values means "equals any of them" (the SQL adapters already read it that way). Sent as
+                    # one JSON-encoded string it matched no row at all, so every `field: [a, b]` lookup silently returned [].
+                    if not orig_v:
+                        return []  # IN () matches nothing
+                    if len(orig_v) > 100:
+                        raise NoSQLError("Azure Table filter: at most 100 values in a list")
+                    parts.append("(" + " or ".join(_eq(sk, x) for x in orig_v) + ")")
+                    continue
+
+                parts.append(_eq(sk, orig_v))
 
             # Empty-filter contract: if the caller passed filter keys but every
             # condition dropped out (e.g. all values None/unresolvable), the query

altcodepro_polydb_python-2.6.1/tests/test_azure_table_list_filter.py

@@ -0,0 +1,53 @@
+"""A list of plain values in a filter means "equals any of them" (it used to be sent as one JSON string and matched nothing)."""
+import pytest
+
+pytest.importorskip("azure.data.tables")
+
+from polydb.adapters.AzureTableStorageAdapter import AzureTableStorageAdapter  # noqa: E402
+
+
+class _Model:
+    __name__ = "artifacts"
+    __polydb__ = {}
+
+
+class _Fake:
+    def __init__(self):
+        self.calls = []
+
+    def query_entities(self, query_filter=None, **kw):
+        self.calls.append(query_filter)
+        return []
+
+
+def _adapter(fake):
+    a = object.__new__(AzureTableStorageAdapter)
+    a._get_table_client = lambda model: fake
+    a.partition_config = None
+    return a
+
+
+def test_a_list_becomes_an_or_group():
+    f = _Fake()
+    _adapter(f)._query_raw(_Model, {"project_id": "p1", "artifact_definition_id": ["a.x", "b.y", "it's"]}, 50)
+    flt = f.calls[0]
+    assert "project_id eq 'p1'" in flt
+    assert "(artifact_definition_id eq 'a.x' or artifact_definition_id eq 'b.y' or artifact_definition_id eq 'it''s')" in flt
+    assert "polydb_json" not in flt
+
+
+def test_numbers_and_a_single_item_list():
+    f = _Fake()
+    _adapter(f)._query_raw(_Model, {"version": [1, 2], "status": ["complete"]}, 50)
+    assert "(version eq 1 or version eq 2)" in f.calls[0] and "(status eq 'complete')" in f.calls[0]
+
+
+def test_an_empty_list_matches_nothing_without_querying():
+    f = _Fake()
+    assert _adapter(f)._query_raw(_Model, {"status": []}, 50) == [] and f.calls == []
+
+
+def test_a_scalar_filter_is_unchanged():
+    f = _Fake()
+    _adapter(f)._query_raw(_Model, {"status": "complete", "version": 3, "ok": True}, 50)
+    assert f.calls[0] == "status eq 'complete' and version eq 3 and ok eq true"