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

Unified Diff: tools/foozzie/v8_foozzie_test.py

Issue 2635923002: [foozzie] Refactoring - move source hashing to main script (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « tools/foozzie/v8_foozzie.py ('k') | tools/foozzie/v8_suppressions.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/foozzie/v8_foozzie_test.py
diff --git a/tools/foozzie/v8_foozzie_test.py b/tools/foozzie/v8_foozzie_test.py
index 13b7e6b6a4331fce4554db4e757886eb04bf6c58..5a32ead620a9ce0328d0c17692ed6ec0d0d9f51f 100644
--- a/tools/foozzie/v8_foozzie_test.py
+++ b/tools/foozzie/v8_foozzie_test.py
@@ -21,12 +21,12 @@ class UnitTest(unittest.TestCase):
'x64', 'fullcode', 'x64', 'default')
one = ''
two = ''
- diff = None, 'none', 'none'
+ diff = None, None
self.assertEquals(diff, suppress.diff(one, two))
one = 'a \n b\nc();'
two = 'a \n b\nc();'
- diff = None, 'none', 'none'
+ diff = None, None
self.assertEquals(diff, suppress.diff(one, two))
# Ignore line before caret, caret position, stack trace char numbers
@@ -49,7 +49,7 @@ stack line :2: foo
Validation of asm.js module failed: baz
undefined
"""
- diff = None, 'none', 'none'
+ diff = None, None
self.assertEquals(diff, suppress.diff(one, two))
one = """
@@ -59,7 +59,7 @@ Extra line
two = """
Still equal
"""
- diff = '- Extra line', 'none', 'none'
+ diff = '- Extra line', None
self.assertEquals(diff, suppress.diff(one, two))
one = """
@@ -69,7 +69,7 @@ Still equal
Still equal
Extra line
"""
- diff = '+ Extra line', 'none', 'none'
+ diff = '+ Extra line', None
self.assertEquals(diff, suppress.diff(one, two))
one = """
@@ -81,7 +81,7 @@ undefined
otherfile.js: TypeError: undefined is not a constructor
"""
diff = """- somefile.js: TypeError: undefined is not a constructor
-+ otherfile.js: TypeError: undefined is not a constructor""", 'none', 'none'
++ otherfile.js: TypeError: undefined is not a constructor""", None
self.assertEquals(diff, suppress.diff(one, two))
« no previous file with comments | « tools/foozzie/v8_foozzie.py ('k') | tools/foozzie/v8_suppressions.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698