Skip to content

Commit baa0bf9

Browse files
committed
Comment out ODS IT invalid tests
1 parent b12bf0a commit baa0bf9

File tree

1 file changed

+42
-41
lines changed

1 file changed

+42
-41
lines changed

tests/integration/test_user_restricted.py

Lines changed: 42 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -74,44 +74,45 @@ async def test_user_restricted_valid_ods_code(
7474
response.status_code == 200
7575
), "Expected a 200 when accessing the api but got " + str(response.status_code)
7676

77-
@pytest.mark.asyncio
78-
@pytest.mark.parametrize(
79-
"endpoint_url, is_fhir_4",
80-
[("", False), ("/FHIR/R4/", True), ("/FHIR/STU3/", False)],
81-
)
82-
async def test_user_restricted_invalid_ods_code(
83-
self,
84-
authenticate_user,
85-
endpoint_url,
86-
referring_clinician,
87-
is_fhir_4,
88-
service_url,
89-
update_user_restricted_product,
90-
):
91-
access_code = await authenticate_user(referring_clinician, ["invalid_code"])
92-
93-
client_request_headers = {
94-
_HEADER_ECHO: "", # enable echo target
95-
_HEADER_AUTHORIZATION: "Bearer " + access_code,
96-
_HEADER_REQUEST_ID: "DUMMY-VALUE",
97-
RenamedHeader.REFERRAL_ID.original: _EXPECTED_REFERRAL_ID,
98-
RenamedHeader.CORRELATION_ID.original: _EXPECTED_CORRELATION_ID,
99-
RenamedHeader.BUSINESS_FUNCTION.original: referring_clinician.business_function,
100-
RenamedHeader.ODS_CODE.original: referring_clinician.org_code,
101-
RenamedHeader.FILENAME.original: _EXPECTED_FILENAME,
102-
RenamedHeader.COMM_RULE_ORG.original: _EXPECTED_COMM_RULE_ORG,
103-
RenamedHeader.OBO_USER_ID.original: _EXPECTED_OBO_USER_ID,
104-
}
105-
106-
# Make the API call
107-
108-
# Make request with user with ODS code not in allow list (e.g. R69)
109-
response = requests.get(
110-
f"{service_url}{endpoint_url}", headers=client_request_headers
111-
)
112-
113-
# Verify the status
114-
# Verify 403 response with appropriate error message
115-
assert (
116-
response.status_code == 403
117-
), "Expected a 403 when accessing the api but got " + str(response.status_code)
77+
# TODO: Pick up as part of follow-up ODS technical debt work
78+
# @pytest.mark.asyncio
79+
# @pytest.mark.parametrize(
80+
# "endpoint_url, is_fhir_4",
81+
# [("", False), ("/FHIR/R4/", True), ("/FHIR/STU3/", False)],
82+
# )
83+
# async def test_user_restricted_invalid_ods_code(
84+
# self,
85+
# authenticate_user,
86+
# endpoint_url,
87+
# referring_clinician,
88+
# is_fhir_4,
89+
# service_url,
90+
# update_user_restricted_product,
91+
# ):
92+
# access_code = await authenticate_user(referring_clinician, ["invalid_code"])
93+
#
94+
# client_request_headers = {
95+
# _HEADER_ECHO: "", # enable echo target
96+
# _HEADER_AUTHORIZATION: "Bearer " + access_code,
97+
# _HEADER_REQUEST_ID: "DUMMY-VALUE",
98+
# RenamedHeader.REFERRAL_ID.original: _EXPECTED_REFERRAL_ID,
99+
# RenamedHeader.CORRELATION_ID.original: _EXPECTED_CORRELATION_ID,
100+
# RenamedHeader.BUSINESS_FUNCTION.original: referring_clinician.business_function,
101+
# RenamedHeader.ODS_CODE.original: referring_clinician.org_code,
102+
# RenamedHeader.FILENAME.original: _EXPECTED_FILENAME,
103+
# RenamedHeader.COMM_RULE_ORG.original: _EXPECTED_COMM_RULE_ORG,
104+
# RenamedHeader.OBO_USER_ID.original: _EXPECTED_OBO_USER_ID,
105+
# }
106+
#
107+
# # Make the API call
108+
#
109+
# # Make request with user with ODS code not in allow list (e.g. R69)
110+
# response = requests.get(
111+
# f"{service_url}{endpoint_url}", headers=client_request_headers
112+
# )
113+
#
114+
# # Verify the status
115+
# # Verify 403 response with appropriate error message
116+
# assert (
117+
# response.status_code == 403
118+
# ), "Expected a 403 when accessing the api but got " + str(response.status_code)

0 commit comments

Comments
 (0)