From: Jonas Devlieghere <jonas@devlieghere.com>
Date: Tue, 4 Aug 2020 02:56:10 +0000 (-0700)
Subject: [lldb/Test] Fix skipTestIfFn for fucntions that return a value
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d6a5cce0e7d65562f081569a61595e53cdb8d5d0;p=platform%2Fupstream%2Fllvm.git

[lldb/Test] Fix skipTestIfFn for fucntions that return a value

Sometimes the decorator is used on a common function rather than the
test method, which can return a value. This fails with decorators that
use skipTestIfFn under the hood.
---

diff --git a/lldb/packages/Python/lldbsuite/test/decorators.py b/lldb/packages/Python/lldbsuite/test/decorators.py
index 8c8f250..873952e 100644
--- a/lldb/packages/Python/lldbsuite/test/decorators.py
+++ b/lldb/packages/Python/lldbsuite/test/decorators.py
@@ -132,7 +132,7 @@ def skipTestIfFn(expected_fn, bugnumber=None):
             if reason is not None:
                 self.skipTest(reason)
             else:
-                func(*args, **kwargs)
+                return func(*args, **kwargs)
         return wrapper
 
     # Some decorators can be called both with no arguments (e.g. @expectedFailureWindows)