From 02183efd5f08c0988f58d5742711318214bd22e7 Mon Sep 17 00:00:00 2001 From: Kian Jones <11655409+kianjones9@users.noreply.github.com> Date: Wed, 11 Feb 2026 11:08:30 -0800 Subject: [PATCH] test: enable SQLAlchemy pooling in CI tests (#9279) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * test: enable SQLAlchemy pooling in CI tests Changes CI test config to use LETTA_DISABLE_SQLALCHEMY_POOLING=false, enabling connection pooling to match production settings. 🐾 Generated with [Letta Code](https://letta.com) Co-Authored-By: Letta * test: remove hardcoded LETTA_DISABLE_SQLALCHEMY_POOLING fixture from conftest Remove the fixture that hardcoded the pooling setting in test code. The value should instead come from the CI workflow environment via vars.LETTA_DISABLE_SQLALCHEMY_POOLING (same source as production). 🐾 Generated with [Letta Code](https://letta.com) Co-authored-by: Kian Jones Co-Authored-By: Letta --------- Co-authored-by: Letta Co-authored-by: letta-code <248085862+letta-code@users.noreply.github.com> Co-authored-by: Kian Jones --- tests/conftest.py | 9 --------- 1 file changed, 9 deletions(-) diff --git a/tests/conftest.py b/tests/conftest.py index fada6cb4..452e682f 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -67,15 +67,6 @@ def client(server_url: str) -> Letta: yield client_instance -@pytest.fixture(scope="session", autouse=True) -def disable_db_pooling_for_tests(): - """Disable database connection pooling for the entire test session.""" - os.environ["LETTA_DISABLE_SQLALCHEMY_POOLING"] = "true" - yield - if "LETTA_DISABLE_SQLALCHEMY_POOLING" in os.environ: - del os.environ["LETTA_DISABLE_SQLALCHEMY_POOLING"] - - @pytest.fixture(autouse=True) async def cleanup_db_connections(): """Cleanup database connections after each test."""