commit f529691e12a89bb3dca99bda1b1ba0bd801de873
author Sean McAllister <smcallis@google.com> Thu Sep 23 09:32:58 2021 -0600
committer Commit Bot <commit-bot@chromium.org> Wed Sep 29 15:56:08 2021 +0000
tree 705f8948d6a5981ba810a7ae32b9533052c6e506
parent 17eed8d5f45731ade7ae9dfce2ef45c9049e66ee

test-api: Fix linting errors in test.py

BUG=none
TEST=run_tests

Change-Id: Ic1a60cebebc61ccba596bea26c36f9b7bf4511be
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3187265
Reviewed-by: Alex Klein <saklein@chromium.org>
Tested-by: Sean McAllister <smcallis@google.com>
Commit-Queue: Sean McAllister <smcallis@google.com>

api/controller/test.py

diff --git a/api/controller/test.py b/api/controller/test.py
index 58f0315..62e4dd1 100644
--- a/api/controller/test.py
+++ b/api/controller/test.py
@@ -165,7 +165,7 @@
   """Fake success response"""
   # pylint: disable=unused-argument
   output_proto.results.append(test_pb2.TestServiceContainerBuildResult(
-      success = test_pb2.TestServiceContainerBuildResult.Success()
+      success=test_pb2.TestServiceContainerBuildResult.Success()
   ))
 
 
@@ -175,7 +175,7 @@
 
   # pylint: disable=unused-argument
   output_proto.results.append(test_pb2.TestServiceContainerBuildResult(
-      failure = test_pb2.TestServiceContainerBuildResult.Failure(
+      failure=test_pb2.TestServiceContainerBuildResult.Failure(
           error_message='fake error'
       )
   ))
@@ -193,7 +193,7 @@
   invalid_chars = set(tag) - allowed_chars
   if invalid_chars:
     return 'saw one or more invalid characters: [{}]'.format(
-      ''.join(invalid_chars),
+        ''.join(invalid_chars),
     )
 
   # Finally, max tag length is 128 characters
@@ -215,7 +215,7 @@
   invalid_chars = set(key) - allowed_chars
   if invalid_chars:
     return 'saw one or more invalid characters: [{}]'.format(
-      ''.join(invalid_chars),
+        ''.join(invalid_chars),
     )
 
   # Repeated . and - aren't allowed
@@ -244,12 +244,12 @@
     cmd_result = cros_build_lib.run(cmd, check=False)
     if cmd_result.returncode == 0:
       output_proto.results.append(test_pb2.TestServiceContainerBuildResult(
-          success = test_pb2.TestServiceContainerBuildResult.Success()
+          success=test_pb2.TestServiceContainerBuildResult.Success()
       ))
     else:
       output_proto.results.append(test_pb2.TestServiceContainerBuildResult(
-          failure = test_pb2.TestServiceContainerBuildResult.Failure(
-              error_message = cmd_result.stderr
+          failure=test_pb2.TestServiceContainerBuildResult.Failure(
+              error_message=cmd_result.stderr
           )
       ))
 
@@ -381,9 +381,9 @@
 
 
 def GetArtifacts(in_proto: common_pb2.ArtifactsByService.Test,
-    chroot: chroot_lib.Chroot, sysroot_class: sysroot_lib.Sysroot,
-    build_target: build_target_lib.BuildTarget,
-    output_dir: str) -> list:
+                 chroot: chroot_lib.Chroot, sysroot_class: sysroot_lib.Sysroot,
+                 build_target: build_target_lib.BuildTarget,
+                 output_dir: str) -> list:
   """Builds and copies test artifacts to specified output_dir.
 
   Copies test artifacts to output_dir, returning a list of (output_dir: str)
@@ -402,11 +402,12 @@
   generated = []
 
   artifact_types = {
-    in_proto.ArtifactType.UNIT_TESTS: test.BuildTargetUnitTestTarball,
-    in_proto.ArtifactType.CODE_COVERAGE_LLVM_JSON:
-        test.BundleCodeCoverageLlvmJson,
-    in_proto.ArtifactType.HWQUAL: functools.partial(test.BundleHwqualTarball,
-      build_target.name, packages_service.determine_full_version()),
+      in_proto.ArtifactType.UNIT_TESTS: test.BuildTargetUnitTestTarball,
+      in_proto.ArtifactType.CODE_COVERAGE_LLVM_JSON:
+          test.BundleCodeCoverageLlvmJson,
+      in_proto.ArtifactType.HWQUAL: functools.partial(
+          test.BundleHwqualTarball,
+          build_target.name, packages_service.determine_full_version()),
   }
 
   for output_artifact in in_proto.output_artifacts: