diff options
| author | Rahul Chaudhry <rahulchaudhry@chromium.org> | 2016-02-11 14:07:18 -0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2016-02-13 01:20:39 +0000 |
| commit | 8997f2f67506d42e005fc62383026f2da1dca31d (patch) | |
| tree | b4454fbfed38ba3ff48cc2f36a68059d1286d74e /crosperf | |
| parent | 906dc58573d86c64aedc290864d0bc84302265bd (diff) | |
| download | toolchain-utils-8997f2f67506d42e005fc62383026f2da1dca31d.tar.gz | |
toolchain-utils: fix bare-except lint warning.
"using a bare except: is almost never a good idea"
- https://docs.python.org/2/howto/doanddont.html#except
BUG=chromium:570458
TEST='crosperf/run_tests.sh' passes.
Change-Id: I2222fbc0fdc116961388c67b51ee259b49d0be88
Reviewed-on: https://chrome-internal-review.googlesource.com/248233
Commit-Ready: Rahul Chaudhry <rahulchaudhry@google.com>
Tested-by: Rahul Chaudhry <rahulchaudhry@google.com>
Reviewed-by: Caroline Tice <cmtice@google.com>
Diffstat (limited to 'crosperf')
| -rwxr-xr-x | crosperf/machine_manager_unittest.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crosperf/machine_manager_unittest.py b/crosperf/machine_manager_unittest.py index c87140e8..321b5878 100755 --- a/crosperf/machine_manager_unittest.py +++ b/crosperf/machine_manager_unittest.py @@ -194,7 +194,7 @@ class MachineManagerTest(unittest.TestCase): mock_run_cmd.return_value = 1 try: self.mm.ImageMachine(machine, LABEL_LUMPY) - except: + except Exception: self.assertEqual(mock_checksummer.call_count, 0) self.assertEqual(mock_run_cmd.call_count, 2) self.assertEqual(mock_run_croscmd.call_count, 1) |