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/android_unittest.py

diff --git a/api/controller/android_unittest.py b/api/controller/android_unittest.py
index 97fd6de..62b60e4 100644
--- a/api/controller/android_unittest.py
+++ b/api/controller/android_unittest.py
@@ -44,14 +44,14 @@
         self._mock.assert_not_called()
 
     def testMockCall(self):
-        """Test that a mock call does not execute logic, returns mocked value."""
+        """Test a mock call does not execute logic, returns mocked value."""
         req = self._GetRequest(android_package="android-package")
         android.GetLatestBuild(req, self._output_proto, self.mock_call_config)
         self._mock.assert_not_called()
         self.assertEqual(self._output_proto.android_version, "7123456")
 
     def testFailsIfBranchAndPackageMissing(self):
-        """Fails if both android_build_branch and android_package are missing."""
+        """Fails if android_build_branch and android_package are missing."""
         req = self._GetRequest()
         with self.assertRaises(cros_build_lib.DieSystemExit):
             android.GetLatestBuild(req, self._output_proto, self.api_config)
@@ -105,14 +105,14 @@
         self.response = android_pb2.MarkStableResponse()
 
     def testValidateOnly(self):
-        """Sanity check that a validate only call does not execute any logic."""
+        """Verify a validate-only call does not execute any logic."""
         android.MarkStable(
             self.input_proto, self.response, self.validate_only_config
         )
         self.uprev.assert_not_called()
 
     def testMockCall(self):
-        """Test that a mock call does not execute logic, returns mocked value."""
+        """Test a mock call does not execute logic, returns mocked value."""
         android.MarkStable(
             self.input_proto, self.response, self.mock_call_config
         )
@@ -204,13 +204,13 @@
         safeunlink = self.PatchObject(osutils, "SafeUnlink")
         self.PatchObject(constants, "_FindSourceRoot", return_value="SRCROOT")
 
-        # This has the side effect of making sure that input and output proto are
-        # not actually used.
+        # This has the side effect of making sure that input and output proto
+        # are not actually used.
         android.UnpinVersion(None, None, self.api_config)
         safeunlink.assert_called_once_with(android.ANDROIDPIN_MASK_PATH)
 
     def testValidateOnly(self):
-        """Sanity check that a validate only call does not execute any logic."""
+        """Verify a validate-only call does not execute any logic."""
         safeunlink = self.PatchObject(osutils, "SafeUnlink")
 
         android.UnpinVersion(None, None, self.validate_only_config)
@@ -250,7 +250,7 @@
         mock_write_lkgb.assert_not_called()
 
     def testMockCall(self):
-        """Test that a mock call does not execute logic, returns mocked value."""
+        """Test a mock call does not execute logic, returns mocked value."""
         mock_write_lkgb = self.PatchObject(service_android, "WriteLKGB")
 
         req = android_pb2.WriteLKGBRequest(