rebaseline-cl: Don't trigger new try jobs for builders that already have jobs started.

third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl_unittest.py

 # Copyright 2016 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.
 
 import json
 import optparse
 
 from webkitpy.common.net.buildbot import Build
 from webkitpy.common.net.rietveld import Rietveld
 from webkitpy.common.net.web_mock import MockWeb
@@ skipping 13 matching lines @@
         LoggingTestCase.setUp(self)
         web = MockWeb(urls={
             'https://codereview.chromium.org/api/11112222': json.dumps({
                 'patchsets': [1, 2],
             }),
             'https://codereview.chromium.org/api/11112222/2': json.dumps({
                 'try_job_results': [
                     {
                         'builder': 'MOCK Try Win',
                         'buildnumber': 5000,
+                        'result': 0,
                     },
                     {
-                        'builder': 'MOCK Mac Try',
+                        'builder': 'MOCK Try Mac',
                         'buildnumber': 4000,
+                        'result': 0,
                     },
                 ],
                 'files': {
                     'third_party/WebKit/LayoutTests/fast/dom/prototype-inheritance.html': {'status': 'M'},
                     'third_party/WebKit/LayoutTests/fast/dom/prototype-taco.html': {'status': 'M'},
                 },
             }),
         })
         self.tool.builders = BuilderList({
             "MOCK Try Win": {
@@ skipping 52 matching lines @@
             'INFO: Rebaselining svg/dynamic-updates/SVGFEDropShadowElement-dom-stdDeviation-attr.html\n',
         ])
 
     def test_execute_with_no_issue_number(self):
         self.command.execute(self.command_options(), [], self.tool)
         self.assertLog(['ERROR: No issue number given and no issue for current branch. This tool requires a CL\n'
                         'to operate on; please run `git cl upload` on this branch first, or use the --issue\n'
                         'option to download baselines for another existing CL.\n'])
 
     def test_execute_with_issue_number_from_branch(self):
-        git_cl = GitCL(MockExecutive2())
+        git_cl = GitCL(self.tool)
         git_cl.get_issue_number = lambda: '11112222'
         self.command.git_cl = lambda: git_cl
         self.command.execute(self.command_options(), [], self.tool)
         self.assertLog([
             'INFO: Tests to rebaseline:\n',
             'INFO:   fast/dom/prototype-newtest.html: MOCK Try Win (5000)\n',
             'INFO:   svg/dynamic-updates/SVGFEDropShadowElement-dom-stdDeviation-attr.html: MOCK Try Win (5000)\n',
             'INFO:   fast/dom/prototype-inheritance.html: MOCK Try Win (5000)\n',
             'INFO:   fast/dom/prototype-taco.html: MOCK Try Win (5000)\n',
             'INFO: Rebaselining fast/dom/prototype-inheritance.html\n',
@@ skipping 48 matching lines @@
 
         self.assertEqual(
             self.tool.executive.calls,
             [
                 [['python', 'echo', 'copy-existing-baselines-internal', '--suffixes', 'txt',
                   '--builder', 'MOCK Try Win', '--test', 'fast/dom/prototype-taco.html']],
                 [['python', 'echo', 'rebaseline-test-internal', '--suffixes', 'txt',
                   '--builder', 'MOCK Try Win', '--test', 'fast/dom/prototype-taco.html', '--build-number', '5000']],
                 [['python', 'echo', 'optimize-baselines', '--suffixes', 'txt', 'fast/dom/prototype-taco.html']]
             ])
+
+    def test_trigger_jobs_for_missing_builds_empty_list(self):
+        # Passing in no builds implies that no try jobs were started.
+        self.assertTrue(self.command.trigger_jobs_for_missing_builds([]))
+        self.assertEqual(
+            self.tool.executive.calls,
+            [['git', 'cl', 'try', '-b', 'MOCK Try Linux'], ['git', 'cl', 'try', '-b', 'MOCK Try Win']])
+        self.assertLog([
+            'INFO: Triggering try jobs for:\n',
+            'INFO:   MOCK Try Linux\n',
+            'INFO:   MOCK Try Win\n',
+        ])
+
+    def test_trigger_jobs_for_missing_builds_started_and_successful(self):
+        # A build number of None implies that a job has been started but not finished yet.
+        self.assertTrue(self.command.trigger_jobs_for_missing_builds([
+            Build('MOCK Try Linux', None),
+            Build('MOCK Try Win', 123),
+        ]))
+        self.assertEqual(self.tool.executive.calls, [])
+        self.assertLog([
+            'INFO: There are existing pending builds for:\n',
+            'INFO:   MOCK Try Linux\n',
+        ])
+
+    def test_trigger_jobs_for_missing_builds_one_started(self):
+        self.assertTrue(self.command.trigger_jobs_for_missing_builds([
+            Build('MOCK Try Linux', None),
+        ]))
+        self.assertEqual(self.tool.executive.calls, [['git', 'cl', 'try', '-b', 'MOCK Try Win']])
+        self.assertLog([
+            'INFO: There are existing pending builds for:\n',
+            'INFO:   MOCK Try Linux\n',
+            'INFO: Triggering try jobs for:\n',
+            'INFO:   MOCK Try Win\n',
+        ])