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

Unified Diff: appengine/findit/waterfall/test/build_failure_analysis_test.py

Issue 2590313002: Added Blame.AddReagions(iterable) method (Closed)
Patch Set: changing Blame.AddRegions to use list.extend 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
« no previous file with comments | « appengine/findit/util_scripts/git_checkout/test/local_git_repository_test.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/findit/waterfall/test/build_failure_analysis_test.py
diff --git a/appengine/findit/waterfall/test/build_failure_analysis_test.py b/appengine/findit/waterfall/test/build_failure_analysis_test.py
index 09f5098dae389f8a7b4345814d0eef0cb640aeef..f42288d5705b11291d5234e6b1d6e336d012cdba 100644
--- a/appengine/findit/waterfall/test/build_failure_analysis_test.py
+++ b/appengine/findit/waterfall/test/build_failure_analysis_test.py
@@ -89,21 +89,22 @@ class BuildFailureAnalysisTest(wf_testcase.WaterfallTestCase):
return None
blame = Blame(revision, path)
- blame.AddRegion(Region(1, 2, '7',
- u'test3@chromium.org', u'test3@chromium.org',
- datetime(2015, 06, 07, 04, 35, 32)))
- blame.AddRegion(Region(3, 3, '5',
- u'test3@chromium.org', u'test3@chromium.org',
- datetime(2015, 06, 05, 04, 35, 32)))
- blame.AddRegion(Region(7, 1, '8',
- u'test2@chromium.org', u'test2@chromium.org',
- datetime(2015, 06, 8, 04, 35, 32)))
- blame.AddRegion(Region(8, 1, '7',
- u'test3@chromium.org', u'test3@chromium.org',
- datetime(2015, 06, 07, 21, 35, 32)))
- blame.AddRegion(Region(9, 10, '12',
- u'test3@chromium.org', u'test3@chromium.org',
- datetime(2015, 06, 12, 04, 35, 32)))
+ blame.AddRegions([
+ Region(1, 2, '7',
+ u'test3@chromium.org', u'test3@chromium.org',
+ datetime(2015, 06, 07, 04, 35, 32)),
+ Region(3, 3, '5',
+ u'test3@chromium.org', u'test3@chromium.org',
+ datetime(2015, 06, 05, 04, 35, 32)),
+ Region(7, 1, '8',
+ u'test2@chromium.org', u'test2@chromium.org',
+ datetime(2015, 06, 8, 04, 35, 32)),
+ Region(8, 1, '7',
+ u'test3@chromium.org', u'test3@chromium.org',
+ datetime(2015, 06, 07, 21, 35, 32)),
+ Region(9, 10, '12',
+ u'test3@chromium.org', u'test3@chromium.org',
+ datetime(2015, 06, 12, 04, 35, 32))])
return blame
def testIsSameFile(self):
« no previous file with comments | « appengine/findit/util_scripts/git_checkout/test/local_git_repository_test.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698