aboutsummaryrefslogtreecommitdiff
path: root/crosperf/machine_manager_unittest.py
diff options
context:
space:
mode:
authorYunlian Jiang <yunlian@google.com>2015-12-10 13:20:48 -0800
committerchrome-bot <chrome-bot@chromium.org>2015-12-11 00:07:31 +0000
commit93950177ce54e64d29b3d4793f0a37cc4b0b4cde (patch)
treef2ef48b064872d8f1271907e61d9d0b4994dbd96 /crosperf/machine_manager_unittest.py
parenta19dc5530d982f07cf829ac954c420654587f5cd (diff)
downloadtoolchain-utils-93950177ce54e64d29b3d4793f0a37cc4b0b4cde.tar.gz
crosperf: clean lint warnings.
BUG=chromium:567921 TEST=machine_manager.py is clean. Change-Id: Ia3355ee78d15419ca531f358812f7882a64a6cbe Reviewed-on: https://chrome-internal-review.googlesource.com/241491 Commit-Ready: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com> Reviewed-by: Caroline Tice <cmtice@google.com>
Diffstat (limited to 'crosperf/machine_manager_unittest.py')
-rwxr-xr-xcrosperf/machine_manager_unittest.py10
1 files changed, 5 insertions, 5 deletions
diff --git a/crosperf/machine_manager_unittest.py b/crosperf/machine_manager_unittest.py
index 034d63e2..281cfa2d 100755
--- a/crosperf/machine_manager_unittest.py
+++ b/crosperf/machine_manager_unittest.py
@@ -347,7 +347,7 @@ class MachineManagerTest(unittest.TestCase):
self.mock_daisy2.checksum = "fake_md5_checksum"
# Test 1. Basic test. Acquire lumpy3.
- m = self.mm.AcquireMachine(LABEL_LUMPY.chromeos_image, LABEL_LUMPY, True)
+ m = self.mm.AcquireMachine(LABEL_LUMPY)
self.assertEqual(m, self.mock_lumpy3)
self.assertTrue(self.mock_lumpy3.locked)
self.assertEqual(mock_checksum.call_count, 1)
@@ -358,7 +358,7 @@ class MachineManagerTest(unittest.TestCase):
# Test 3. Not all machines are the same, throw = False
self.assertEqual(len(self.log_fatal_msgs), 0)
- m = self.mm.AcquireMachine(LABEL_MIX.chromeos_image, LABEL_MIX, False)
+ m = self.mm.AcquireMachine(LABEL_MIX)
self.assertEqual(len(self.log_fatal_msgs), 1)
self.assertEqual(self.log_fatal_msgs[0],
'-- not all the machines are identical')
@@ -367,7 +367,7 @@ class MachineManagerTest(unittest.TestCase):
save_locked = self.mock_lumpy1.locked
self.mock_lumpy1.locked = False
self.mock_lumpy1.checksum = None
- m = self.mm.AcquireMachine(LABEL_LUMPY.chromeos_image, LABEL_LUMPY, True)
+ m = self.mm.AcquireMachine(LABEL_LUMPY)
self.assertEqual(m, self.mock_lumpy1)
self.assertTrue(self.mock_lumpy1.locked)
@@ -378,12 +378,12 @@ class MachineManagerTest(unittest.TestCase):
self.mock_lumpy1.locked = False
self.mock_lumpy1.checksum = "123"
self.mock_lumpy1.released_time = time.time() - 8
- m = self.mm.AcquireMachine(LABEL_LUMPY.chromeos_image, LABEL_LUMPY, True)
+ m = self.mm.AcquireMachine(LABEL_LUMPY)
self.assertEqual(m, self.mock_lumpy1)
self.assertTrue(self.mock_lumpy1.locked)
# Test all machines are already locked.
- m = self.mm.AcquireMachine(LABEL_LUMPY.chromeos_image, LABEL_LUMPY, True)
+ m = self.mm.AcquireMachine(LABEL_LUMPY)
self.assertIsNone(m)
# Restore values of mock_lumpy1, so other tests succeed.