python-general-be-lib 0.11.0__tar.gz → 0.11.2__tar.gz

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-general-be-lib
-Version: 0.11.0
+Version: 0.11.2
 Summary: General purpose backend library — SQLAlchemy CRUD/Geometry repositories, FastAPI exceptions, Pydantic base models, logger utilities.
 Author-email: Andrea Di Placido <a.diplacido@arpes.it>, "Arpes S.r.l." <it.admin@arpes.it>
 License: MIT

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/general/interface/base/declarative_base.py

@@ -31,6 +31,11 @@ class Base:
         or nothing: passing ``__config__`` puts the whole configuration in the hands of the
         caller, and passing ``__base__`` lets the base carry its own.
 
+        The generated class is stamped with the module of the entity, unless the caller
+        passes ``__module__``: ``create_model`` reads the calling frame otherwise, so every
+        generated model would claim to live in this module and a ``repr`` would point at
+        the library instead of the entity.
+
         Args:
             model_name (str, optional): Name of the generated model. When not given,
                 ``{ClassName}Model`` is used.
@@ -49,6 +54,8 @@ class Base:
         Returns:
             Type[BaseModel]: Dynamically generated Pydantic class.
         """
+        create_model_kwargs.setdefault("__module__", cls.__module__)
+
         if "__config__" not in create_model_kwargs.keys() and "__base__" not in create_model_kwargs.keys():
             create_model_kwargs["__config__"] = ConfigDict(from_attributes=True, populate_by_name=True, arbitrary_types_allowed=True, validate_default=True)
 

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/general/interface/metadata/crud_metadata.py

@@ -342,7 +342,7 @@ class CrudMetadata:
         if rows:
             return rows[0]
         if strict:
-            raise EntityNotFoundException()
+            raise EntityNotFoundException(custom_message=table.name)
         return None
 
     def _get(self, table: Table, connection: Connection, pk: Any, columns: list[str] = None, **kwhere):
@@ -429,7 +429,7 @@ class CrudMetadata:
         if rows:
             return rows[0]
         if strict:
-            raise EntityNotFoundException()
+            raise EntityNotFoundException(custom_message=table.name)
         return None
 
     def insert(self, from_table: str, connection: Connection = None, models: list[Type[BaseModel] | BaseModel] = None, values: list[dict[str, Any]] = None, returning: bool = True,

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/general/interface/repository/crud_repository.py

@@ -423,7 +423,7 @@ class CrudRepository[Entity: Base | Type[Base]]:
         if records:
             return records[0]
         if strict:
-            raise EntityNotFoundException()
+            raise EntityNotFoundException(custom_message=self.entity.__name__)
         return None
 
     def _get(self, session: Session, pk: Any, columns: list[str] = None, exclude_columns: list[str] = None, **kwhere):
@@ -523,7 +523,7 @@ class CrudRepository[Entity: Base | Type[Base]]:
         if records:
             return records[0]
         if strict:
-            raise EntityNotFoundException()
+            raise EntityNotFoundException(custom_message=self.entity.__name__)
         return None
 
     def _find(self, session: Session, where: dict[str, Any] = None, columns: list[str] = None, exclude_columns: list[str] = None, limit: int = 0, orders_by: dict[str, bool] = None, **kwhere):

python_general_be_lib-0.11.2/general/utils/__init__.py

@@ -0,0 +1,3 @@
+from .consts import *
+
+__all__ = ["ENGINE_DEFAULTS", "ENGINE_DEFAULTS_QUEUE_POOL", "SESSION_DEFAULTS"]

python_general_be_lib-0.11.2/general/utils/consts.py

@@ -0,0 +1,6 @@
+__all__ = ["ENGINE_DEFAULTS", "ENGINE_DEFAULTS_QUEUE_POOL", "SESSION_DEFAULTS"]
+
+ENGINE_DEFAULTS = {"pool_pre_ping": True,
+                   "pool_recycle": 1800}
+ENGINE_DEFAULTS_QUEUE_POOL = {"pool_use_lifo": True}
+SESSION_DEFAULTS = {"expire_on_commit": False}

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "python-general-be-lib"
-version = "0.11.0"
+version = "0.11.2"
 description = "General purpose backend library — SQLAlchemy CRUD/Geometry repositories, FastAPI exceptions, Pydantic base models, logger utilities."
 readme = "README.md"
 license = { text = "MIT" }

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/python_general_be_lib.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-general-be-lib
-Version: 0.11.0
+Version: 0.11.2
 Summary: General purpose backend library — SQLAlchemy CRUD/Geometry repositories, FastAPI exceptions, Pydantic base models, logger utilities.
 Author-email: Andrea Di Placido <a.diplacido@arpes.it>, "Arpes S.r.l." <it.admin@arpes.it>
 License: MIT

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/python_general_be_lib.egg-info/SOURCES.txt

@@ -36,6 +36,8 @@ general/interface/repository/handler/json_ilike_handler.py
 general/interface/repository/handler/or_handler.py
 general/interface/repository/handler/soft_delete_handler.py
 general/interface/repository/handler/statement_manager.py
+general/utils/__init__.py
+general/utils/consts.py
 python_general_be_lib.egg-info/PKG-INFO
 python_general_be_lib.egg-info/SOURCES.txt
 python_general_be_lib.egg-info/dependency_links.txt
@@ -43,6 +45,7 @@ python_general_be_lib.egg-info/requires.txt
 python_general_be_lib.egg-info/top_level.txt
 tests/test_caricamento_parziale.py
 tests/test_compiled_cache.py
+tests/test_engine_defaults.py
 tests/test_entity_has_columns.py
 tests/test_execute_contratto.py
 tests/test_exists.py

python_general_be_lib-0.11.2/tests/test_engine_defaults.py

@@ -0,0 +1,103 @@
+"""`ENGINE_DEFAULTS` e `SESSION_DEFAULTS` (3.1 di `.claude/piano-ottimizzazioni.md`).
+
+La voce 3.1 proponeva un `build_engine(...)` ed è stata **scartata**: la configurazione
+dell'engine dipende dal profilo di carico del singolo BE, e una libreria di accesso ai dati
+che decide al posto dell'applicazione sposta la responsabilità dove ci sono meno
+informazioni per deciderla.
+
+Le costanti chiudono la stessa esigenza senza prendere possesso di niente: sono **dati** che
+l'applicativo espande nella propria `create_engine`/`sessionmaker`, e che può ispezionare o
+scartare. La regola su cosa può entrarci è ciò che questi test difendono: **solo chiavi
+portabili fra le classi di pool**. Le altre stanno in una costante separata.
+
+Nessun test qui apre una connessione: `create_engine` non si connette, quindi anche l'URL
+PostgreSQL è verificabile senza database.
+"""
+
+from __future__ import annotations
+
+import pytest
+from sqlalchemy import create_engine
+from sqlalchemy.orm import sessionmaker
+from sqlalchemy.pool import QueuePool, SingletonThreadPool
+
+from general.utils import ENGINE_DEFAULTS, ENGINE_DEFAULTS_QUEUE_POOL, SESSION_DEFAULTS
+
+PG_URL = "postgresql+psycopg://utente:password@localhost:1/inesistente"
+
+
+@pytest.fixture
+def url_su_file(tmp_path) -> str:
+    return f"sqlite:///{tmp_path / 'prova.sqlite'}"
+
+
+# ------------------------------------------------------------------ portabilità
+
+
+def test_la_classe_di_pool_dipende_dall_url_non_dal_dialetto(url_su_file: str) -> None:
+    """È la premessa di tutto il resto, e non è ovvia.
+
+    SQLite **in memoria** usa `SingletonThreadPool`, SQLite **su file** usa `QueuePool`
+    come PostgreSQL: lo stesso dialetto accetta o rifiuta le stesse opzioni a seconda
+    dell'URL.
+    """
+    assert isinstance(create_engine("sqlite://").pool, SingletonThreadPool)
+    assert isinstance(create_engine(url_su_file).pool, QueuePool)
+    assert isinstance(create_engine(PG_URL).pool, QueuePool)
+
+
+@pytest.mark.parametrize("url", ["sqlite://", "file", PG_URL], ids=["sqlite memoria", "sqlite file", "postgresql"])
+def test_engine_defaults_si_applica_su_ogni_pool(url: str, url_su_file: str) -> None:
+    """Il contratto della costante portabile: nessuna chiave che un pool rifiuti."""
+    engine = create_engine(url_su_file if url == "file" else url, **ENGINE_DEFAULTS)
+
+    assert engine.pool._pre_ping is True
+    engine.dispose()
+
+
+def test_le_opzioni_di_queue_pool_stanno_in_una_costante_separata(url_su_file: str) -> None:
+    """E il motivo per cui è separata: su `SingletonThreadPool` sollevano.
+
+    È il difetto che la separazione previene — un consumatore che espande tutto sulla
+    propria suite di test, che gira su SQLite in memoria come questa, non arriverebbe alla
+    prima query.
+    """
+    with pytest.raises(TypeError, match="pool_use_lifo"):
+        create_engine("sqlite://", **ENGINE_DEFAULTS, **ENGINE_DEFAULTS_QUEUE_POOL)
+
+    for url in (url_su_file, PG_URL):
+        engine = create_engine(url, **ENGINE_DEFAULTS, **ENGINE_DEFAULTS_QUEUE_POOL)
+        assert engine.pool._pre_ping is True
+        engine.dispose()
+
+
+# ------------------------------------------------------------------ i valori
+
+
+def test_i_valori_arrivano_davvero_sul_pool(url_su_file: str) -> None:
+    """Una costante che non ha effetto è peggio di nessuna costante."""
+    engine = create_engine(url_su_file, **ENGINE_DEFAULTS, **ENGINE_DEFAULTS_QUEUE_POOL)
+
+    assert engine.pool._pre_ping is True
+    assert engine.pool._recycle == 1800
+    assert engine.pool._pool.use_lifo is True, "il flag vive sulla coda del pool, non sul pool"
+
+
+
+def test_session_defaults_arriva_sulla_sessione() -> None:
+    """``expire_on_commit`` è di ``sessionmaker``, non di ``create_engine``: costante a parte."""
+    sessione = sessionmaker(bind=create_engine("sqlite://"), **SESSION_DEFAULTS)()
+
+    assert sessione.expire_on_commit is False
+
+
+def test_le_costanti_non_sono_condivise_per_riferimento() -> None:
+    """Chi le espande con ``**`` ottiene una copia; chi le muta rompe tutti gli altri.
+
+    Il test non impedisce la mutazione — un ``dict`` è mutabile — ma fissa che l'uso
+    documentato (`create_engine(url, **ENGINE_DEFAULTS)`) non la propaga.
+    """
+    kwargs = {**ENGINE_DEFAULTS}
+    kwargs["pool_pre_ping"] = False
+
+    assert ENGINE_DEFAULTS["pool_pre_ping"] is True

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/tests/test_find_one.py

@@ -56,6 +56,14 @@ def test_con_strict_alza_il_404(repo: CrudRepository) -> None:
     assert errore.value.status_code == 404
 
 
+def test_il_404_nomina_l_entita(repo: CrudRepository) -> None:
+    """Con più letture in fila il 404 deve dire **quale** è andata a vuoto."""
+    with pytest.raises(EntityNotFoundException) as errore:
+        repo.find_one(nome="nessuno", strict=True)
+
+    assert errore.value.detail == "Dato non trovato - Utente"
+
+
 def test_con_strict_la_sessione_viene_chiusa_prima_del_raise(repo: CrudRepository, engine: Engine) -> None:
     """Il ``raise`` sta **dopo** ``_return``, che è quello che chiude la sessione.
 

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/tests/test_get.py

@@ -62,6 +62,14 @@ def test_con_strict_alza_il_404(repo: CrudRepository) -> None:
     assert errore.value.status_code == 404
 
 
+def test_il_404_nomina_l_entita(repo: CrudRepository) -> None:
+    """Con più letture in fila il 404 deve dire **quale** è andata a vuoto."""
+    with pytest.raises(EntityNotFoundException) as errore:
+        repo.get(999, strict=True)
+
+    assert errore.value.detail == "Dato non trovato - Utente"
+
+
 def test_costa_una_sola_query(repo: CrudRepository, engine: Engine) -> None:
     with assert_query_count(engine, 1, verbs=["SELECT"]):
         repo.get(1)

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/tests/test_import_pubblici.py

@@ -19,6 +19,11 @@ import pytest
 
 # (package, simboli che devono essere importabili da lì)
 SUPERFICIE = {
+    "general.utils": [
+        "ENGINE_DEFAULTS",
+        "ENGINE_DEFAULTS_QUEUE_POOL",
+        "SESSION_DEFAULTS",
+    ],
     "general.exception": [
         "raise_http_exception",
         "ExceptionInterface",

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/tests/test_metadata_scorciatoie.py

@@ -72,9 +72,11 @@ def test_get_per_chiave_composta_come_tupla_e_come_dizionario(meta: CrudMetadata
 
 def test_get_senza_corrispondenza_e_con_strict(meta: CrudMetadata) -> None:
     assert meta.get(from_table="m21_semplice", pk=999) is None
-    with pytest.raises(EntityNotFoundException):
+    with pytest.raises(EntityNotFoundException) as errore:
         meta.get(from_table="m21_semplice", pk=999, strict=True)
 
+    assert errore.value.detail == "Dato non trovato - m21_semplice"
+
 
 def test_get_su_una_tabella_senza_chiave_primaria_lo_dice(meta: CrudMetadata) -> None:
     """Caso reale di questo layer: tabelle legacy e viste senza chiave dichiarata."""
@@ -116,9 +118,11 @@ def test_find_one_restituisce_una_riga(meta: CrudMetadata) -> None:
 
 def test_find_one_senza_corrispondenza_e_con_strict(meta: CrudMetadata) -> None:
     assert meta.find_one(from_table="m21_semplice", reparto="inesistente") is None
-    with pytest.raises(EntityNotFoundException):
+    with pytest.raises(EntityNotFoundException) as errore:
         meta.find_one(from_table="m21_semplice", reparto="inesistente", strict=True)
 
+    assert errore.value.detail == "Dato non trovato - m21_semplice"
+
 
 def test_find_one_orders_by_decide_quale_e_la_prima(meta: CrudMetadata) -> None:
     assert meta.find_one(from_table="m21_semplice", reparto="legale", orders_by={"id": True})["id"] == 1

{python_general_be_lib-0.11.0 → python_general_be_lib-0.11.2}/tests/test_to_pydantic.py

@@ -189,3 +189,28 @@ def test_include_only_restringe_i_campi() -> None:
     """Comportamento preesistente, fissato perché è la base dei modelli ridotti."""
     modello = Utente.to_pydantic(full_optional=True, include_only=["id", "nome"])
     assert set(modello.model_fields) == {"id", "nome"}
+
+
+# ------------------------------------------------------- modulo della classe generata
+
+
+def test_il_modello_generato_porta_il_modulo_dell_entita() -> None:
+    """``create_model`` legge il frame del chiamante, che è ``declarative_base``.
+
+    Senza lo stamp esplicito il ``repr`` del tipo — quello che si legge in un ``print``,
+    in un log o in un errore di validazione — indicava la libreria invece dell'entità:
+    `general.interface.base.declarative_base.Utente_1a2b3c4d`, che si scambia facilmente
+    per un'entità ORM. È il caso dei modelli ridotti di ``find(columns=[...])``, generati
+    con un nome che l'entità non dichiara da nessuna parte.
+    """
+    modello = Utente.to_pydantic(full_optional=True)
+
+    assert modello.__module__ == Utente.__module__
+    assert "declarative_base" not in repr(modello)
+
+
+def test_il_modulo_passato_dal_chiamante_vince() -> None:
+    """Lo stamp è un default, non un'imposizione: ``__module__`` resta un parametro."""
+    modello = Utente.to_pydantic(full_optional=True, __module__="mio.modulo")
+
+    assert modello.__module__ == "mio.modulo"