Improvement

Author: harishkannarao

Makefile

@@ -27,7 +27,7 @@ upgrade:
 	uv sync --upgrade
 
 ruff:
-	uv run ruff check
+	uv run ruff check --fix
 	uv run ruff format
 
 flake8:

tests_integration/conftest.py

@@ -11,6 +11,8 @@
 from testcontainers.core.container import DockerContainer
 
 import app.config as config
+import app.dao.customer_dao as customer_dao
+import app.routers.customer as customer
 import app.db.database_config as database_config
 import app.db_schema_migrations.yoyo_migration as yoyo_migration
 import app.main as main
@@ -61,11 +63,15 @@ def change_postgres_db_host(
     new_settings = patch_env_var(monkeypatch, name, new_value)
     reload(yoyo_migration)
     reload(database_config)
+    reload(customer)
+    reload(customer_dao)
     yield new_settings
     # reset to original value and reload module
     patch_env_var(monkeypatch, name, original_value)
     reload(yoyo_migration)
     reload(database_config)
+    reload(customer)
+    reload(customer_dao)
 
 
 @pytest.fixture
@@ -79,11 +85,15 @@ def change_postgres_db_port(
     new_settings = patch_env_var(monkeypatch, name, new_value)
     reload(yoyo_migration)
     reload(database_config)
+    reload(customer)
+    reload(customer_dao)
     yield new_settings
     # reset to original value and reload module
     patch_env_var(monkeypatch, name, original_value)
     reload(yoyo_migration)
     reload(database_config)
+    reload(customer)
+    reload(customer_dao)
 
 
 @pytest.fixture

tests_integration/routers/test_customer.py

@@ -1,28 +1,18 @@
 from typing import Any
 
-import pytest
 from assertpy import assert_that
 from fastapi.encoders import jsonable_encoder
 from fastapi.testclient import TestClient
-from importlib import reload
 
 from httpx import Response
 
-import app.routers.customer as customer
 from app.model.customer import Customer
 
 CUSTOMERS_ENDPOINT = "/context/customers"
 
 
-@pytest.fixture
-def reload_customer_router() -> None:
-    reload(customer)
-    return None
-
-
 def test_customers_insert_read_delete(
     test_client: TestClient,
-    reload_customer_router: None,
 ):
     initial_delete_response: Response = test_client.delete(CUSTOMERS_ENDPOINT)
     assert_that(initial_delete_response.status_code).is_equal_to(204)