update comments

Author: parthea

tests/system/conftest.py

@@ -381,9 +381,8 @@ async def intercept_stream_stream(
 
 @pytest.fixture
 def intercepted_echo_grpc(use_mtls):
-    # The interceptor adds 'showcase-trailer' client metadata. Showcase server
-    # echoes any metadata with key 'showcase-trailer', so the same metadata
-    # should appear as trailing metadata in the response.
+    # The interceptor reads request
+    # and response metadata.
     interceptor = EchoMetadataClientGrpcInterceptor()
     host = "localhost:7469"
     channel = (
@@ -401,9 +400,8 @@ def intercepted_echo_grpc(use_mtls):
 
 @pytest_asyncio.fixture
 async def intercepted_echo_grpc_async():
-    # The interceptor adds 'showcase-trailer' client metadata. Showcase server
-    # echoes any metadata with key 'showcase-trailer', so the same metadata
-    # should appear as trailing metadata in the response.
+    # The interceptor reads request
+    # and response metadata.
     interceptor = EchoMetadataClientGrpcAsyncInterceptor()
     host = "localhost:7469"
     channel = grpc.aio.insecure_channel(host, interceptors=[interceptor])

tests/system/test_grpc_interceptor_streams.py

@@ -15,10 +15,10 @@
 from google import showcase
 
 
-# intercetped_metadata will be added by the interceptor automatically, and
+# `_METADATA` will be sent as part of the request, and the
 # showcase server will echo it (since it has key 'showcase-trailer') as trailing
 # metadata.
-intercepted_metadata = (("showcase-trailer", "intercepted"),)
+_METADATA = (("showcase-trailer", "intercepted"),)
 
 
 def test_unary_stream(intercepted_echo_grpc):
@@ -28,7 +28,7 @@ def test_unary_stream(intercepted_echo_grpc):
         {
             "content": content,
         },
-        metadata=intercepted_metadata,
+        metadata=_METADATA,
     )
 
     for ground_truth, response in zip(content.split(" "), responses):
@@ -38,7 +38,7 @@ def test_unary_stream(intercepted_echo_grpc):
     response_metadata = [
         (metadata.key, metadata.value) for metadata in responses.trailing_metadata()
     ]
-    assert intercepted_metadata[0] in response_metadata
+    assert _METADATA[0] in response_metadata
     interceptor.response_metadata = response_metadata
 
 
@@ -47,13 +47,13 @@ def test_stream_stream(intercepted_echo_grpc):
     requests = []
     requests.append(showcase.EchoRequest(content="hello"))
     requests.append(showcase.EchoRequest(content="world!"))
-    responses = client.chat(iter(requests), metadata=intercepted_metadata)
+    responses = client.chat(iter(requests), metadata=_METADATA)
 
     contents = [response.content for response in responses]
     assert contents == ["hello", "world!"]
 
     response_metadata = [
         (metadata.key, metadata.value) for metadata in responses.trailing_metadata()
     ]
-    assert intercepted_metadata[0] in response_metadata
+    assert _METADATA[0] in response_metadata
     interceptor.response_metadata = response_metadata

tests/system/test_streams.py

@@ -19,6 +19,9 @@
 from google import showcase
 
 
+# `_METADATA` will be sent as part of the request, and the
+# showcase server will echo it (since it has key 'showcase-trailer') as trailing
+# metadata.
 _METADATA = (("showcase-trailer", "hello world"),)