clean up API Gateway v1/v2 proxy tests: remove logger, use pytest.raises

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: whummer

aws-proxy/tests/proxy/test_apigateway.py

@@ -1,6 +1,4 @@
-# Note/disclosure: This file has been (partially or fully) generated by an AI agent.
-import logging
-
+# Note: This file has been (partially or fully) generated by an AI agent.
 import boto3
 import pytest
 from botocore.exceptions import ClientError
@@ -9,8 +7,6 @@
 
 from aws_proxy.shared.models import ProxyConfig
 
-logger = logging.getLogger(__name__)
-
 
 def test_apigateway_rest_api_requests(start_aws_proxy, cleanups):
     api_name_aws = f"test-api-aws-{short_uid()}"
@@ -238,8 +234,8 @@ def test_apigateway_read_only_mode(start_aws_proxy, cleanups):
     # allow write operations to be proxied
     original_description = get_api_aws.get("description", "")
 
-    # Attempt write operation - should either be blocked or not proxied
-    try:
+    # Attempt write operation - should be blocked in read-only mode
+    with pytest.raises(Exception):
         apigw_client.update_rest_api(
             restApiId=api_id_aws,
             patchOperations=[
@@ -250,8 +246,6 @@ def test_apigateway_read_only_mode(start_aws_proxy, cleanups):
                 }
             ],
         )
-    except Exception as e:
-        logger.info(f"Read-only mode blocked write operation: {e}")
 
     # Verify the API description was not changed in AWS
     get_api_aws_after = apigw_client_aws.get_rest_api(restApiId=api_id_aws)

aws-proxy/tests/proxy/test_apigatewayv2.py

@@ -1,6 +1,4 @@
-# Note/disclosure: This file has been (partially or fully) generated by an AI agent.
-
-import logging
+# Note: This file has been (partially or fully) generated by an AI agent.
 
 import boto3
 import pytest
@@ -10,8 +8,6 @@
 
 from aws_proxy.shared.models import ProxyConfig
 
-logger = logging.getLogger(__name__)
-
 
 def test_apigatewayv2_http_api_requests(start_aws_proxy, cleanups):
     """Test basic API Gateway v2 HTTP API operations with proxy."""
@@ -225,11 +221,9 @@ def test_apigatewayv2_read_only_mode(start_aws_proxy, cleanups):
     # write operations should not be proxied in read-only mode
     original_description = get_api_aws.get("Description", "")
 
-    # Attempt write operation - should either be blocked or not proxied
-    try:
+    # Attempt write operation - should be blocked in read-only mode
+    with pytest.raises(Exception):
         apigwv2_client.update_api(ApiId=api_id_aws, Description="Should not reach AWS")
-    except Exception as e:
-        logger.info(f"Read-only mode blocked write operation: {e}")
 
     # Verify the API description was not changed in AWS
     get_api_aws_after = apigwv2_client_aws.get_api(ApiId=api_id_aws)