minor refactoring

Author: harishkannarao

tests_integration/routers/test_customer.py

@@ -11,6 +11,8 @@
 import app.routers.customer as customer
 from app.model.customer import Customer
 
+CUSTOMERS_ENDPOINT = "/context/customers"
+
 
 @pytest.fixture
 def reload_customer_router() -> None:
@@ -22,17 +24,17 @@ def test_customers_insert_read_delete(
     test_client: TestClient,
     reload_customer_router: None,
 ):
-    initial_delete_response: Response = test_client.delete("/context/customers")
+    initial_delete_response: Response = test_client.delete(CUSTOMERS_ENDPOINT)
     assert_that(initial_delete_response.status_code).is_equal_to(204)
 
     customer1 = Customer(first_name="fname1", last_name="lname1")
     customer2 = Customer(first_name="fname2", last_name="lname2")
     input_json = jsonable_encoder([customer1, customer2])
 
-    create_response: Response = test_client.put("/context/customers", json=input_json)
+    create_response: Response = test_client.put(CUSTOMERS_ENDPOINT, json=input_json)
     assert_that(create_response.status_code).is_equal_to(204)
 
-    read_response: Response = test_client.get("/context/customers")
+    read_response: Response = test_client.get(CUSTOMERS_ENDPOINT)
 
     assert_that(read_response.status_code).is_equal_to(200)
     response_json: list[dict[str, Any]] = read_response.json()
@@ -41,10 +43,10 @@ def test_customers_insert_read_delete(
     assert_that(response_json).contains(jsonable_encoder(customer1))
     assert_that(response_json).contains(jsonable_encoder(customer2))
 
-    delete_response: Response = test_client.delete("/context/customers")
+    delete_response: Response = test_client.delete(CUSTOMERS_ENDPOINT)
     assert_that(delete_response.status_code).is_equal_to(204)
 
-    read_response_after_delete: Response = test_client.get("/context/customers")
+    read_response_after_delete: Response = test_client.get(CUSTOMERS_ENDPOINT)
 
     assert_that(read_response_after_delete.status_code).is_equal_to(200)
     response_json_after_delete: list[dict[str, Any]] = read_response_after_delete.json()

tests_unit/routers/test_customer.py

@@ -10,6 +10,8 @@
 
 from app.model.customer import Customer
 
+CUSTOMERS_ENDPOINT = "/context/customers"
+
 
 @pytest.fixture
 def mock_execute(mocker: MockerFixture) -> AsyncMock:
@@ -34,7 +36,7 @@ def mock_fetch_all(mocker: MockerFixture) -> AsyncMock:
 
 
 def test_customers_delete(mock_execute: AsyncMock, test_client: TestClient):
-    response: Response = test_client.delete("/context/customers")
+    response: Response = test_client.delete(CUSTOMERS_ENDPOINT)
     assert_that(response.status_code).is_equal_to(204)
 
     assert len(mock_execute.call_args_list) == 1
@@ -46,7 +48,7 @@ def test_customers_insert(mock_execute_many: AsyncMock, test_client: TestClient)
     customer2 = Customer(first_name="fname2", last_name="lname2")
     input_json = jsonable_encoder([customer1, customer2])
 
-    response: Response = test_client.put("/context/customers", json=input_json)
+    response: Response = test_client.put(CUSTOMERS_ENDPOINT, json=input_json)
     assert_that(response.status_code).is_equal_to(204)
 
     assert len(mock_execute_many.call_args_list) == 1
@@ -65,7 +67,7 @@ def test_customers_read(mock_fetch_all: AsyncMock, test_client: TestClient):
 
     mock_fetch_all.return_value = jsonable_encoder([customer1, customer2])
 
-    response: Response = test_client.get("/context/customers")
+    response: Response = test_client.get(CUSTOMERS_ENDPOINT)
 
     assert_that(response.status_code).is_equal_to(200)
     response_json: list[dict[str, Any]] = response.json()