From 93950177ce54e64d29b3d4793f0a37cc4b0b4cde Mon Sep 17 00:00:00 2001 From: Yunlian Jiang Date: Thu, 10 Dec 2015 13:20:48 -0800 Subject: 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 Tested-by: Yunlian Jiang Reviewed-by: Caroline Tice --- crosperf/machine_manager_unittest.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'crosperf/machine_manager_unittest.py') 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. -- cgit v1.2.3