[Lldb-commits] [PATCH] D64583: [lldb][test_suite] Fix skipIfTargetAndroid decorator
Wanyi Ye via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Jul 11 12:27:59 PDT 2019
kusmour created this revision.
kusmour added a reviewer: xiaobai.
Herald added subscribers: lldb-commits, srhines.
Herald added a project: LLDB.
Delete the duplicate func `skipIfTargetAndroid`
Fix the old one
Repository:
rLLDB LLDB
https://reviews.llvm.org/D64583
Files:
lldb/packages/Python/lldbsuite/test/decorators.py
Index: lldb/packages/Python/lldbsuite/test/decorators.py
===================================================================
--- lldb/packages/Python/lldbsuite/test/decorators.py
+++ lldb/packages/Python/lldbsuite/test/decorators.py
@@ -596,11 +596,6 @@
"""Decorate the item to skip tests that should be skipped on Windows."""
return skipIfPlatform(["windows"])(func)
-def skipIfTargetAndroid(func):
- return unittest2.skipIf(lldbplatformutil.target_is_android(),
- "skip on target Android")(func)
-
-
def skipUnlessWindows(func):
"""Decorate the item to skip tests that should be skipped on any non-Windows platform."""
return skipUnlessPlatform(["windows"])(func)
@@ -652,7 +647,7 @@
"requires one of %s" % (", ".join(oslist)))
-def skipIfTargetAndroid(api_levels=None, archs=None):
+def skipIfTargetAndroid(bugnumber=None, api_levels=None, archs=None):
"""Decorator to skip tests when the target is Android.
Arguments:
@@ -665,7 +660,8 @@
_skip_for_android(
"skipping for android",
api_levels,
- archs))
+ archs),
+ bugnumber)
def skipUnlessSupportedTypeAttribute(attr):
"""Decorate the item to skip test unless Clang supports type __attribute__(attr)."""
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D64583.209297.patch
Type: text/x-patch
Size: 1332 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20190711/6412677e/attachment-0001.bin>
More information about the lldb-commits
mailing list