From: Sebastián Ramírez Date: Thu, 19 Jun 2025 13:23:26 +0000 (+0200) Subject: ✅ Refactor tests to use autouse `clear_sqlmodel` (#1406) X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=75a79a3fcd2b5e2e0b4d24ac33dc5ae13ca70373;p=thirdparty%2Ffastapi%2Fsqlmodel.git ✅ Refactor tests to use autouse `clear_sqlmodel` (#1406) --- diff --git a/tests/conftest.py b/tests/conftest.py index a95eb327..9e8a45cc 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -14,8 +14,8 @@ top_level_path = Path(__file__).resolve().parent.parent docs_src_path = top_level_path / "docs_src" -@pytest.fixture() -def clear_sqlmodel(): +@pytest.fixture(autouse=True) +def clear_sqlmodel() -> Any: # Clear the tables in the metadata for the default base model SQLModel.metadata.clear() # Clear the Models associated with the registry, to avoid warnings diff --git a/tests/test_advanced/test_decimal/test_tutorial001.py b/tests/test_advanced/test_decimal/test_tutorial001.py index 1dafdfb3..2dc56220 100644 --- a/tests/test_advanced/test_decimal/test_tutorial001.py +++ b/tests/test_advanced/test_decimal/test_tutorial001.py @@ -30,7 +30,7 @@ expected_calls = [ ] -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.advanced.decimal import tutorial001 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_advanced/test_decimal/test_tutorial001_py310.py b/tests/test_advanced/test_decimal/test_tutorial001_py310.py index f58ea11a..4cda8b46 100644 --- a/tests/test_advanced/test_decimal/test_tutorial001_py310.py +++ b/tests/test_advanced/test_decimal/test_tutorial001_py310.py @@ -31,7 +31,7 @@ expected_calls = [ @needs_py310 -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.advanced.decimal import tutorial001_py310 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_advanced/test_uuid/test_tutorial001.py b/tests/test_advanced/test_uuid/test_tutorial001.py index 405195f8..b9d5a368 100644 --- a/tests/test_advanced/test_uuid/test_tutorial001.py +++ b/tests/test_advanced/test_uuid/test_tutorial001.py @@ -6,7 +6,7 @@ from sqlmodel import create_engine from ...conftest import get_testing_print_function -def test_tutorial(clear_sqlmodel) -> None: +def test_tutorial() -> None: from docs_src.advanced.uuid import tutorial001 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_advanced/test_uuid/test_tutorial001_py310.py b/tests/test_advanced/test_uuid/test_tutorial001_py310.py index ee8cb085..1250c328 100644 --- a/tests/test_advanced/test_uuid/test_tutorial001_py310.py +++ b/tests/test_advanced/test_uuid/test_tutorial001_py310.py @@ -7,7 +7,7 @@ from ...conftest import get_testing_print_function, needs_py310 @needs_py310 -def test_tutorial(clear_sqlmodel) -> None: +def test_tutorial() -> None: from docs_src.advanced.uuid import tutorial001_py310 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_advanced/test_uuid/test_tutorial002.py b/tests/test_advanced/test_uuid/test_tutorial002.py index cefd95ba..c9f4e5a3 100644 --- a/tests/test_advanced/test_uuid/test_tutorial002.py +++ b/tests/test_advanced/test_uuid/test_tutorial002.py @@ -6,7 +6,7 @@ from sqlmodel import create_engine from ...conftest import get_testing_print_function -def test_tutorial(clear_sqlmodel) -> None: +def test_tutorial() -> None: from docs_src.advanced.uuid import tutorial002 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_advanced/test_uuid/test_tutorial002_py310.py b/tests/test_advanced/test_uuid/test_tutorial002_py310.py index 96f85c53..ba472e30 100644 --- a/tests/test_advanced/test_uuid/test_tutorial002_py310.py +++ b/tests/test_advanced/test_uuid/test_tutorial002_py310.py @@ -7,7 +7,7 @@ from ...conftest import get_testing_print_function, needs_py310 @needs_py310 -def test_tutorial(clear_sqlmodel) -> None: +def test_tutorial() -> None: from docs_src.advanced.uuid import tutorial002_py310 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_automatic_id_none_refresh/test_tutorial001_tutorial002.py b/tests/test_tutorial/test_automatic_id_none_refresh/test_tutorial001_tutorial002.py index 399f431b..5c250471 100644 --- a/tests/test_tutorial/test_automatic_id_none_refresh/test_tutorial001_tutorial002.py +++ b/tests/test_tutorial/test_automatic_id_none_refresh/test_tutorial001_tutorial002.py @@ -133,7 +133,7 @@ def check_calls(calls: List[List[Union[str, Dict[str, Any]]]]): ] -def test_tutorial_001(clear_sqlmodel): +def test_tutorial_001(): from docs_src.tutorial.automatic_id_none_refresh import tutorial001 as mod mod.sqlite_url = "sqlite://" @@ -147,7 +147,7 @@ def test_tutorial_001(clear_sqlmodel): check_calls(calls) -def test_tutorial_002(clear_sqlmodel): +def test_tutorial_002(): from docs_src.tutorial.automatic_id_none_refresh import tutorial002 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_code_structure/test_tutorial001.py b/tests/test_tutorial/test_code_structure/test_tutorial001.py index c6e31583..109c1ef5 100644 --- a/tests/test_tutorial/test_code_structure/test_tutorial001.py +++ b/tests/test_tutorial/test_code_structure/test_tutorial001.py @@ -22,7 +22,7 @@ expected_calls = [ ] -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.code_structure.tutorial001 import app, database database.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_code_structure/test_tutorial001_py310.py b/tests/test_tutorial/test_code_structure/test_tutorial001_py310.py index 44d9d920..126bef25 100644 --- a/tests/test_tutorial/test_code_structure/test_tutorial001_py310.py +++ b/tests/test_tutorial/test_code_structure/test_tutorial001_py310.py @@ -23,7 +23,7 @@ expected_calls = [ @needs_py310 -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.code_structure.tutorial001_py310 import app, database database.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_code_structure/test_tutorial001_py39.py b/tests/test_tutorial/test_code_structure/test_tutorial001_py39.py index b17917cf..02f692ea 100644 --- a/tests/test_tutorial/test_code_structure/test_tutorial001_py39.py +++ b/tests/test_tutorial/test_code_structure/test_tutorial001_py39.py @@ -23,7 +23,7 @@ expected_calls = [ @needs_py39 -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.code_structure.tutorial001_py39 import app, database database.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_code_structure/test_tutorial002.py b/tests/test_tutorial/test_code_structure/test_tutorial002.py index 8e7ac8f1..ccbb8490 100644 --- a/tests/test_tutorial/test_code_structure/test_tutorial002.py +++ b/tests/test_tutorial/test_code_structure/test_tutorial002.py @@ -22,7 +22,7 @@ expected_calls = [ ] -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.code_structure.tutorial002 import app, database database.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_code_structure/test_tutorial002_py310.py b/tests/test_tutorial/test_code_structure/test_tutorial002_py310.py index 3eafdee8..be284866 100644 --- a/tests/test_tutorial/test_code_structure/test_tutorial002_py310.py +++ b/tests/test_tutorial/test_code_structure/test_tutorial002_py310.py @@ -23,7 +23,7 @@ expected_calls = [ @needs_py310 -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.code_structure.tutorial002_py310 import app, database database.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_code_structure/test_tutorial002_py39.py b/tests/test_tutorial/test_code_structure/test_tutorial002_py39.py index 9b5eb670..55f6a435 100644 --- a/tests/test_tutorial/test_code_structure/test_tutorial002_py39.py +++ b/tests/test_tutorial/test_code_structure/test_tutorial002_py39.py @@ -23,7 +23,7 @@ expected_calls = [ @needs_py39 -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.code_structure.tutorial002_py39 import app, database database.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_create_connected_tables/test_tutorial001.py b/tests/test_tutorial/test_connect/test_create_connected_tables/test_tutorial001.py index e3e07992..265a0593 100644 --- a/tests/test_tutorial/test_connect/test_create_connected_tables/test_tutorial001.py +++ b/tests/test_tutorial/test_connect/test_create_connected_tables/test_tutorial001.py @@ -3,7 +3,7 @@ from sqlalchemy.engine.reflection import Inspector from sqlmodel import create_engine -def test_tutorial001(clear_sqlmodel): +def test_tutorial001(): from docs_src.tutorial.connect.create_tables import tutorial001 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_create_connected_tables/test_tutorial001_py310.py b/tests/test_tutorial/test_connect/test_create_connected_tables/test_tutorial001_py310.py index ec2990eb..95f15a42 100644 --- a/tests/test_tutorial/test_connect/test_create_connected_tables/test_tutorial001_py310.py +++ b/tests/test_tutorial/test_connect/test_create_connected_tables/test_tutorial001_py310.py @@ -6,7 +6,7 @@ from ....conftest import needs_py310 @needs_py310 -def test_tutorial001(clear_sqlmodel): +def test_tutorial001(): from docs_src.tutorial.connect.create_tables import tutorial001_py310 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_delete/test_tutorial001.py b/tests/test_tutorial/test_connect/test_delete/test_tutorial001.py index a5db3867..1a9fe293 100644 --- a/tests/test_tutorial/test_connect/test_delete/test_tutorial001.py +++ b/tests/test_tutorial/test_connect/test_delete/test_tutorial001.py @@ -58,7 +58,7 @@ expected_calls = [ ] -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.connect.delete import tutorial001 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_delete/test_tutorial001_py310.py b/tests/test_tutorial/test_connect/test_delete/test_tutorial001_py310.py index edc70b8a..f1bef3ed 100644 --- a/tests/test_tutorial/test_connect/test_delete/test_tutorial001_py310.py +++ b/tests/test_tutorial/test_connect/test_delete/test_tutorial001_py310.py @@ -59,7 +59,7 @@ expected_calls = [ @needs_py310 -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.connect.delete import tutorial001_py310 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_insert/test_tutorial001.py b/tests/test_tutorial/test_connect/test_insert/test_tutorial001.py index 8c8a303a..cfc08ee8 100644 --- a/tests/test_tutorial/test_connect/test_insert/test_tutorial001.py +++ b/tests/test_tutorial/test_connect/test_insert/test_tutorial001.py @@ -38,7 +38,7 @@ expected_calls = [ ] -def test_tutorial001(clear_sqlmodel): +def test_tutorial001(): from docs_src.tutorial.connect.insert import tutorial001 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_insert/test_tutorial001_py310.py b/tests/test_tutorial/test_connect/test_insert/test_tutorial001_py310.py index 854c0068..6dabc10b 100644 --- a/tests/test_tutorial/test_connect/test_insert/test_tutorial001_py310.py +++ b/tests/test_tutorial/test_connect/test_insert/test_tutorial001_py310.py @@ -39,7 +39,7 @@ expected_calls = [ @needs_py310 -def test_tutorial001(clear_sqlmodel): +def test_tutorial001(): from docs_src.tutorial.connect.insert import tutorial001_py310 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_select/test_tutorial001_py310_tutorial002_py310.py b/tests/test_tutorial/test_connect/test_select/test_tutorial001_py310_tutorial002_py310.py index d3bab7f6..4809d79b 100644 --- a/tests/test_tutorial/test_connect/test_select/test_tutorial001_py310_tutorial002_py310.py +++ b/tests/test_tutorial/test_connect/test_select/test_tutorial001_py310_tutorial002_py310.py @@ -63,7 +63,7 @@ expected_calls = [ @needs_py310 -def test_tutorial001(clear_sqlmodel): +def test_tutorial001(): from docs_src.tutorial.connect.select import tutorial001_py310 as mod mod.sqlite_url = "sqlite://" @@ -78,7 +78,7 @@ def test_tutorial001(clear_sqlmodel): @needs_py310 -def test_tutorial002(clear_sqlmodel): +def test_tutorial002(): from docs_src.tutorial.connect.select import tutorial002_py310 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_select/test_tutorial001_tutorial002.py b/tests/test_tutorial/test_connect/test_select/test_tutorial001_tutorial002.py index 541a8ee0..c0d6b59d 100644 --- a/tests/test_tutorial/test_connect/test_select/test_tutorial001_tutorial002.py +++ b/tests/test_tutorial/test_connect/test_select/test_tutorial001_tutorial002.py @@ -62,7 +62,7 @@ expected_calls = [ ] -def test_tutorial001(clear_sqlmodel): +def test_tutorial001(): from docs_src.tutorial.connect.select import tutorial001 as mod mod.sqlite_url = "sqlite://" @@ -76,7 +76,7 @@ def test_tutorial001(clear_sqlmodel): assert calls == expected_calls -def test_tutorial002(clear_sqlmodel): +def test_tutorial002(): from docs_src.tutorial.connect.select import tutorial002 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_select/test_tutorial003.py b/tests/test_tutorial/test_connect/test_select/test_tutorial003.py index 2eab135a..f309e1c4 100644 --- a/tests/test_tutorial/test_connect/test_select/test_tutorial003.py +++ b/tests/test_tutorial/test_connect/test_select/test_tutorial003.py @@ -74,7 +74,7 @@ expected_calls = [ ] -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.connect.select import tutorial003 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_select/test_tutorial003_py310.py b/tests/test_tutorial/test_connect/test_select/test_tutorial003_py310.py index 5b710c43..e826ce44 100644 --- a/tests/test_tutorial/test_connect/test_select/test_tutorial003_py310.py +++ b/tests/test_tutorial/test_connect/test_select/test_tutorial003_py310.py @@ -75,7 +75,7 @@ expected_calls = [ @needs_py310 -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.connect.select import tutorial003_py310 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_select/test_tutorial004.py b/tests/test_tutorial/test_connect/test_select/test_tutorial004.py index ebc273fe..a33c8148 100644 --- a/tests/test_tutorial/test_connect/test_select/test_tutorial004.py +++ b/tests/test_tutorial/test_connect/test_select/test_tutorial004.py @@ -48,7 +48,7 @@ expected_calls = [ ] -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.connect.select import tutorial004 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_select/test_tutorial004_py310.py b/tests/test_tutorial/test_connect/test_select/test_tutorial004_py310.py index 72974ec6..33dd8a43 100644 --- a/tests/test_tutorial/test_connect/test_select/test_tutorial004_py310.py +++ b/tests/test_tutorial/test_connect/test_select/test_tutorial004_py310.py @@ -49,7 +49,7 @@ expected_calls = [ @needs_py310 -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.connect.select import tutorial004_py310 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_select/test_tutorial005.py b/tests/test_tutorial/test_connect/test_select/test_tutorial005.py index 400c6483..f7ad78dc 100644 --- a/tests/test_tutorial/test_connect/test_select/test_tutorial005.py +++ b/tests/test_tutorial/test_connect/test_select/test_tutorial005.py @@ -50,7 +50,7 @@ expected_calls = [ ] -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.connect.select import tutorial005 as mod mod.sqlite_url = "sqlite://" diff --git a/tests/test_tutorial/test_connect/test_select/test_tutorial005_py310.py b/tests/test_tutorial/test_connect/test_select/test_tutorial005_py310.py index a7332c18..8cddb645 100644 --- a/tests/test_tutorial/test_connect/test_select/test_tutorial005_py310.py +++ b/tests/test_tutorial/test_connect/test_select/test_tutorial005_py310.py @@ -51,7 +51,7 @@ expected_calls = [ @needs_py310 -def test_tutorial(clear_sqlmodel): +def test_tutorial(): from docs_src.tutorial.connect.select import tutorial005_py310 as mod mod.sqlite_url = "sqlite://"