Conversation
(cherry picked from commit 5416cc7)
| get_letter_data = requests.get(f"{url}/{LETTERS_ENDPOINT}/{letter_id}/data", headers=headers) | ||
|
|
||
| ErrorHandler.handle_retry(get_letter_data) | ||
| assert get_letter_data.status_code == 200, f"Response: {get_letter_data.status_code}: {get_letter_data.text}" |
There was a problem hiding this comment.
Hmmm - the expected response from the API itself is a 303 that redirects to the download URL
If the client follows the redirect and resolves the link I think you get a 200 at that point
I assume that the test client resolves to the PDF?
|
|
||
| def test_202_with_valid_headers(url, bearer_token): |
There was a problem hiding this comment.
Niggly - but can we remove the gaps between the annotations and the method please?
| @pytest.mark.devtest | ||
| @pytest.mark.inttest | ||
| @pytest.mark.prodtest | ||
|
|
There was a problem hiding this comment.
Niggly - but can we remove the gaps between the annotations and the method please?
| @pytest.mark.devtest | ||
| @pytest.mark.inttest | ||
| @pytest.mark.prodtest | ||
|
|
There was a problem hiding this comment.
Niggly - but can we remove the gaps between the annotations and the method please?
| @pytest.mark.devtest | ||
| @pytest.mark.inttest | ||
| @pytest.mark.prodtest | ||
|
|
There was a problem hiding this comment.
Niggly - but can we remove the gaps between the annotations and the method please?
3 participants
Description
https://nhsd-jira.digital.nhs.uk/browse/CCM-12963
Context
Type of changes
Checklist
Sensitive Information Declaration
To ensure the utmost confidentiality and protect your and others privacy, we kindly ask you to NOT including PII (Personal Identifiable Information) / PID (Personal Identifiable Data) or any other sensitive data in this PR (Pull Request) and the codebase changes. We will remove any PR that do contain any sensitive information. We really appreciate your cooperation in this matter.