shylie/llvm-project
1
0
Fork 0
Code Issues Pull Requests Actions 6 Packages Projects Releases Wiki Activity

Fix @skipUnlessPlatform and @skipIfPlatform decorators to work on classes.

Browse Source 
The decorators to skip if or unless certain platforms relied on calling
skipTest, which only worked if they were decorating individual test cases.
However for decorating entire classes it needs different behavior. This behavior
is already encapsulated in unittest2.skipIf and unittest2.skipUnless so this
modifies the platform decorators to use these unittest skip decorators.

Test Plan:
./dotest.py -t -p TestObjCMethods2.py
See skipped tests on linux, and see tests run on macosx.

Differential Revision: http://reviews.llvm.org/D8903

llvm-svn: 234514
This commit is contained in:
Robert Flack 2015-04-09 18:07:58 +00:00
parent 53703d0862
commit 068898cb83
2 changed files with 21 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
lldb/test/dotest.py
View File

@ -1320,16 +1320,6 @@ setupCrashInfoHook()
if not skip_long_running_test:
os.environ["LLDB_SKIP_LONG_RUNNING_TEST"] = "NO"
#
# Walk through the testdirs while collecting tests.
#
for testdir in testdirs:
os.path.walk(testdir, visit, 'Test')
#
# Now that we have loaded all the test cases, run the whole test suite.
#
# For the time being, let's bracket the test runner within the
# lldb.SBDebugger.Initialize()/Terminate() pair.
import lldb
@ -1383,6 +1373,16 @@ dont_do_debugserver_test = "linux" in target_platform or "freebsd" in target_pla
# Don't do lldb-server (llgs) tests on anything except Linux.
dont_do_llgs_test = not ("linux" in target_platform)
#
# Walk through the testdirs while collecting tests.
#
for testdir in testdirs:
os.path.walk(testdir, visit, 'Test')
#
# Now that we have loaded all the test cases, run the whole test suite.
#
# Put the blacklist in the lldb namespace, to be used by lldb.TestBase.
lldb.blacklist = blacklist

37
lldb/test/lldbtest.py
View File

@ -683,33 +683,21 @@ def skipUnlessDarwin(func):
"""Decorate the item to skip tests that should be skipped on any non Darwin platform."""
return skipUnlessPlatform(getDarwinOSTriples())(func)
def getPlatform():
platform = lldb.DBG.GetSelectedPlatform().GetTriple().split('-')[2]
if platform.startswith('freebsd'):
platform = 'freebsd'
return platform
def skipIfPlatform(oslist):
"""Decorate the item to skip tests if running on one of the listed platforms."""
def decorator(func):
@wraps(func)
def wrapper(*args, **kwargs):
from unittest2 import case
self = args[0]
if self.getPlatform() in oslist:
self.skipTest("skip on %s" % (", ".join(oslist)))
else:
func(*args, **kwargs)
return wrapper
return decorator
return unittest2.skipIf(getPlatform() in oslist,
"skip on %s" % (", ".join(oslist)))
def skipUnlessPlatform(oslist):
"""Decorate the item to skip tests unless running on one of the listed platforms."""
def decorator(func):
@wraps(func)
def wrapper(*args, **kwargs):
from unittest2 import case
self = args[0]
if not (self.getPlatform() in oslist):
self.skipTest("requires one of %s" % (", ".join(oslist)))
else:
func(*args, **kwargs)
return wrapper
return decorator
return unittest2.skipUnless(getPlatform() in oslist,
"requires on of %s" % (", ".join(oslist)))
def skipIfLinuxClang(func):
"""Decorate the item to skip tests that should be skipped if building on
@ -1405,10 +1393,7 @@ class Base(unittest2.TestCase):
def getPlatform(self):
"""Returns the platform the test suite is running on."""
platform = lldb.DBG.GetSelectedPlatform().GetTriple().split('-')[2]
if platform.startswith('freebsd'):
platform = 'freebsd'
return platform
return getPlatform()
def isIntelCompiler(self):
""" Returns true if using an Intel (ICC) compiler, false otherwise. """