Add --auto-update flag which uploads committed CL and initiates the CQ

third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_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 unittest
+
+from webkitpy.w3c.deps_updater import DepsUpdater
+from webkitpy.common.host_mock import MockHost
+
+
+class DepsUpdaterTest(unittest.TestCase):
+
+    def test_parse_try_job_results(self):
+        output = """Successes:
+linux_chromium_asan_rel_ng                      http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_asan_rel_ng/builds/111111
+mac_chromium_compile_dbg_ng                     http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_compile_dbg_ng/builds/222222
+win_chromium_rel_ng                             http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_ng/builds/333333
+Failures:
+linux_chromium_rel_ng                           http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/444444
+mac_chromium_10.10_rel_ng                       http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_10.10_rel_ng/builds/555555
+win_chromium_rel_ng                             http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_ng/builds/333333
+Started:
+chromeos_amd64-generic_chromium_compile_only_ng http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-generic_chromium_compile_only_ng/builds/777777
+chromeos_daisy_chromium_compile_only_ng         http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_chromium_compile_only_ng/builds/888888
+Total: 30 tryjobs
+"""

[qyearsley, 2016/07/07 02:35:47]

This can be made narrower (and easier to read) if the bot names and parts of the
URL are replaced with fake, shorter strings, e.g.:

"""Successes:
linux_builder     http://example.com/linux_builder/builds/222
mac_builder       http://example.com/mac_builder/builds/222
win_builder       http://example.com/win_builder/builds/222
Failures:
linux_builder     http://example.com/linux_builder/builds/111
mac_builder       http://example.com/mac_builder/builds/111
win_builder       http://example.com/win_builder/builds/111
Started:
chromeos_generic  http://example.com/chromeos_generic/builds/111
chromeos_daisy    http://example.com/chromeos_daisy/builds/111
Total: 8 tryjobs
"""

+        host = MockHost()
+        updater = DepsUpdater(host)
+        self.assertEqual(updater.parse_try_job_results(output), {'Successes': set(['mac_chromium_compile_dbg_ng', 'win_chromium_rel_ng',
+                                                                                   'linux_chromium_asan_rel_ng']),
+                                                                 'Failures': set(['mac_chromium_10.10_rel_ng', 'linux_chromium_rel_ng']),
+                                                                 'Started': set(['chromeos_amd64-generic_chromium_compile_only_ng',
+                                                                                 'chromeos_daisy_chromium_compile_only_ng'])})

[qyearsley, 2016/07/07 02:35:47]

Formatting suggestion: This will be easier to read if it's flattened out a bit,
e.g.

        self.assertEqual(updater.parse_try_job_results(output), {
            'Successes': set([
                'mac_chromium_compile_dbg_ng',
                'win_chromium_rel_ng',
                'linux_chromium_asan_rel_ng',
            ]),
            'Failures': set([
                'mac_chromium_10.10_rel_ng',
                'linux_chromium_rel_ng'
            ]),
            'Started': set([
                'chromeos_amd64-generic_chromium_compile_only_ng',
                'chromeos_daisy_chromium_compile_only_ng',
            ])
        })