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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py

Issue 2107993002: Add --auto-update flag which uploads committed CL and initiates the CQ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: style changes Created 4 years, 5 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
Index: third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py
new file mode 100644
index 0000000000000000000000000000000000000000..b2a2aa73256fe616bf085583395678967455edfb
--- /dev/null
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py
@@ -0,0 +1,33 @@
+# 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
+ 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

Powered by Google App Engine
This is Rietveld 408576698