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>
diff --git a/api/controller/firmware_unittest.py b/api/controller/firmware_unittest.py
index 93c5911..8df5dae 100644
--- a/api/controller/firmware_unittest.py
+++ b/api/controller/firmware_unittest.py
@@ -45,7 +45,7 @@
return proto
def testBuildAllFirmware(self):
- """Test invocation of endpoint by verifying call to cros_build_lib.run."""
+ """Test endpoint by verifying call to cros_build_lib.run."""
for fw_loc in common_pb2.FwLocation.values():
fw_path = firmware.get_fw_loc(fw_loc)
if not fw_path:
@@ -56,15 +56,15 @@
code_coverage=True,
)
# TODO(mmortensen): Consider refactoring firmware._call_entry code
- # (putting the parsing of the output file in a function) so that we don't
- # have to mock something as generic as 'json_format.Parse' to avoid an
- # error on parsing an empty(due to mock call) file.
+ # (putting the parsing of the output file in a function) so that we
+ # don't have to mock something as generic as 'json_format.Parse' to
+ # avoid an error on parsing an empty(due to mock call) file.
json_format_patch = self.PatchObject(json_format, "Parse")
response = firmware_pb2.BuildAllFirmwareResponse()
# Call the method under test.
firmware.BuildAllFirmware(request, response, self.api_config)
- # Because we mock out the function, we verify that it is called as we
- # expect it to be called.
+ # Because we mock out the function, we verify that it is called as
+ # we expect it to be called.
called_function = os.path.join(
constants.SOURCE_ROOT, fw_path, "firmware_builder.py"
)
@@ -82,7 +82,7 @@
json_format_patch.assert_called()
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."""
for fw_loc in common_pb2.FwLocation.values():
if not firmware.get_fw_loc(fw_loc):
continue
@@ -98,7 +98,7 @@
self.cros_build_run_patch.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."""
for fw_loc in common_pb2.FwLocation.values():
if not firmware.get_fw_loc(fw_loc):
continue