Index: lldb/trunk/test/dotest.py =================================================================== --- lldb/trunk/test/dotest.py +++ lldb/trunk/test/dotest.py @@ -1320,16 +1320,6 @@ 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 @@ # 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 Index: lldb/trunk/test/lldbtest.py =================================================================== --- lldb/trunk/test/lldbtest.py +++ lldb/trunk/test/lldbtest.py @@ -683,33 +683,21 @@ """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 @@ 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. """