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/image_unittest.py b/api/controller/image_unittest.py
index 4641c08..c49498d 100644
--- a/api/controller/image_unittest.py
+++ b/api/controller/image_unittest.py
@@ -53,7 +53,7 @@
)
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."""
patch = self.PatchObject(image_service, "Build")
request = self._GetRequest(board="board")
@@ -63,7 +63,7 @@
patch.assert_not_called()
def testMockCall(self):
- """Test that mock call does not execute any logic, returns mocked value."""
+ """Test mock call does not execute any logic, returns mocked value."""
patch = self.PatchObject(image_service, "Build")
request = self._GetRequest(board="board")
@@ -232,6 +232,7 @@
):
"""GetArtifacts function tests."""
+ # pylint: disable=line-too-long
_artifact_funcs = {
common_pb2.ArtifactsByService.Image.ArtifactType.DLC_IMAGE: image_service.copy_dlc_image,
common_pb2.ArtifactsByService.Image.ArtifactType.LICENSE_CREDITS: image_service.copy_license_credits,
@@ -239,6 +240,7 @@
common_pb2.ArtifactsByService.Image.ArtifactType.STRIPPED_PACKAGES: image_service.create_stripped_packages_tar,
common_pb2.ArtifactsByService.Image.ArtifactType.IMAGE_SCRIPTS: image_service.create_image_scripts_archive,
}
+ # pylint: enable=line-too-long
def setUp(self):
self._mocks = {}
@@ -291,7 +293,7 @@
patch.assert_called_once()
def testArtifactsException(self):
- """Test GetArtifacts with all artifact types when one type throws an exception."""
+ """Test with all artifact types when one type throws an exception."""
self._mocks[
common_pb2.ArtifactsByService.Image.ArtifactType.STRIPPED_PACKAGES
@@ -384,8 +386,8 @@
"""Helper to create Mock build_image results.
Args:
- image_types: A list of image types for which the mock BuildResult has to
- be creates.
+ image_types: A list of image types for which the mock BuildResult
+ has to be created.
Returns:
A list of mock BuildResult.
@@ -468,7 +470,7 @@
patch.assert_not_called()
def testMockCall(self):
- """Test that mock call does not execute any logic, returns mocked value."""
+ """Test mock call does not execute any logic, returns mocked value."""
patch = self.PatchObject(image_lib, "SecurityTest", return_value=True)
input_proto = image_pb2.TestImageRequest()
input_proto.image.path = self.image_path
@@ -540,7 +542,7 @@
osutils.Touch(self.image_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."""
patch = self.PatchObject(image_service, "Test")
input_proto = image_pb2.TestImageRequest()
@@ -555,7 +557,7 @@
patch.assert_not_called()
def testMockCall(self):
- """Test that mock call does not execute any logic, returns mocked value."""
+ """Test mock call does not execute any logic, returns mocked value."""
patch = self.PatchObject(image_service, "Test")
input_proto = image_pb2.TestImageRequest()
@@ -690,7 +692,7 @@
@mock.patch.object(pushimage, "PushImage", return_value={})
def testMockCall(self, MockPushImage):
- """Test that mock call does not execute any logic, returns mocked value."""
+ """Test mock call does not execute any logic, returns mocked value."""
rc = image_controller.PushImage(
self._GetRequest(), self._GetResponse(), self.mock_call_config
)