watchlist: If the style check fails, then the watchlist will not be run.

• Component: Tools / Tests
• Status: RESOLVED FIXED

[David Levin]

Right now, we run the watchlist after the stylebot (see http://trac.webkit.org/changeset/96541/trunk/Tools/Scripts/webkitpy/tool/commands/queues.py) but if the style bot fails, it will throw and the watchlist will not run.

The ordering seems fine but we should consider doing something about not running watchlist.

[Adam Barth]

We can just catch the exception and re-throw it.

[David Levin]

Created attachment 111078 [details]
Patch

[Adam Barth]

Comment on attachment 111078 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=111078&action=review

I like executive_throws_when_run.  :)

> Tools/Scripts/webkitpy/tool/commands/queues.py:440
> +        # Raise any exceptions that occurred while running check-style
> +        if exception:
> +            raise exception

There's a slightly snazzier way of doing using the "raise" keyword with no arguments, but this is more than fine.

> Tools/Scripts/webkitpy/tool/mocktool.py:726
> -    def __init__(self, should_log=False, should_throw=False):
> +    def __init__(self, should_log=False, should_throw=False, should_throw_when_run=set()):

Eric will tell you to use Non for the default paramater value.  The problem is that the default paramater value is a static and we modify it by accident, all future callers will get the modified value!

> Tools/Scripts/webkitpy/tool/mocktool.py:729
> +        self._should_throw_when_run = should_throw_when_run

If you use None, then this line becomes:

self._should_throw_when_run = should_throw_when_run or set()

[David Levin]

Committed as http://trac.webkit.org/changeset/97515