Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1259)

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/common/net/network_transaction_unittest.py

Issue 2578213005: Use underscores to separate words in filenames in webkitpy. (Closed)
Patch Set: Fix check for attribute in output_capture.py. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Tools/Scripts/webkitpy/common/net/network_transaction_unittest.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/common/net/networktransaction_unittest.py b/third_party/WebKit/Tools/Scripts/webkitpy/common/net/network_transaction_unittest.py
similarity index 92%
rename from third_party/WebKit/Tools/Scripts/webkitpy/common/net/networktransaction_unittest.py
rename to third_party/WebKit/Tools/Scripts/webkitpy/common/net/network_transaction_unittest.py
index 8b696eeca2385e1381cf5e02fadb727791f42721..62fb6082673bcb97bbf7b725df961f1ec25204ae 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/common/net/networktransaction_unittest.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/common/net/network_transaction_unittest.py
@@ -28,8 +28,8 @@
from urllib2 import HTTPError
-from webkitpy.common.net.networktransaction import NetworkTransaction, NetworkTimeout
-from webkitpy.common.system.logtesting import LoggingTestCase
+from webkitpy.common.net.network_transaction import NetworkTransaction, NetworkTimeout
+from webkitpy.common.system.log_testing import LoggingTestCase
class NetworkTransactionTest(LoggingTestCase):
@@ -53,9 +53,9 @@ class NetworkTransactionTest(LoggingTestCase):
try:
transaction.run(self._raise_exception)
did_throw_exception = False
- except Exception as e:
+ except Exception as error: # pylint: disable=broad-except
did_process_exception = True
- self.assertEqual(e, self.exception)
+ self.assertEqual(error, self.exception)
self.assertTrue(did_throw_exception)
self.assertTrue(did_process_exception)
@@ -77,7 +77,7 @@ class NetworkTransactionTest(LoggingTestCase):
'WARNING: Received HTTP status 500 loading "http://example.com/". '
'Retrying in 0.0 seconds...\n'])
- def test_convert_404_to_None(self):
+ def test_convert_404_to_none(self):
transaction = NetworkTransaction(convert_404_to_None=True)
self.assertIsNone(transaction.run(self._raise_404_error))

Powered by Google App Engine
This is Rietveld 408576698