1 files changed, 45 insertions, 44 deletions

diff --git a/src/python/grpcio_tests/tests/_loader.py b/src/python/grpcio_tests/tests/_loader.py
index 621bedc7bb..42cf9ab4ca 100644
--- a/src/python/grpcio_tests/tests/_loader.py
+++ b/src/python/grpcio_tests/tests/_loader.py
@@ -40,7 +40,7 @@ TEST_MODULE_REGEX = r'^.*_test$'
 
 
 class Loader(object):
-  """Test loader for setuptools test suite support.
+    """Test loader for setuptools test suite support.
 
   Attributes:
     suite (unittest.TestSuite): All tests collected by the loader.
@@ -51,57 +51,57 @@ class Loader(object):
       contributes to the test suite.
   """
 
-  def __init__(self):
-    self.suite = unittest.TestSuite()
-    self.loader = unittest.TestLoader()
-    self.module_matcher = re.compile(TEST_MODULE_REGEX)
+    def __init__(self):
+        self.suite = unittest.TestSuite()
+        self.loader = unittest.TestLoader()
+        self.module_matcher = re.compile(TEST_MODULE_REGEX)
 
-  def loadTestsFromNames(self, names, module=None):
-    """Function mirroring TestLoader::loadTestsFromNames, as expected by
+    def loadTestsFromNames(self, names, module=None):
+        """Function mirroring TestLoader::loadTestsFromNames, as expected by
     setuptools.setup argument `test_loader`."""
-    # ensure that we capture decorators and definitions (else our coverage
-    # measure unnecessarily suffers)
-    coverage_context = coverage.Coverage(data_suffix=True)
-    coverage_context.start()
-    modules = [importlib.import_module(name) for name in names]
-    for module in modules:
-      self.visit_module(module)
-    for module in modules:
-      try:
-        package_paths = module.__path__
-      except:
-        continue
-      self.walk_packages(package_paths)
-    coverage_context.stop()
-    coverage_context.save()
-    return self.suite
-
-  def walk_packages(self, package_paths):
-    """Walks over the packages, dispatching `visit_module` calls.
+        # ensure that we capture decorators and definitions (else our coverage
+        # measure unnecessarily suffers)
+        coverage_context = coverage.Coverage(data_suffix=True)
+        coverage_context.start()
+        modules = [importlib.import_module(name) for name in names]
+        for module in modules:
+            self.visit_module(module)
+        for module in modules:
+            try:
+                package_paths = module.__path__
+            except:
+                continue
+            self.walk_packages(package_paths)
+        coverage_context.stop()
+        coverage_context.save()
+        return self.suite
+
+    def walk_packages(self, package_paths):
+        """Walks over the packages, dispatching `visit_module` calls.
 
     Args:
       package_paths (list): A list of paths over which to walk through modules
         along.
     """
-    for importer, module_name, is_package in (
-        pkgutil.walk_packages(package_paths)):
-      module = importer.find_module(module_name).load_module(module_name)
-      self.visit_module(module)
+        for importer, module_name, is_package in (
+                pkgutil.walk_packages(package_paths)):
+            module = importer.find_module(module_name).load_module(module_name)
+            self.visit_module(module)
 
-  def visit_module(self, module):
-    """Visits the module, adding discovered tests to the test suite.
+    def visit_module(self, module):
+        """Visits the module, adding discovered tests to the test suite.
 
     Args:
       module (module): Module to match against self.module_matcher; if matched
         it has its tests loaded via self.loader into self.suite.
     """
-    if self.module_matcher.match(module.__name__):
-      module_suite = self.loader.loadTestsFromModule(module)
-      self.suite.addTest(module_suite)
+        if self.module_matcher.match(module.__name__):
+            module_suite = self.loader.loadTestsFromModule(module)
+            self.suite.addTest(module_suite)
 
 
 def iterate_suite_cases(suite):
-  """Generator over all unittest.TestCases in a unittest.TestSuite.
+    """Generator over all unittest.TestCases in a unittest.TestSuite.
 
   Args:
     suite (unittest.TestSuite): Suite to iterate over in the generator.
@@ -109,11 +109,12 @@ def iterate_suite_cases(suite):
   Returns:
     generator: A generator over all unittest.TestCases in `suite`.
   """
-  for item in suite:
-    if isinstance(item, unittest.TestSuite):
-      for child_item in iterate_suite_cases(item):
-        yield child_item
-    elif isinstance(item, unittest.TestCase):
-      yield item
-    else:
-      raise ValueError('unexpected suite item of type {}'.format(type(item)))
+    for item in suite:
+        if isinstance(item, unittest.TestSuite):
+            for child_item in iterate_suite_cases(item):
+                yield child_item
+        elif isinstance(item, unittest.TestCase):
+            yield item
+        else:
+            raise ValueError('unexpected suite item of type {}'.format(
+                type(item)))