Skip to content

Commit abefb01

Browse files
igba1980igba.ujege
andauthored
Reinstate skipped test and remove print statement (#99)
Reenabled the previously skipped 'test_status' function in the test_endpoints.py file and renamed it to 'test_status_endpoint'. Removed a redundant print statement from the 'test_ping' function. The updated 'test_status_endpoint' now includes additional assertions to verify the response status. Co-authored-by: igba.ujege <igba.ujege@bjss.com>
1 parent 6bffb42 commit abefb01

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

tests/test_endpoints.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
def test_ping(nhsd_apim_proxy_url):
1313
os.environ["PROXY_NAME"] = "dev"
14-
# print(env)
1514
resp = requests.get(f"{nhsd_apim_proxy_url}/_ping")
1615
assert resp.status_code == 200
1716

@@ -37,13 +36,13 @@ def test_wait_for_ping(nhsd_apim_proxy_url):
3736

3837

3938
@pytest.mark.smoketest
40-
@pytest.mark.skip(reason="No longer required 30/10/2023")
41-
def test_status(nhsd_apim_proxy_url, status_endpoint_auth_headers):
39+
def test_status_endpoint(nhsd_apim_proxy_url, status_endpoint_auth_headers):
4240
resp = requests.get(
4341
f"{nhsd_apim_proxy_url}/_status", headers=status_endpoint_auth_headers
4442
)
4543
assert resp.status_code == 200
46-
# Make some additional assertions about your status response here!
44+
status_json = resp.json()
45+
assert status_json["status"] == "pass"
4746

4847

4948
def test_wait_for_status(nhsd_apim_proxy_url, status_endpoint_auth_headers):

0 commit comments

Comments
 (0)