Attachment 341198 Details for Bug 185950 – Proposed patch

[patch] Proposed patch

ews_buildbot_checkconfig.patch (text/plain), 1.60 KB, created by Aakash Jain on 2018-05-24 09:31:21 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Aakash Jain

Created: 2018-05-24 09:31:21 PDT

Size: 1.60 KB

patch

obsolete

>Index: Tools/ChangeLog
>===================================================================
>--- Tools/ChangeLog	(revision 232153)
>+++ Tools/ChangeLog	(working copy)
>@@ -1,3 +1,13 @@
>+2018-05-24  Aakash Jain  <aakash_jain@apple.com>
>+
>+        [ews-build] Unit-tests should run buildbot checkconfig
>+        https://bugs.webkit.org/show_bug.cgi?id=185950
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * BuildSlaveSupport/ews-build/loadConfig_unittest.py:
>+        (BuildbotCheckconfig.test_buildbot_checkconfig): Added unit-test to run buildbot checkconfig.
>+
> 2018-05-24  Carlos Garcia Campos  <cgarcia@igalia.com>
> 
>         WebDriver: implement maximize, minimize and fullscreen window commands
>Index: Tools/BuildSlaveSupport/ews-build/loadConfig_unittest.py
>===================================================================
>--- Tools/BuildSlaveSupport/ews-build/loadConfig_unittest.py	(revision 232153)
>+++ Tools/BuildSlaveSupport/ews-build/loadConfig_unittest.py	(working copy)
>@@ -23,6 +23,7 @@
> # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> 
> 
>+import subprocess
> import unittest
> 
> import loadConfig
>@@ -33,6 +34,12 @@ class ConfigDotJSONTest(unittest.TestCas
>         loadConfig.loadBuilderConfig({})
> 
> 
>+class BuildbotCheckconfig(unittest.TestCase):
>+    def test_buildbot_checkconfig(self):
>+        output = subprocess.check_output(['buildbot', 'checkconfig'])
>+        self.assertEqual(output, 'Config file is good!\n')
>+
>+
> class TagsForBuilderTeest(unittest.TestCase):
>     def verifyTags(self, builderName, expectedTags):
>         tags = loadConfig.getTagsForBuilder({'name': builderName})

Actions: View | Formatted Diff | Diff

Attachments on bug 185950: 341198 | 341205