diff options
author | Yunlian Jiang <yunlian@chromium.org> | 2015-12-29 08:46:50 -0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2015-12-29 18:53:51 +0000 |
commit | 26d821d63ace64a4ea72ff1b7ad262b232f2e607 (patch) | |
tree | d214c009e32fc3caf0f06b70896d397ff75a367f /crosperf/download_images_unittest.py | |
parent | 054a1230c49dc49d09c015ed83f9aeb3a96b87da (diff) | |
download | toolchain-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-x | crosperf/download_images_unittest.py | 16 |
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) |