Added Blame.AddReagions(iterable) method

appengine/findit/waterfall/test/build_failure_analysis_test.py

 # Copyright 2015 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 from datetime import datetime
 
 from common.waterfall import failure_type
 from libs.gitiles.blame import Blame
 from libs.gitiles.blame import Region
 from libs.gitiles.change_log import FileChangeInfo
@@ skipping 71 matching lines @@
     return MOCK_CHANGE_LOGS.get(revision, MockChangeLog('12', []))
 
   def _MockGetCommitsBetweenRevisions(self, start_revision, end_revision):
     return map(str, range(int(start_revision) + 1, int(end_revision) + 1))
 
   def _MockGetBlame(self, path, revision):
     if revision == '10' or revision == '11' or path == 'f_not_exist.cc':
       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):
     self.assertTrue(build_failure_analysis._IsSameFile('a/b/x.cc', 'x.cc'))
     self.assertTrue(build_failure_analysis._IsSameFile('a/b/x.cc', 'b/x.cc'))
     self.assertTrue(build_failure_analysis._IsSameFile('a/b/x.cc', 'a/b/x.cc'))
     self.assertTrue(build_failure_analysis._IsSameFile('A/B/X.cc', 'a/b/x.cc'))
 
     self.assertFalse(
         build_failure_analysis._IsSameFile('a/prefix_x.cc.', 'x.cc'))
@@ skipping 1096 matching lines @@
         touched_file, file_path_in_log, justification, file_name_occurrences,
         line_numbers, repo_info, commit_revision)
 
     expected_justification = {
         'score': 4,
         'hints': {
             'modified c.cc[1, 3] (and it was in log)': 4
         }
     }
     self.assertEqual(expected_justification, justification.ToDict())