altcodepro-polydb-python 2.5.19__tar.gz → 2.5.21__tar.gz

{altcodepro_polydb_python-2.5.19/src/altcodepro_polydb_python.egg-info → altcodepro_polydb_python-2.5.21}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: altcodepro-polydb-python
-Version: 2.5.19
+Version: 2.5.21
 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.5.19 → altcodepro_polydb_python-2.5.21}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "altcodepro-polydb-python"
-version = "2.5.19"
+version = "2.5.21"
 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.5.19 → altcodepro_polydb_python-2.5.21/src/altcodepro_polydb_python.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: altcodepro-polydb-python
-Version: 2.5.19
+Version: 2.5.21
 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.5.19 → altcodepro_polydb_python-2.5.21}/src/altcodepro_polydb_python.egg-info/SOURCES.txt

@@ -90,15 +90,18 @@ tests/test_audit_chain_cutover.py
 tests/test_audit_hmac.py
 tests/test_aws.py
 tests/test_azure.py
+tests/test_azure_keyvault_credential.py
 tests/test_azure_queue_receipt_handle.py
 tests/test_azure_table_empty_filter.py
 tests/test_azure_table_id_property_persisted.py
 tests/test_azure_table_id_return_value_sanitization_mismatch.py
 tests/test_azure_table_large_record_patch_roundtrip.py
+tests/test_azure_table_self_heal.py
 tests/test_azure_table_unpack_entity.py
 tests/test_blockchain.py
 tests/test_blockchain_overflow_guard.py
 tests/test_cloud_factory.py
+tests/test_factory_wiring.py
 tests/test_field_encryption.py
 tests/test_gcp.py
 tests/test_kafka.py

{altcodepro_polydb_python-2.5.19 → altcodepro_polydb_python-2.5.21}/src/polydb/__init__.py

@@ -4,7 +4,7 @@ PolyDB - Enterprise Cloud-Independent Database Abstraction
 Full LINQ support, field-level audit, cache, soft delete, overflow storage
 """
 
-__version__ = "2.5.18"
+__version__ = "2.5.20"
 
 from .aio import AsyncDatabaseFactory, AsyncPolyDB
 from .audit.context import AuditContext

{altcodepro_polydb_python-2.5.19 → altcodepro_polydb_python-2.5.21}/src/polydb/adapters/AzureKeyVaultAdapter.py

@@ -19,10 +19,24 @@ class AzureKeyVaultAdapter(SecretsAdapter):
         self._initialize_client()
 
     def _initialize_client(self) -> None:
-        from azure.identity import DefaultAzureCredential
         from azure.keyvault.secrets import SecretClient
 
-        self._client = SecretClient(vault_url=self.vault_url, credential=DefaultAzureCredential())
+        self._client = SecretClient(vault_url=self.vault_url, credential=self._build_credential())
+
+    @staticmethod
+    def _build_credential():
+        """Inside Azure Container Apps / App Service / Functions a managed identity is available
+        (IDENTITY_ENDPOINT / MSI_ENDPOINT). DefaultAzureCredential tries EnvironmentCredential FIRST, and a deployment that
+        also exports AZURE_CLIENT_ID + AZURE_CLIENT_SECRET (for some other purpose) then fails with AADSTS7000232
+        "MSI identity should not use ClientSecretCredential". So when a managed identity exists it goes first
+        (AZURE_CLIENT_ID, if set, names a user-assigned identity); the normal chain stays as the fallback, so local
+        development (az login, environment variables) behaves as before."""
+        from azure.identity import ChainedTokenCredential, DefaultAzureCredential, ManagedIdentityCredential
+
+        if os.getenv("IDENTITY_ENDPOINT") or os.getenv("MSI_ENDPOINT"):
+            client_id = os.getenv("AZURE_CLIENT_ID") or None
+            return ChainedTokenCredential(ManagedIdentityCredential(client_id=client_id), DefaultAzureCredential())
+        return DefaultAzureCredential()
 
     # Key Vault secret names may only contain alphanumerics and dashes --
     # our secret keys (e.g. "tenant-42/stripe/api_key") use slashes, so

{altcodepro_polydb_python-2.5.19 → altcodepro_polydb_python-2.5.21}/src/polydb/adapters/AzureTableStorageAdapter.py

@@ -47,6 +47,59 @@ _RETRY_TOTAL = int(os.getenv("AZURE_TABLE_RETRY_TOTAL", "3"))
 logging.getLogger("azure").setLevel(logging.ERROR)
 
 
+
+def _is_table_missing(exc: BaseException) -> bool:
+    text = f"{type(exc).__name__} {exc}"
+    return "TableNotFound" in text or "The table specified does not exist" in text
+
+
+class _SelfHealingTableClient:
+    """Wraps an Azure TableClient so a table that disappears while the process is running (deleted by hand, a data
+    reset, a restore) is created again and the operation retried once, instead of failing with TableNotFound until
+    the process restarts. Lazy iterators (query_entities / list_entities) are healed only if the failure happens
+    before the first row was produced."""
+
+    _LAZY = ("query_entities", "list_entities")
+
+    def __init__(self, adapter: "AzureTableStorageAdapter", table_name: str, client: Any):
+        self._adapter = adapter
+        self._table_name = table_name
+        self._client = client
+
+    def _heal(self) -> None:
+        self._adapter._ensured_tables.discard(self._table_name)
+        self._adapter._client.create_table_if_not_exists(self._table_name)
+        self._adapter._ensured_tables.add(self._table_name)
+        logger.warning(f"Azure Table {self._table_name} was missing; created it again")
+
+    def __getattr__(self, name: str):
+        target = getattr(self._client, name)
+        if not callable(target):
+            return target
+        if name in self._LAZY:
+            def lazy(*args, **kwargs):
+                produced = False
+                try:
+                    for item in target(*args, **kwargs):
+                        produced = True
+                        yield item
+                except Exception as exc:  # noqa: BLE001
+                    if produced or not _is_table_missing(exc):
+                        raise
+                    self._heal()
+                    yield from getattr(self._client, name)(*args, **kwargs)
+            return lazy
+
+        def call(*args, **kwargs):
+            try:
+                return target(*args, **kwargs)
+            except Exception as exc:  # noqa: BLE001
+                if not _is_table_missing(exc):
+                    raise
+                self._heal()
+                return getattr(self._client, name)(*args, **kwargs)
+        return call
+
 class AzureTableStorageAdapter(NoSQLKVAdapter):
     """
     Azure Table Storage adapter with:
@@ -441,17 +494,21 @@ class AzureTableStorageAdapter(NoSQLKVAdapter):
                 try:
                     self._client.create_table_if_not_exists(table_name)
                     logger.info(f"✅ Azure Table ensured/created: {table_name}")
+                    self._ensured_tables.add(table_name)
                 except Exception as e:
                     msg = str(e)
-                    if "TableAlreadyExists" not in msg and "already exists" not in msg.lower():
-                        logger.warning(f"Could not create table {table_name}: {e}")
-                self._ensured_tables.add(table_name)
+                    if "TableAlreadyExists" in msg or "already exists" in msg.lower():
+                        self._ensured_tables.add(table_name)
+                    else:
+                        # Do NOT remember a failed creation as "ensured": the next call tries again instead of
+                        # failing with TableNotFound for the rest of the process's life.
+                        logger.error(f"Could not create table {table_name}: {e}")
 
             if cached is None:
                 cached = self._client.get_table_client(table_name=table_name)
                 self._table_clients_cache[table_name] = cached
 
-            return cached
+            return _SelfHealingTableClient(self, table_name, cached)
 
     def _restore_overflow_properties(self, entity_dict: JsonDict) -> JsonDict:
         restored = {}

{altcodepro_polydb_python-2.5.19 → altcodepro_polydb_python-2.5.21}/src/polydb/adapters/DynamoDBAdapter.py

@@ -196,6 +196,13 @@ class DynamoDBAdapter(NoSQLKVAdapter):
     def _blob_key(self, model: type, pk: str, rk: str, checksum: str) -> str:
         return f"overflow/{self._table_name(model)}/{pk}/{rk}/{checksum}.json"
 
+    def _check_overflow(self, data: JsonDict):  # type: ignore[override]
+        """DynamoDB overflows inside `_put_raw` (`_maybe_overflow_to_s3`, into this adapter's own bucket), never in the
+        base class. The base whole-record overflow replaced the record with a stub pointing at a key in the separate
+        object store, which `_resolve_overflow` (it reads this adapter's bucket) can never find -- the record was lost
+        on read. Returning the data untouched leaves the one working path in charge."""
+        return data, None
+
     def _maybe_overflow_to_s3(
         self, model: type, pk: str, rk: str, payload: JsonDict
     ) -> Optional[JsonDict]:

{altcodepro_polydb_python-2.5.19 → altcodepro_polydb_python-2.5.21}/src/polydb/base/NoSQLKVAdapter.py

@@ -291,7 +291,9 @@ class NoSQLKVAdapter:
     ) -> Tuple[List[JsonDict], Optional[str]]:
         # Basic implementation - override per provider
         offset = int(continuation_token) if continuation_token else 0
-        results = self.query(model, query, limit=page_size + 1)
+        # Fetch through the end of THIS page (+1 to learn whether another follows) and drop what earlier pages already
+        # returned -- reading from the start every time made every page after the first a repeat of the first.
+        results = self.query(model, query, limit=offset + page_size + 1)[offset:]
 
         has_more = len(results) > page_size
         if has_more:

{altcodepro_polydb_python-2.5.19 → altcodepro_polydb_python-2.5.21}/src/polydb/databaseFactory.py

@@ -19,7 +19,7 @@ from dataclasses import dataclass, field
 from datetime import datetime
 from typing import Any, Callable, Dict, List, Optional, Set, Tuple, Union
 
-from tenacity import retry, stop_after_attempt, wait_exponential
+from tenacity import retry, retry_if_exception, stop_after_attempt, wait_exponential
 
 from .adapters.PostgreSQLAdapter import PostgreSQLAdapter
 from .audit.context import AuditContext
@@ -32,6 +32,7 @@ from .errors import AdapterConfigurationError
 from .models import PageRequest, PageResult
 from .monitoring import HealthCheck, MetricsCollector, PerformanceMonitor
 from .query import QueryBuilder
+from .retry import _is_non_retryable
 from .security import DataMasking, FieldEncryption
 from .types import JsonDict, Lookup, ModelMeta
 
@@ -53,6 +54,31 @@ _DEFAULT_RETRY = retry(
     reraise=True,
 )
 
+_TRANSIENT_MARKERS = (
+    "timeout", "timed out", "connection reset", "connection refused", "connection closed", "connection aborted",
+    "server closed the connection", "temporarily unavailable", "too many requests", "throttl", "service unavailable",
+    "serverbusy", "503", "could not connect",
+)
+
+
+def _is_transient(exc: BaseException) -> bool:
+    if _is_non_retryable(exc):
+        return False
+    if isinstance(exc, (ConnectionError, TimeoutError)):
+        return True
+    s = str(exc).lower()
+    return any(m in s for m in _TRANSIENT_MARKERS)
+
+
+def _TRANSIENT_RETRY(fn):
+    return retry(
+        retry=retry_if_exception(_is_transient),
+        wait=wait_exponential(multiplier=0.5, min=0.5, max=6),
+        stop=stop_after_attempt(3),
+        reraise=True,
+    )(fn)
+
+
 _UNIQUE_VIOLATION_MARKERS = (
     "23505",  # Postgres SQLSTATE
     "duplicate key value violates",  # Postgres message
@@ -140,6 +166,7 @@ def _extract_meta(model: Union[type, str]) -> ModelMeta:
                 provider=raw.get("provider"),
                 cache=raw.get("cache", False),
                 cache_ttl=raw.get("cache_ttl"),
+                encrypted_fields=tuple(raw.get("encrypted_fields") or ()),
             )
     return ModelMeta(storage="nosql", table=None, collection=None)
 
@@ -348,7 +375,42 @@ class DatabaseFactory:
         return result
 
     def _run(self, fn: Callable[[], Any]) -> Any:
-        return fn()
+        """Run one storage operation, retrying only TRANSIENT failures (connection loss, timeouts, throttling) with
+        exponential backoff when retries are enabled. Constraint violations, bad arguments and anything the retry
+        classifier marks non-retryable are raised at once -- a retry could not change their outcome."""
+        if not self._enable_retries:
+            return fn()
+        return _TRANSIENT_RETRY(fn)()
+
+    def _audit_write(
+        self, action: str, name: str, meta: ModelMeta, entity_id: Any, success: bool,
+        before: Optional[JsonDict], after: Optional[JsonDict], error: Optional[str],
+    ) -> None:
+        """Record a write in the audit log. Never raises: an audit failure must not turn a completed write into an
+        error. Encrypted fields are masked so the log never holds their plaintext."""
+        if not (self._enable_audit and self._audit):
+            return
+
+        def _mask(row: Optional[JsonDict]) -> Optional[JsonDict]:
+            if not row or not meta.encrypted_fields:
+                return row
+            return {k: ("[encrypted]" if k in meta.encrypted_fields else v) for k, v in row.items()}
+
+        try:
+            before_m, after_m = _mask(before), _mask(after)
+            changed = (
+                sorted(k for k in set(after_m or {}) | set(before_m or {}) if (before_m or {}).get(k) != (after_m or {}).get(k))
+                if before_m is not None and after_m is not None else None
+            )
+            if isinstance(entity_id, dict):
+                entity_id = entity_id.get("id")
+            self._audit.record(
+                action=action, model=name, entity_id=None if entity_id is None else str(entity_id),
+                storage_type=meta.storage, provider=meta.provider or "default", success=success,
+                before=before_m, after=after_m, error=error, changed_fields=changed,
+            )
+        except Exception as exc:  # noqa: BLE001
+            logger.warning("Audit write failed (non-fatal): %s", exc)
 
     def _is_sql(self, meta: ModelMeta, override: Optional[EngineOverride] = None) -> bool:
         if override and override.force_sql:
@@ -432,9 +494,12 @@ class DatabaseFactory:
                 with monitor as m:
                     result = self._run(_op)
                     m.rows_affected = 1
-                    return result
-            return self._run(_op)
-        except Exception:
+            else:
+                result = self._run(_op)
+            self._audit_write("create", name, meta, entity_id, True, None, after_plain, None)
+            return result
+        except Exception as exc:
+            self._audit_write("create", name, meta, entity_id, False, None, None, str(exc))
             raise
 
     # ──────────────────────────────────────────────────────────
@@ -467,14 +532,26 @@ class DatabaseFactory:
         projected = bool(fields or omit)
         sql_columns = select_columns(fields, omit, columns)
 
+        drop_deleted = False
         if self._soft_delete and not include_deleted:
-            query = self._apply_soft_delete_filter(query)
+            if self._is_sql(meta, engine_override):
+                query = self._apply_soft_delete_filter(query)
+            else:
+                # A NoSQL equality filter on `deleted_at = None` matches nothing when the attribute was never written
+                # (Azure Table, DynamoDB), so filter the returned rows instead.
+                drop_deleted = True
 
         adapters = self._adapters_for(model, meta, engine_override)
         use_external_cache = (
             self._enable_cache and self._cache and getattr(meta, "cache", False) and not projected
         )
         encrypted_fields = getattr(meta, "encrypted_fields", [])
+        # The cache key must cover everything that changes the result: the query, the window, and the session variables
+        # (e.g. the tenant a row-level-security policy filters by) -- otherwise two callers share an entry.
+        cache_key_query = {
+            "q": query or {}, "limit": limit, "offset": offset,
+            "sv": sorted((session_vars or {}).items()), "e": bool(engine_override),
+        }
 
         def _op() -> List[JsonDict]:
             if self._is_sql(meta, engine_override):
@@ -497,22 +574,27 @@ class DatabaseFactory:
                 nosql_kwargs: Dict[str, Any] = {}
                 if projected:
                     nosql_kwargs = {"fields": fields, "omit": omit}
+                skip = offset or 0
                 raw = adapters.nosql.query(
-                    cls, query=query, limit=limit, no_cache=no_cache or bool(use_external_cache),
-                    **nosql_kwargs,
+                    cls, query=query, limit=(limit + skip) if limit else None,
+                    no_cache=no_cache or bool(use_external_cache), **nosql_kwargs,
                 )
+                if skip:
+                    raw = raw[skip:]  # NoSQL adapters take no offset: read through it, then drop the skipped rows
+            if drop_deleted:
+                raw = [r for r in raw if not r.get("deleted_at")]
             if self.encryption and encrypted_fields:
                 raw = [self.encryption.decrypt_fields(r, encrypted_fields) for r in raw]
             if self._cache and use_external_cache and not no_cache:
                 ttl = cache_ttl or getattr(meta, "cache_ttl", 300)
                 try:
-                    self._cache.set(name, query or {}, raw, ttl)
+                    self._cache.set(name, cache_key_query, raw, ttl)
                 except Exception as _ce:
                     logger.warning("Cache set failed (non-fatal): %s", _ce)
             return raw
 
         if self._cache and use_external_cache and not no_cache:
-            cached = self._cache.get(name, query or {})
+            cached = self._cache.get(name, cache_key_query)
             if cached is not None:
                 return cached
 
@@ -677,9 +759,12 @@ class DatabaseFactory:
                 with monitor as m:
                     result = self._run(_op)
                     m.rows_affected = 1
-                    return result
-            return self._run(_op)
-        except Exception:
+            else:
+                result = self._run(_op)
+            self._audit_write("update", name, meta, entity_id, True, before, after_plain, None)
+            return result
+        except Exception as exc:
+            self._audit_write("update", name, meta, entity_id, False, before, None, str(exc))
             raise
 
     # ──────────────────────────────────────────────────────────
@@ -737,9 +822,12 @@ class DatabaseFactory:
                 with monitor as m:
                     result = self._run(_op)
                     m.rows_affected = 1
-                    return result
-            return self._run(_op)
-        except Exception:
+            else:
+                result = self._run(_op)
+            self._audit_write("upsert", name, meta, (after_plain or {}).get('id'), True, None, after_plain, None)
+            return result
+        except Exception as exc:
+            self._audit_write("upsert", name, meta, None, False, None, None, str(exc))
             raise
 
     # ──────────────────────────────────────────────────────────
@@ -850,9 +938,12 @@ class DatabaseFactory:
                 with monitor as m:
                     result = self._run(_op)
                     m.rows_affected = 1
-                    return result
-            return self._run(_op)
-        except Exception:
+            else:
+                result = self._run(_op)
+            self._audit_write("delete", name, meta, entity_id, True, before, None, None)
+            return result
+        except Exception as exc:
+            self._audit_write("delete", name, meta, entity_id, False, before, None, str(exc))
             raise
 
     # ──────────────────────────────────────────────────────────
@@ -906,8 +997,12 @@ class DatabaseFactory:
         name = _model_name(model)
         meta = _extract_meta(model)
 
+        drop_deleted = False
         if self._soft_delete and not include_deleted:
-            query = self._apply_soft_delete_filter(query)
+            if self._is_sql(meta, engine_override):
+                query = self._apply_soft_delete_filter(query)
+            else:
+                drop_deleted = True
 
         adapters = self._adapters_for(model, meta, engine_override)
         encrypted_fields = getattr(meta, "encrypted_fields", [])
@@ -924,6 +1019,8 @@ class DatabaseFactory:
                     else type(name, (), {"__polydb__": meta.__dict__})
                 )
                 raw, token = adapters.nosql.query_page(cls, query, page_size, continuation_token)
+                if drop_deleted:
+                    raw = [r for r in raw if not r.get("deleted_at")]
             if self.encryption and encrypted_fields:
                 raw = [self.encryption.decrypt_fields(r, encrypted_fields) for r in raw]
             return raw, token

{altcodepro_polydb_python-2.5.19 → altcodepro_polydb_python-2.5.21}/src/polydb/schema.py

@@ -131,6 +131,27 @@ class SchemaBuilder:
 
         return sql
 
+    def to_add_missing_columns(self, table_name: str) -> List[str]:
+        """`ALTER TABLE ... ADD COLUMN IF NOT EXISTS` for every non-key column. `CREATE TABLE IF NOT EXISTS` never adds a
+        column that was declared after the table was first created; appending these to the same migration makes an edited
+        column list reach existing tables. Added columns are always nullable (existing rows have no value) and never
+        UNIQUE; a declared default is kept."""
+        validate_table_name(table_name)
+        keys = set(self.primary_keys or [])
+        out: List[str] = []
+        for col in self.columns:
+            if col.name in keys or col.primary_key:
+                continue
+            parts = [validate_column_name(col.name)]
+            if col.type == ColumnType.VARCHAR and col.max_length:
+                parts.append(f"VARCHAR({col.max_length})")
+            else:
+                parts.append(col.type.value)
+            if col.default is not None:
+                parts.append(_render_default(col.default))
+            out.append(f"ALTER TABLE {table_name} ADD COLUMN IF NOT EXISTS {' '.join(parts)};")
+        return out
+
     def to_create_indexes(self, table_name: str) -> List[str]:
         """Generate CREATE INDEX statements"""
         statements = []
@@ -185,15 +206,31 @@ class MigrationManager:
 
         # Check if already applied
         existing = self.sql.execute(
-            "SELECT version FROM polydb_migrations WHERE version = %s", [version], fetch_one=True
+            "SELECT version, checksum FROM polydb_migrations WHERE version = %s", [version], fetch_one=True
         )
 
-        if existing:
-            return False
-
         # Calculate checksum
         checksum = hashlib.sha256(up_sql.encode()).hexdigest()
 
+        if existing:
+            stored = existing.get("checksum") if isinstance(existing, dict) else None
+            if not stored or stored == checksum:
+                return False
+            # The DDL under this version was edited (a column added, an index changed). Re-apply it -- the statements
+            # this codebase generates are idempotent (IF NOT EXISTS) -- and remember the new checksum. A failure here
+            # must not stop a deployment from starting: keep the old checksum so the next start tries again.
+            try:
+                self.sql.execute(up_sql)
+                self.sql.execute(
+                    "UPDATE polydb_migrations SET checksum = %s WHERE version = %s", [checksum, version]
+                )
+                return True
+            except Exception as exc:  # noqa: BLE001
+                import logging
+
+                logging.getLogger(__name__).warning("Re-applying edited migration %s failed (kept old): %s", version, exc)
+                return False
+
         try:
             # Execute migration
             self.sql.execute(up_sql)

{altcodepro_polydb_python-2.5.19 → altcodepro_polydb_python-2.5.21}/src/polydb/types.py

@@ -24,6 +24,8 @@ class ModelMeta:
     provider: Optional[str] = None
     cache: bool = False
     cache_ttl: Optional[int] = None
+    # Fields stored encrypted when the factory has encryption enabled (a tuple: the dataclass is frozen/hashable).
+    encrypted_fields: Tuple[str, ...] = ()
 
 
 @runtime_checkable

altcodepro_polydb_python-2.5.21/tests/test_azure_keyvault_credential.py

@@ -0,0 +1,34 @@
+"""Inside a managed-identity host the managed identity is tried first, so a stray AZURE_CLIENT_SECRET cannot break Key Vault."""
+import sys
+import types
+from unittest.mock import MagicMock
+
+import pytest
+
+
+@pytest.fixture
+def fake_identity(monkeypatch):
+    mod = types.ModuleType("azure.identity")
+    mod.ChainedTokenCredential = MagicMock(name="Chained")
+    mod.DefaultAzureCredential = MagicMock(name="Default")
+    mod.ManagedIdentityCredential = MagicMock(name="MI")
+    monkeypatch.setitem(sys.modules, "azure.identity", mod)
+    return mod
+
+
+def test_a_managed_identity_host_puts_the_managed_identity_first(fake_identity, monkeypatch):
+    from polydb.adapters.AzureKeyVaultAdapter import AzureKeyVaultAdapter
+    monkeypatch.setenv("IDENTITY_ENDPOINT", "http://169.254.0.1/msi")
+    monkeypatch.setenv("AZURE_CLIENT_ID", "mi-client-id")
+    AzureKeyVaultAdapter._build_credential()
+    fake_identity.ManagedIdentityCredential.assert_called_once_with(client_id="mi-client-id")
+    chain_args = fake_identity.ChainedTokenCredential.call_args.args
+    assert chain_args[0] is fake_identity.ManagedIdentityCredential.return_value and chain_args[1] is fake_identity.DefaultAzureCredential.return_value
+
+
+def test_elsewhere_the_default_chain_is_unchanged(fake_identity, monkeypatch):
+    from polydb.adapters.AzureKeyVaultAdapter import AzureKeyVaultAdapter
+    monkeypatch.delenv("IDENTITY_ENDPOINT", raising=False)
+    monkeypatch.delenv("MSI_ENDPOINT", raising=False)
+    assert AzureKeyVaultAdapter._build_credential() is fake_identity.DefaultAzureCredential.return_value
+    fake_identity.ManagedIdentityCredential.assert_not_called()

altcodepro_polydb_python-2.5.21/tests/test_azure_table_self_heal.py

@@ -0,0 +1,64 @@
+"""A missing Azure Table is created again and the call retried; a failed creation is not remembered as success."""
+from unittest.mock import MagicMock
+
+from polydb.adapters.AzureTableStorageAdapter import _SelfHealingTableClient, _is_table_missing
+
+
+class _Err(Exception):
+    pass
+
+
+def _adapter():
+    a = MagicMock()
+    a._ensured_tables = {"t"}
+    return a
+
+
+def test_detects_the_missing_table_error():
+    assert _is_table_missing(_Err("ErrorCode:TableNotFound"))
+    assert _is_table_missing(_Err("The table specified does not exist."))
+    assert not _is_table_missing(_Err("timeout"))
+
+
+def test_a_call_on_a_missing_table_creates_it_and_retries_once():
+    client = MagicMock()
+    client.get_entity.side_effect = [_Err("TableNotFound"), {"ok": 1}]
+    a = _adapter()
+    got = _SelfHealingTableClient(a, "t", client).get_entity(partition_key="p", row_key="r")
+    assert got == {"ok": 1} and a._client.create_table_if_not_exists.called and "t" in a._ensured_tables
+
+
+def test_other_errors_are_not_swallowed():
+    client = MagicMock()
+    client.get_entity.side_effect = _Err("boom")
+    try:
+        _SelfHealingTableClient(_adapter(), "t", client).get_entity()
+        raise AssertionError("should raise")
+    except _Err:
+        pass
+
+
+def test_a_lazy_query_on_a_missing_table_heals_before_the_first_row():
+    client = MagicMock()
+    def gone():
+        raise _Err("TableNotFound")
+        yield
+    client.query_entities.side_effect = [gone(), iter([{"a": 1}, {"a": 2}])]
+    a = _adapter()
+    rows = list(_SelfHealingTableClient(a, "t", client).query_entities("x"))
+    assert rows == [{"a": 1}, {"a": 2}] and a._client.create_table_if_not_exists.called
+
+
+def test_a_failure_after_rows_were_produced_is_raised_not_replayed():
+    client = MagicMock()
+    def partial():
+        yield {"a": 1}
+        raise _Err("TableNotFound")
+    client.query_entities.return_value = partial()
+    it = _SelfHealingTableClient(_adapter(), "t", client).query_entities("x")
+    assert next(it) == {"a": 1}
+    try:
+        next(it)
+        raise AssertionError("should raise")
+    except _Err:
+        pass