feat(tests): add fixture with TestClient definition

Author: nanotaboada

tests/conftest.py

@@ -0,0 +1,13 @@
+import pytest
+import warnings
+from fastapi.testclient import TestClient
+from main import app
+
+# Suppress the DeprecationWarning from httpx
+warnings.filterwarnings("ignore", category=DeprecationWarning)
+
+
+@pytest.fixture(scope="module")
+def client():
+    with TestClient(app) as client:
+        yield client

tests/test_main.py

@@ -4,25 +4,18 @@
 
 import json
 from tests.player_stub import existing_player, non_existing_player, unknown_player
-from main import app
-from fastapi.testclient import TestClient
-import warnings
-# Suppress the DeprecationWarning from httpx
-warnings.filterwarnings("ignore", category=DeprecationWarning)
 
-
-client = TestClient(app)
 PATH = "/players/"
 
 # GET /players/ ----------------------------------------------------------------
 
 
-def test_given_get_when_request_path_has_no_id_then_response_status_code_should_be_200_ok():
+def test_given_get_when_request_path_has_no_id_then_response_status_code_should_be_200_ok(client):
     response = client.get(PATH)
     assert response.status_code == 200
 
 
-def test_given_get_when_request_path_has_no_id_then_response_body_should_be_collection_of_players():
+def test_given_get_when_request_path_has_no_id_then_response_body_should_be_collection_of_players(client):
     response = client.get(PATH)
     players = response.json()
     player_id = 0
@@ -33,19 +26,19 @@ def test_given_get_when_request_path_has_no_id_then_response_body_should_be_coll
 # GET /players/{player_id} -----------------------------------------------------
 
 
-def test_given_get_when_request_path_is_non_existing_id_then_response_status_code_should_be_404_not_found():
+def test_given_get_when_request_path_is_non_existing_id_then_response_status_code_should_be_404_not_found(client):
     player_id = 999
     response = client.get(PATH + str(player_id))
     assert response.status_code == 404
 
 
-def test_given_get_when_request_path_is_existing_id_then_response_status_code_should_be_200_ok():
+def test_given_get_when_request_path_is_existing_id_then_response_status_code_should_be_200_ok(client):
     player_id = 1
     response = client.get(PATH + str(player_id))
     assert response.status_code == 200
 
 
-def test_given_get_when_request_path_is_existing_id_then_response_body_should_be_matching_player():
+def test_given_get_when_request_path_is_existing_id_then_response_body_should_be_matching_player(client):
     player_id = 1
     response = client.get(PATH + str(player_id))
     player = response.json()
@@ -54,19 +47,19 @@ def test_given_get_when_request_path_is_existing_id_then_response_body_should_be
 # GET /players/squadnumber/{squad_number} --------------------------------------
 
 
-def test_given_get_when_request_path_is_non_existing_squad_number_then_response_status_code_should_be_404_not_found():
+def test_given_get_when_request_path_is_non_existing_squad_number_then_response_status_code_should_be_404_not_found(client):
     squad_number = 999
     response = client.get(PATH + "squadnumber" + "/" + str(squad_number))
     assert response.status_code == 404
 
 
-def test_given_get_when_request_path_is_existing_squad_number_then_response_status_code_should_be_200_ok():
+def test_given_get_when_request_path_is_existing_squad_number_then_response_status_code_should_be_200_ok(client):
     squad_number = 10
     response = client.get(PATH + "squadnumber" + "/" + str(squad_number))
     assert response.status_code == 200
 
 
-def test_given_get_when_request_path_is_existing_squad_number_then_response_body_should_be_matching_player():
+def test_given_get_when_request_path_is_existing_squad_number_then_response_body_should_be_matching_player(client):
     squad_number = 10
     response = client.get(PATH + "squadnumber" + "/" + str(squad_number))
     player = response.json()
@@ -75,20 +68,20 @@ def test_given_get_when_request_path_is_existing_squad_number_then_response_body
 # POST /players/ ---------------------------------------------------------------
 
 
-def test_given_post_when_request_body_is_empty_then_response_status_code_should_be_422_unprocessable_entity():
+def test_given_post_when_request_body_is_empty_then_response_status_code_should_be_422_unprocessable_entity(client):
     body = json.dumps({})
     response = client.post(PATH, data=body)
     assert response.status_code == 422
 
 
-def test_given_post_when_request_body_is_existing_player_then_response_status_code_should_be_409_conflict():
+def test_given_post_when_request_body_is_existing_player_then_response_status_code_should_be_409_conflict(client):
     player = existing_player()
     body = json.dumps(player.__dict__)
     response = client.post(PATH, data=body)
     assert response.status_code == 409
 
 
-def test_given_post_when_request_body_is_non_existing_player_then_response_status_code_should_be_201_created():
+def test_given_post_when_request_body_is_non_existing_player_then_response_status_code_should_be_201_created(client):
     player = non_existing_player()
     body = json.dumps(player.__dict__)
     response = client.post(PATH, data=body)
@@ -97,22 +90,22 @@ def test_given_post_when_request_body_is_non_existing_player_then_response_statu
 # PUT /players/{player_id} -----------------------------------------------------
 
 
-def test_given_put_when_request_body_is_empty_then_response_status_code_should_be_422_unprocessable_entity():
+def test_given_put_when_request_body_is_empty_then_response_status_code_should_be_422_unprocessable_entity(client):
     player_id = 1
     body = {}
     response = client.put(PATH + str(player_id), data=body)
     assert response.status_code == 422
 
 
-def test_given_put_when_request_body_is_unknown_player_then_response_status_code_should_be_404_not_found():
+def test_given_put_when_request_body_is_unknown_player_then_response_status_code_should_be_404_not_found(client):
     player_id = 999
     player = unknown_player()
     body = json.dumps(player.__dict__)
     response = client.put(PATH + str(player_id), data=body)
     assert response.status_code == 404
 
 
-def test_given_put_when_request_body_is_existing_player_then_response_status_code_should_be_204_no_content():
+def test_given_put_when_request_body_is_existing_player_then_response_status_code_should_be_204_no_content(client):
     player_id = 1
     player = existing_player()
     player.first_name = "Emiliano"
@@ -124,13 +117,13 @@ def test_given_put_when_request_body_is_existing_player_then_response_status_cod
 # DELETE /players/{player_id} --------------------------------------------------
 
 
-def test_given_delete_when_request_path_is_non_existing_id_then_response_status_code_should_be_404_not_found():
+def test_given_delete_when_request_path_is_non_existing_id_then_response_status_code_should_be_404_not_found(client):
     player_id = 999
     response = client.delete(PATH + str(player_id))
     assert response.status_code == 404
 
 
-def test_given_delete_when_request_path_is_existing_id_then_response_status_code_should_be__204_no_content():
+def test_given_delete_when_request_path_is_existing_id_then_response_status_code_should_be__204_no_content(client):
     player_id = 12
     response = client.delete(PATH + str(player_id))
     assert response.status_code == 204