commit ab87cebda9434790247fc2219570190f70cd7c47
author Alex Klein <saklein@chromium.org> Tue Jan 24 12:00:51 2023 -0700
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Jan 25 23:08:12 2023 +0000
tree 96b0f556ef4a1ee8052e21f784091bf348b70342
parent 6e10252ea5b302b747a5c3baa6ced07d5acad051

api: controller: Lint fixes.

Fix line-too-long and docstring-section-indent errors.

BUG=b:233893248
TEST=cros lint

Change-Id: I52ac19eb3a38066d4d09d56d27d61c904269a0e7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4190794
Tested-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Alex Klein <saklein@chromium.org>
Auto-Submit: Alex Klein <saklein@chromium.org>
Reviewed-by: Cindy Lin <xcl@google.com>

api/controller/toolchain_unittest.py

diff --git a/api/controller/toolchain_unittest.py b/api/controller/toolchain_unittest.py
index 25adec0..17dc677 100644
--- a/api/controller/toolchain_unittest.py
+++ b/api/controller/toolchain_unittest.py
@@ -65,8 +65,10 @@
             toolchain,
             "_TOOLCHAIN_ARTIFACT_HANDLERS",
             {
-                BuilderConfig.Artifacts.UNVERIFIED_CHROME_LLVM_ORDERFILE: toolchain._Handlers(
-                    "UnverifiedChromeLlvmOrderfile", self.prep, self.bundle
+                BuilderConfig.Artifacts.UNVERIFIED_CHROME_LLVM_ORDERFILE: (
+                    toolchain._Handlers(
+                        "UnverifiedChromeLlvmOrderfile", self.prep, self.bundle
+                    )
                 ),
             },
         )
@@ -130,6 +132,7 @@
         )
 
     def testPassesProfileInfo(self):
+        # pylint: disable=line-too-long
         request = toolchain_pb2.PrepareForToolchainBuildRequest(
             artifact_types=[
                 BuilderConfig.Artifacts.UNVERIFIED_CHROME_LLVM_ORDERFILE
@@ -150,6 +153,7 @@
                 chrome_cwp_profile="CWPVERSION"
             ),
         )
+        # pylint: enable=line-too-long
         toolchain.PrepareForBuild(request, self.response, self.api_config)
         self.prep.assert_called_once_with(
             "UnverifiedChromeLlvmOrderfile",
@@ -167,6 +171,7 @@
         )
 
     def testPassesProfileInfoAfdoRelease(self):
+        # pylint: disable=line-too-long
         request = toolchain_pb2.PrepareForToolchainBuildRequest(
             artifact_types=[
                 BuilderConfig.Artifacts.UNVERIFIED_CHROME_LLVM_ORDERFILE
@@ -189,6 +194,7 @@
                 )
             ),
         )
+        # pylint: enable=line-too-long
         toolchain.PrepareForBuild(request, self.response, self.api_config)
         self.prep.assert_called_once_with(
             "UnverifiedChromeLlvmOrderfile",
@@ -206,6 +212,7 @@
         )
 
     def testHandlesDuplicateInputArtifacts(self):
+        # pylint: disable=line-too-long
         request = toolchain_pb2.PrepareForToolchainBuildRequest(
             artifact_types=[
                 BuilderConfig.Artifacts.UNVERIFIED_CHROME_LLVM_ORDERFILE
@@ -223,6 +230,7 @@
                 ),
             ],
         )
+        # pylint: enable=line-too-long
         toolchain.PrepareForBuild(request, self.response, self.api_config)
         self.prep.assert_called_once_with(
             "UnverifiedChromeLlvmOrderfile",
@@ -259,8 +267,10 @@
             toolchain,
             "_TOOLCHAIN_ARTIFACT_HANDLERS",
             {
-                BuilderConfig.Artifacts.UNVERIFIED_CHROME_LLVM_ORDERFILE: toolchain._Handlers(
-                    "UnverifiedChromeLlvmOrderfile", self.prep, self.bundle
+                BuilderConfig.Artifacts.UNVERIFIED_CHROME_LLVM_ORDERFILE: (
+                    toolchain._Handlers(
+                        "UnverifiedChromeLlvmOrderfile", self.prep, self.bundle
+                    )
                 ),
             },
         )
@@ -411,7 +421,7 @@
         return toolchain_pb2.ToolchainsRequest(board=board)
 
     def testValidateOnly(self):
-        """Confidence check that a validate only call does not execute any logic."""
+        """Verify a validate-only call does not execute any logic."""
         request = self._GetRequest()
         toolchain.GetToolchainsForBoard(
             request, self.response, self.validate_only_config