Fix typos, add some missed args and ensure consistency on where logger is defined.
Bug: None
Test: acloud-dev create --local-image
acloud-dev create --local-image --local-instance
Change-Id: I272cd82d68139e7d6ac27b84fdf725ce9818df3d
diff --git a/internal/lib/gcompute_client_test.py b/internal/lib/gcompute_client_test.py
index df84a87..6dbf8d3 100644
--- a/internal/lib/gcompute_client_test.py
+++ b/internal/lib/gcompute_client_test.py
@@ -30,12 +30,14 @@
from acloud.internal.lib import gcompute_client
from acloud.internal.lib import utils
+
GS_IMAGE_SOURCE_URI = "https://storage.googleapis.com/fake-bucket/fake.tar.gz"
GS_IMAGE_SOURCE_DISK = (
"https://www.googleapis.com/compute/v1/projects/fake-project/zones/"
"us-east1-d/disks/fake-disk")
PROJECT = "fake-project"
+
# pylint: disable=protected-access, too-many-public-methods
class ComputeClientTest(driver_test_lib.BaseDriverTest):
"""Test ComputeClient."""
@@ -583,9 +585,12 @@
@mock.patch.object(gcompute_client.ComputeClient, "GetSubnetworkUrl")
@mock.patch.object(gcompute_client.ComputeClient, "GetMachineType")
@mock.patch.object(gcompute_client.ComputeClient, "WaitOnOperation")
- def testCreateInstanceWithTags(self, mock_wait, mock_get_mach_type,
- mock_get_subnetwork_url, mock_get_network_url,
- mock_get_image):
+ def testCreateInstanceWithTags(self,
+ mock_wait,
+ mock_get_mach_type,
+ mock_get_subnetwork_url,
+ mock_get_network_url,
+ mock_get_image):
"""Test CreateInstance."""
mock_get_mach_type.return_value = {"selfLink": self.MACHINE_TYPE_URL}
mock_get_network_url.return_value = self.NETWORK_URL