aboutsummaryrefslogtreecommitdiff
path: root/crosperf/download_images_unittest.py
diff options
context:
space:
mode:
authorYunlian Jiang <yunlian@chromium.org>2015-12-29 08:46:50 -0800
committerchrome-bot <chrome-bot@chromium.org>2015-12-29 18:53:51 +0000
commit26d821d63ace64a4ea72ff1b7ad262b232f2e607 (patch)
treed214c009e32fc3caf0f06b70896d397ff75a367f /crosperf/download_images_unittest.py
parent054a1230c49dc49d09c015ed83f9aeb3a96b87da (diff)
downloadtoolchain-utils-26d821d63ace64a4ea72ff1b7ad262b232f2e607.tar.gz
crosperf: fix lint warnings.
BUG=chromium:570454 TEST=unittest passes, lint warning is gone. Change-Id: I4801d560990ad2ed20e30d5cefd5f1ec2c6c5a3d Reviewed-on: https://chrome-internal-review.googlesource.com/243408 Commit-Ready: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com> Reviewed-by: Yunlian Jiang <yunlian@google.com>
Diffstat (limited to 'crosperf/download_images_unittest.py')
-rwxr-xr-xcrosperf/download_images_unittest.py16
1 files changed, 8 insertions, 8 deletions
diff --git a/crosperf/download_images_unittest.py b/crosperf/download_images_unittest.py
index 2cb79a15..1a885c61 100755
--- a/crosperf/download_images_unittest.py
+++ b/crosperf/download_images_unittest.py
@@ -45,7 +45,7 @@ class ImageDownloaderTestcast(unittest.TestCase):
# Set os.path.exists to always return False and run downloader
mock_path_exists.return_value = False
test_flag.SetTestMode(True)
- downloader._DownloadImage(test_chroot, test_build_id, image_path)
+ downloader.DownloadImage(test_chroot, test_build_id, image_path)
# Verify os.path.exists was called twice, with proper arguments.
self.assertEqual(mock_path_exists.call_count, 2)
@@ -75,7 +75,7 @@ class ImageDownloaderTestcast(unittest.TestCase):
mock_path_exists.return_value = True
# Run downloader
- downloader._DownloadImage(test_chroot, test_build_id, image_path)
+ downloader.DownloadImage(test_chroot, test_build_id, image_path)
# Verify os.path.exists was called twice, with proper arguments.
self.assertEqual(mock_path_exists.call_count, 2)
@@ -103,7 +103,7 @@ class ImageDownloaderTestcast(unittest.TestCase):
# Set os.path.exists to always return False and run uncompress.
mock_path_exists.return_value = False
- downloader._UncompressImage(test_chroot, test_build_id)
+ downloader.UncompressImage(test_chroot, test_build_id)
# Verify os.path.exists was called once, with correct arguments.
self.assertEqual(mock_path_exists.call_count, 1)
@@ -122,7 +122,7 @@ class ImageDownloaderTestcast(unittest.TestCase):
mock_path_exists.reset_mock()
mock_cmd_exec.reset_mock()
mock_path_exists.return_value = True
- downloader._UncompressImage(test_chroot, test_build_id)
+ downloader.UncompressImage(test_chroot, test_build_id)
# Verify os.path.exists was called once, with correct arguments.
self.assertEqual(mock_path_exists.call_count, 1)
@@ -171,9 +171,9 @@ class ImageDownloaderTestcast(unittest.TestCase):
downloader = download_images.ImageDownloader(logger_to_use=MOCK_LOGGER)
# Set downloader to call fake stubs.
- downloader._GetBuildID = FakeGetBuildID
- downloader._UncompressImage = FakeUncompressImage
- downloader._DownloadImage = GoodDownloadImage
+ downloader.GetBuildID = FakeGetBuildID
+ downloader.UncompressImage = FakeUncompressImage
+ downloader.DownloadImage = GoodDownloadImage
# Call Run.
downloader.Run(test_chroot, test_build_id)
@@ -185,7 +185,7 @@ class ImageDownloaderTestcast(unittest.TestCase):
# Reset values; Now use fake stub that simulates DownloadImage failing.
self.called_download_image = False
self.called_uncompress_image = False
- downloader._DownloadImage = BadDownloadImage
+ downloader.DownloadImage = BadDownloadImage
# Call Run again.
downloader.Run(test_chroot, test_build_id)