Skip to content

Commit

Permalink
Improve docstring
Browse files Browse the repository at this point in the history
  • Loading branch information
Cito committed Dec 10, 2024
1 parent 27c3f23 commit 162868c
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions tests/integration/auth_adapter/test_api.py
Original file line number Diff line number Diff line change
Expand Up @@ -71,9 +71,11 @@ def non_mocked_hosts() -> list:
def assert_has_authorization_header(response, session):
"""Check that the response contains the expected authorization header.
Also tests that the correlation ID is made available in a response header.
Note that only authorized requests need this, since for other requests,
response is directly sent back to the client, so no correlation is needed.
Also tests that the correlation ID is available in a response header.
Note that we only need to test this header for authorized requests with
status code 200. For all other kinds of requests (unauthorized or served by
the auth adapter itself), the response is sent back to the client directly,
so in that case no correlation across services is needed.
"""
assert response.status_code == status.HTTP_200_OK

Expand Down

0 comments on commit 162868c

Please sign in to comment.