From 316e0d164d2de861caf8e6afed6dc374c92e8e3e Mon Sep 17 00:00:00 2001 From: Sherwin Varghese <141290943+Sherwin-14@users.noreply.github.com> Date: Sat, 27 Jul 2024 15:49:42 +0530 Subject: [PATCH] Remove logger info tests/integration/test_auth.py Co-authored-by: Chuck Daniels --- tests/integration/test_auth.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/tests/integration/test_auth.py b/tests/integration/test_auth.py index 8c98df47..ff85d403 100644 --- a/tests/integration/test_auth.py +++ b/tests/integration/test_auth.py @@ -85,14 +85,12 @@ def test_auth_can_fetch_s3_credentials(daac): auth = earthaccess.login(strategy="environment") assert auth.authenticated try: - logger.info(f"Testing S3 credentials for {daac['short-name']}") credentials = earthaccess.get_s3_credentials(daac["short-name"]) + except requests.RequestException as e: + logger.error(f"Failed to fetch S3 credentials: {e}") + else: assert isinstance(credentials, dict) assert "accessKeyId" in credentials - except Exception as e: - logger.error( - f"An error occured while trying to fetch S3 credentials for {daac['short-name']}: {e}" - ) @pytest.mark.parametrize("location", ({"daac": "podaac"}, {"provider": "pocloud"}))