[Lldb-commits] [PATCH] D47579: dotest: make inline tests compatible with -f

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Jun 4 09:30:30 PDT 2018


JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.

LGTM

> Can you try inserting something like this instead of the `ApplyDecoratorsToFunction` line and see if your problems go away?
> 
>   @wraps(InlineTest._test)
>   def test_func(*args, **kwargs):
>       return InlineTest._test(*args, **kwargs)
>    
>   test_func = ApplyDecoratorsToFunction(test_func, decorators)
>    
> 
> This should make sure each test class gets a fresh function object with an independent set of categories.

Alright, I'll give that a shot. Thanks for the suggestion! :-)


https://reviews.llvm.org/D47579





More information about the lldb-commits mailing list