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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-general-be-lib
-Version: 0.11.0
+Version: 0.11.1
 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.1/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.1/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.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "python-general-be-lib"
-version = "0.11.0"
+version = "0.11.1"
 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.1}/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.1
 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.1}/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.1/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.1}/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",