1 files changed, 164 insertions, 152 deletions
diff --git a/src/python/grpcio_tests/tests/unit/_invocation_defects_test.py b/src/python/grpcio_tests/tests/unit/_invocation_defects_test.py
index 4312679bb9..efeb237874 100644
--- a/src/python/grpcio_tests/tests/unit/_invocation_defects_test.py
+++ b/src/python/grpcio_tests/tests/unit/_invocation_defects_test.py
@@ -50,106 +50,117 @@ _STREAM_STREAM = '/test/StreamStream'
 
 
 class _Callback(object):
-  def __init__(self):
-    self._condition = threading.Condition()
-    self._value = None
-    self._called = False
 
-  def __call__(self, value):
-    with self._condition:
-      self._value = value
-      self._called = True
-      self._condition.notify_all()
+    def __init__(self):
+        self._condition = threading.Condition()
+        self._value = None
+        self._called = False
 
-  def value(self):
-    with self._condition:
-      while not self._called:
-        self._condition.wait()
-      return self._value
+    def __call__(self, value):
+        with self._condition:
+            self._value = value
+            self._called = True
+            self._condition.notify_all()
+
+    def value(self):
+        with self._condition:
+            while not self._called:
+                self._condition.wait()
+            return self._value
 
 
 class _Handler(object):
-  def __init__(self, control):
-    self._control = control
-
-  def handle_unary_unary(self, request, servicer_context):
-    self._control.control()
-    if servicer_context is not None:
-      servicer_context.set_trailing_metadata((('testkey', 'testvalue',),))
-    return request
-
-  def handle_unary_stream(self, request, servicer_context):
-    for _ in range(test_constants.STREAM_LENGTH):
-      self._control.control()
-      yield request
-    self._control.control()
-    if servicer_context is not None:
-      servicer_context.set_trailing_metadata((('testkey', 'testvalue',),))
-
-  def handle_stream_unary(self, request_iterator, servicer_context):
-    if servicer_context is not None:
-      servicer_context.invocation_metadata()
-    self._control.control()
-    response_elements = []
-    for request in request_iterator:
-      self._control.control()
-      response_elements.append(request)
-    self._control.control()
-    if servicer_context is not None:
-      servicer_context.set_trailing_metadata((('testkey', 'testvalue',),))
-    return b''.join(response_elements)
-
-  def handle_stream_stream(self, request_iterator, servicer_context):
-    self._control.control()
-    if servicer_context is not None:
-      servicer_context.set_trailing_metadata((('testkey', 'testvalue',),))
-    for request in request_iterator:
-      self._control.control()
-      yield request
-    self._control.control()
+
+    def __init__(self, control):
+        self._control = control
+
+    def handle_unary_unary(self, request, servicer_context):
+        self._control.control()
+        if servicer_context is not None:
+            servicer_context.set_trailing_metadata(((
+                'testkey',
+                'testvalue',),))
+        return request
+
+    def handle_unary_stream(self, request, servicer_context):
+        for _ in range(test_constants.STREAM_LENGTH):
+            self._control.control()
+            yield request
+        self._control.control()
+        if servicer_context is not None:
+            servicer_context.set_trailing_metadata(((
+                'testkey',
+                'testvalue',),))
+
+    def handle_stream_unary(self, request_iterator, servicer_context):
+        if servicer_context is not None:
+            servicer_context.invocation_metadata()
+        self._control.control()
+        response_elements = []
+        for request in request_iterator:
+            self._control.control()
+            response_elements.append(request)
+        self._control.control()
+        if servicer_context is not None:
+            servicer_context.set_trailing_metadata(((
+                'testkey',
+                'testvalue',),))
+        return b''.join(response_elements)
+
+    def handle_stream_stream(self, request_iterator, servicer_context):
+        self._control.control()
+        if servicer_context is not None:
+            servicer_context.set_trailing_metadata(((
+                'testkey',
+                'testvalue',),))
+        for request in request_iterator:
+            self._control.control()
+            yield request
+        self._control.control()
 
 
 class _MethodHandler(grpc.RpcMethodHandler):
-  def __init__(
-    self, request_streaming, response_streaming, request_deserializer,
-    response_serializer, unary_unary, unary_stream, stream_unary,
-    stream_stream):
-    self.request_streaming = request_streaming
-    self.response_streaming = response_streaming
-    self.request_deserializer = request_deserializer
-    self.response_serializer = response_serializer
-    self.unary_unary = unary_unary
-    self.unary_stream = unary_stream
-    self.stream_unary = stream_unary
-    self.stream_stream = stream_stream
+
+    def __init__(self, request_streaming, response_streaming,
+                 request_deserializer, response_serializer, unary_unary,
+                 unary_stream, stream_unary, stream_stream):
+        self.request_streaming = request_streaming
+        self.response_streaming = response_streaming
+        self.request_deserializer = request_deserializer
+        self.response_serializer = response_serializer
+        self.unary_unary = unary_unary
+        self.unary_stream = unary_stream
+        self.stream_unary = stream_unary
+        self.stream_stream = stream_stream
 
 
 class _GenericHandler(grpc.GenericRpcHandler):
-  def __init__(self, handler):
-    self._handler = handler
-
-  def service(self, handler_call_details):
-    if handler_call_details.method == _UNARY_UNARY:
-      return _MethodHandler(
-        False, False, None, None, self._handler.handle_unary_unary, None,
-        None, None)
-    elif handler_call_details.method == _UNARY_STREAM:
-      return _MethodHandler(
-        False, True, _DESERIALIZE_REQUEST, _SERIALIZE_RESPONSE, None,
-        self._handler.handle_unary_stream, None, None)
-    elif handler_call_details.method == _STREAM_UNARY:
-      return _MethodHandler(
-        True, False, _DESERIALIZE_REQUEST, _SERIALIZE_RESPONSE, None, None,
-        self._handler.handle_stream_unary, None)
-    elif handler_call_details.method == _STREAM_STREAM:
-      return _MethodHandler(
-        True, True, None, None, None, None, None,
-        self._handler.handle_stream_stream)
-    else:
-      return None
+
+    def __init__(self, handler):
+        self._handler = handler
+
+    def service(self, handler_call_details):
+        if handler_call_details.method == _UNARY_UNARY:
+            return _MethodHandler(False, False, None, None,
+                                  self._handler.handle_unary_unary, None, None,
+                                  None)
+        elif handler_call_details.method == _UNARY_STREAM:
+            return _MethodHandler(False, True, _DESERIALIZE_REQUEST,
+                                  _SERIALIZE_RESPONSE, None,
+                                  self._handler.handle_unary_stream, None, None)
+        elif handler_call_details.method == _STREAM_UNARY:
+            return _MethodHandler(True, False, _DESERIALIZE_REQUEST,
+                                  _SERIALIZE_RESPONSE, None, None,
+                                  self._handler.handle_stream_unary, None)
+        elif handler_call_details.method == _STREAM_STREAM:
+            return _MethodHandler(True, True, None, None, None, None, None,
+                                  self._handler.handle_stream_stream)
+        else:
+            return None
 
 
 class FailAfterFewIterationsCounter(object):
+
     def __init__(self, high, bytestring):
         self._current = 0
         self._high = high
@@ -167,81 +178,82 @@ class FailAfterFewIterationsCounter(object):
 
 
 def _unary_unary_multi_callable(channel):
-  return channel.unary_unary(_UNARY_UNARY)
+    return channel.unary_unary(_UNARY_UNARY)
 
 
 def _unary_stream_multi_callable(channel):
-  return channel.unary_stream(
-    _UNARY_STREAM,
-    request_serializer=_SERIALIZE_REQUEST,
-    response_deserializer=_DESERIALIZE_RESPONSE)
+    return channel.unary_stream(
+        _UNARY_STREAM,
+        request_serializer=_SERIALIZE_REQUEST,
+        response_deserializer=_DESERIALIZE_RESPONSE)
 
 
 def _stream_unary_multi_callable(channel):
-  return channel.stream_unary(
-    _STREAM_UNARY,
-    request_serializer=_SERIALIZE_REQUEST,
-    response_deserializer=_DESERIALIZE_RESPONSE)
+    return channel.stream_unary(
+        _STREAM_UNARY,
+        request_serializer=_SERIALIZE_REQUEST,
+        response_deserializer=_DESERIALIZE_RESPONSE)
 
 
 def _stream_stream_multi_callable(channel):
-  return channel.stream_stream(_STREAM_STREAM)
+    return channel.stream_stream(_STREAM_STREAM)
 
 
 class InvocationDefectsTest(unittest.TestCase):
-  def setUp(self):
-    self._control = test_control.PauseFailControl()
-    self._handler = _Handler(self._control)
-    self._server_pool = logging_pool.pool(test_constants.THREAD_CONCURRENCY)
-
-    self._server = grpc.server(self._server_pool)
-    port = self._server.add_insecure_port('[::]:0')
-    self._server.add_generic_rpc_handlers((_GenericHandler(self._handler),))
-    self._server.start()
-
-    self._channel = grpc.insecure_channel('localhost:%d' % port)
-
-  def tearDown(self):
-    self._server.stop(0)
-
-  def testIterableStreamRequestBlockingUnaryResponse(self):
-    requests = [b'\x07\x08' for _ in range(test_constants.STREAM_LENGTH)]
-    multi_callable = _stream_unary_multi_callable(self._channel)
-
-    with self.assertRaises(grpc.RpcError):
-      response = multi_callable(
-        requests,
-        metadata=(('test', 'IterableStreamRequestBlockingUnaryResponse'),))
-
-  def testIterableStreamRequestFutureUnaryResponse(self):
-    requests = [b'\x07\x08' for _ in range(test_constants.STREAM_LENGTH)]
-    multi_callable = _stream_unary_multi_callable(self._channel)
-    response_future = multi_callable.future(
-      requests,
-      metadata=(
-        ('test', 'IterableStreamRequestFutureUnaryResponse'),))
-
-    with self.assertRaises(grpc.RpcError):
-      response = response_future.result()
-
-  def testIterableStreamRequestStreamResponse(self):
-    requests = [b'\x77\x58' for _ in range(test_constants.STREAM_LENGTH)]
-    multi_callable = _stream_stream_multi_callable(self._channel)
-    response_iterator = multi_callable(
-      requests,
-      metadata=(('test', 'IterableStreamRequestStreamResponse'),))
-
-    with self.assertRaises(grpc.RpcError):
-      next(response_iterator)
-
-  def testIteratorStreamRequestStreamResponse(self):
-    requests_iterator = FailAfterFewIterationsCounter(
-      test_constants.STREAM_LENGTH // 2, b'\x07\x08')
-    multi_callable = _stream_stream_multi_callable(self._channel)
-    response_iterator = multi_callable(
-      requests_iterator,
-      metadata=(('test', 'IteratorStreamRequestStreamResponse'),))
-
-    with self.assertRaises(grpc.RpcError):
-      for _ in range(test_constants.STREAM_LENGTH // 2 + 1):
-        next(response_iterator)
+
+    def setUp(self):
+        self._control = test_control.PauseFailControl()
+        self._handler = _Handler(self._control)
+        self._server_pool = logging_pool.pool(test_constants.THREAD_CONCURRENCY)
+
+        self._server = grpc.server(self._server_pool)
+        port = self._server.add_insecure_port('[::]:0')
+        self._server.add_generic_rpc_handlers((_GenericHandler(self._handler),))
+        self._server.start()
+
+        self._channel = grpc.insecure_channel('localhost:%d' % port)
+
+    def tearDown(self):
+        self._server.stop(0)
+
+    def testIterableStreamRequestBlockingUnaryResponse(self):
+        requests = [b'\x07\x08' for _ in range(test_constants.STREAM_LENGTH)]
+        multi_callable = _stream_unary_multi_callable(self._channel)
+
+        with self.assertRaises(grpc.RpcError):
+            response = multi_callable(
+                requests,
+                metadata=(
+                    ('test', 'IterableStreamRequestBlockingUnaryResponse'),))
+
+    def testIterableStreamRequestFutureUnaryResponse(self):
+        requests = [b'\x07\x08' for _ in range(test_constants.STREAM_LENGTH)]
+        multi_callable = _stream_unary_multi_callable(self._channel)
+        response_future = multi_callable.future(
+            requests,
+            metadata=(('test', 'IterableStreamRequestFutureUnaryResponse'),))
+
+        with self.assertRaises(grpc.RpcError):
+            response = response_future.result()
+
+    def testIterableStreamRequestStreamResponse(self):
+        requests = [b'\x77\x58' for _ in range(test_constants.STREAM_LENGTH)]
+        multi_callable = _stream_stream_multi_callable(self._channel)
+        response_iterator = multi_callable(
+            requests,
+            metadata=(('test', 'IterableStreamRequestStreamResponse'),))
+
+        with self.assertRaises(grpc.RpcError):
+            next(response_iterator)
+
+    def testIteratorStreamRequestStreamResponse(self):
+        requests_iterator = FailAfterFewIterationsCounter(
+            test_constants.STREAM_LENGTH // 2, b'\x07\x08')
+        multi_callable = _stream_stream_multi_callable(self._channel)
+        response_iterator = multi_callable(
+            requests_iterator,
+            metadata=(('test', 'IteratorStreamRequestStreamResponse'),))
+
+        with self.assertRaises(grpc.RpcError):
+            for _ in range(test_constants.STREAM_LENGTH // 2 + 1):
+                next(response_iterator)