<div dir="ltr">> <span style="font-size:13px">We've already had a case when a test wasn't running for several months before I noticed (see r257901) that it had a bad decorator applied at the class level.</span><br style="font-size:13px"><div><span style="font-size:13px"><br></span></div><div>I've been thinking about this in the background lately.  Several times over the last couple years I found decorator logic that had errors in them (literally, the code was throwing an exception within the decorator logic, such as using a module that isn't imported), but this will go by silently unless some work is put into logging errors there.</div><div><br></div><div>If we haven't already, we should probably have some kind of exception wrapper around our decorators that catches non-unittest-related (i.e. unexpected) exceptions and somehow makes them more prevalent - maybe a hard error on the test or an abort or something.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 2, 2016 at 7:58 AM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">zturner added inline comments.<br>
<span class=""><br>
================<br>
Comment at: packages/Python/lldbsuite/test/lldbtest.py:1102<br>
@@ -1164,2 +1101,3 @@<br>
+                return func(*args, **kwargs)<br>
         return wrapper<br>
<br>
----------------<br>
</span><span class="">labath wrote:<br>
> This return statement is the root cause of the problem. If `func` is a class, you will replace it by a strange function-like object.<br>
</span>This return statement is newly added anyway (and looks to be a mistake).  Does this mean if I remove the return statement, the all of the skip decorators will be able to be used at class level?<br>
<br>
<br>
<br>
<a href="http://reviews.llvm.org/D16741" rel="noreferrer" target="_blank">http://reviews.llvm.org/D16741</a><br>
<br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div>