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

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

Issue 2130093003: Fix pylint warnings in webkitpy/common/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased 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 a00791335807e782455f84f835f9a7799cc23026..3e37525719720b8dc9c2e01cf40c1ebed0528a96 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,10 +27,11 @@
# 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
@@ -105,7 +106,8 @@ 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",
@@ -115,8 +117,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_mnemonicprefix(self):
- p = re.compile(r' ([a|b])/')
+ def test_git_mnemonic_prefix(self):
+ pattern = re.compile(r' ([a|b])/')
prefixes = [
{'a': 'i', 'b': 'w'}, # git-diff (compares the (i)ndex and the (w)ork tree)
@@ -126,9 +128,11 @@ 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:
- patch = p.sub(lambda x: " %s/" % prefix[x.group(1)], DIFF_TEST_DATA)
- self.test_diff_parser(diff_parser.DiffParser(patch.splitlines()))
+ self.test_diff_parser(diff_parser.DiffParser(patch(prefix).splitlines()))
def test_git_diff_to_svn_diff(self):
output = """\

Powered by Google App Engine
This is Rietveld 408576698