Skip to content
Permalink
Browse files

Fix @skipIfSanitized decorator

To run the test the decorator function should return None, not False.
Returning anything other than None skips the test.

llvm-svn: 366903
  • Loading branch information...
labath committed Jul 24, 2019
1 parent 565c543 commit 3a12e73f6729db2ad4e3c3c8624aff56263f06ba
Showing with 4 additions and 2 deletions.
  1. +4 −2 lldb/packages/Python/lldbsuite/test/decorators.py
@@ -827,6 +827,8 @@ def is_feature_enabled(self):
def skipIfSanitized(func):
"""Skip this test if the environment is set up to run LLDB itself under ASAN."""
def is_sanitized():
return (('DYLD_INSERT_LIBRARIES' in os.environ) and
'libclang_rt.asan' in os.environ['DYLD_INSERT_LIBRARIES'])
if (('DYLD_INSERT_LIBRARIES' in os.environ) and
'libclang_rt.asan' in os.environ['DYLD_INSERT_LIBRARIES']):
return "ASAN unsupported"
return None
return skipTestIfFn(is_sanitized)(func)

0 comments on commit 3a12e73

Please sign in to comment.
You can’t perform that action at this time.