Index: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/port_testcase.py |
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/port_testcase.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/port_testcase.py |
index 9ec7f6d59a5e400cc39ec62680a86a7983607117..7c081326d34922807b85bdfbef7ac1be83067da6 100644 |
--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/port_testcase.py |
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/port_testcase.py |
@@ -114,7 +114,7 @@ class PortTestCase(unittest.TestCase): |
self.assertEqual(port.check_build(needs_http=True, printer=FakePrinter()), |
test_run_results.OK_EXIT_STATUS) |
finally: |
- out, err, logs = oc.restore_output() |
+ _, _, logs = oc.restore_output() |
self.assertIn('pretty patches', logs) # We should get a warning about PrettyPatch being missing, |
self.assertNotIn('build requirements', logs) # but not the driver itself. |
@@ -125,7 +125,7 @@ class PortTestCase(unittest.TestCase): |
self.assertEqual(port.check_build(needs_http=True, printer=FakePrinter()), |
test_run_results.UNEXPECTED_ERROR_EXIT_STATUS) |
finally: |
- out, err, logs = oc.restore_output() |
+ _, _, logs = oc.restore_output() |
self.assertIn('pretty patches', logs) # And, here we should get warnings about both. |
self.assertIn('build requirements', logs) |
@@ -182,7 +182,7 @@ class PortTestCase(unittest.TestCase): |
try: |
test_socket = socket.socket() |
test_socket.connect((host, port)) |
- except IOError as e: |
+ except IOError: |
self.fail('failed to connect to %s:%d' % (host, port)) |
finally: |
test_socket.close() |
@@ -231,7 +231,7 @@ class PortTestCase(unittest.TestCase): |
exception_raised = False |
try: |
port.diff_image("EXPECTED", "ACTUAL") |
- except ValueError as e: |
+ except ValueError: |
exception_raised = True |
self.assertFalse(exception_raised) |