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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/diff_parser_unittest.py

Issue 2248653002: Revert of Fix pylint warnings in webkitpy/common/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Manual Revert (Patch Set 1 causes patch failure in read_checksum_from_png_unittest.py) Created 4 years, 4 months 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/checkout/diff_parser_unittest.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/diff_parser_unittest.py b/third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/diff_parser_unittest.py
index 3e37525719720b8dc9c2e01cf40c1ebed0528a96..a00791335807e782455f84f835f9a7799cc23026 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/diff_parser_unittest.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/diff_parser_unittest.py
@@ -27,11 +27,10 @@
# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
import cStringIO as StringIO
-
+import diff_parser
import re
import unittest
-from webkitpy.common.checkout import diff_parser
from webkitpy.common.checkout.diff_test_data import DIFF_TEST_DATA
@@ -106,8 +105,7 @@ class DiffParserTest(unittest.TestCase):
self.assertEqual(diff_parser.get_diff_converter(revision_lines + svn_diff_lines), diff_parser.svn_diff_to_svn_diff)
git_diff_lines = [
- ("diff --git a/Tools/Scripts/webkitpy/common/checkout/diff_parser.py "
- "b/Tools/Scripts/webkitpy/common/checkout/diff_parser.py\n"),
+ "diff --git a/Tools/Scripts/webkitpy/common/checkout/diff_parser.py b/Tools/Scripts/webkitpy/common/checkout/diff_parser.py\n",
"index 3c5b45b..0197ead 100644\n",
"--- a/Tools/Scripts/webkitpy/common/checkout/diff_parser.py\n",
"+++ b/Tools/Scripts/webkitpy/common/checkout/diff_parser.py\n",
@@ -117,8 +115,8 @@ class DiffParserTest(unittest.TestCase):
self.assertEqual(diff_parser.get_diff_converter(comment_lines + git_diff_lines), diff_parser.git_diff_to_svn_diff)
self.assertEqual(diff_parser.get_diff_converter(revision_lines + git_diff_lines), diff_parser.git_diff_to_svn_diff)
- def test_git_mnemonic_prefix(self):
- pattern = re.compile(r' ([a|b])/')
+ def test_git_mnemonicprefix(self):
+ p = re.compile(r' ([a|b])/')
prefixes = [
{'a': 'i', 'b': 'w'}, # git-diff (compares the (i)ndex and the (w)ork tree)
@@ -128,11 +126,9 @@ class DiffParserTest(unittest.TestCase):
{'a': '1', 'b': '2'}, # git diff --no-index a b (compares two non-git things (1) and (2))
]
- def patch(prefix):
- return pattern.sub(lambda match: " %s/" % prefix[match.group(1)], DIFF_TEST_DATA)
-
for prefix in prefixes:
- self.test_diff_parser(diff_parser.DiffParser(patch(prefix).splitlines()))
+ patch = p.sub(lambda x: " %s/" % prefix[x.group(1)], DIFF_TEST_DATA)
+ self.test_diff_parser(diff_parser.DiffParser(patch.splitlines()))
def test_git_diff_to_svn_diff(self):
output = """\

Powered by Google App Engine
This is Rietveld 408576698