From 162868c32d5e0a89fd120cef55ea2ead629c6f4c Mon Sep 17 00:00:00 2001 From: Christoph Zwerschke Date: Tue, 10 Dec 2024 16:03:46 +0000 Subject: [PATCH] Improve docstring --- tests/integration/auth_adapter/test_api.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/tests/integration/auth_adapter/test_api.py b/tests/integration/auth_adapter/test_api.py index b21a820..12c8d82 100644 --- a/tests/integration/auth_adapter/test_api.py +++ b/tests/integration/auth_adapter/test_api.py @@ -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