[TestGdbRemoteAbort] Skip on API 16 Android devices

[sivachandra]

Summary

This change also adds the infrastructure required to specify the API
levels for which tests should be skipped.

[tberghammer]

test/tools/lldb-server/inferior-crash/TestGdbRemoteAbort.py

35

Why do we want to skip this test for API=16?

[sivachandra]

test/tools/lldb-server/inferior-crash/TestGdbRemoteAbort.py

35

For API 16, std::abort raises SIGSEGV and not SIGABRT.

[labath]

test/lldbtest.py

945

Why do we need two levels of nested functions here?
The original code seemed to manage just fine with one...

test/tools/lldb-server/inferior-crash/TestGdbRemoteAbort.py

35

Please add a short comment explaining why is this skipped.

[sivachandra]

test/lldbtest.py

945

This is per my knowledge of decorators: I think when a decorator takes args, it needs to return a decorator itself. Correct me if I am wrong.

test/tools/lldb-server/inferior-crash/TestGdbRemoteAbort.py

35

Will do.

[labath]

lgtm

[sivachandra]

Address comments.
Fix other occurances of skipIfTargetAndroid.

test/lldbtest.py

def wrapper(*args, **kwargs):
         from unittest2 import case
         self = args[0]
         if "i386" == self.getArchitecture():
             self.skipTest("skipping because i386 is not a supported architecture")
         else:
             func(*args, **kwargs)
     return wrapper
 
-def skipIfTargetAndroid(func):
-    """Decorate the item to skip tests that should be skipped when the target is Android."""
+def skipIfTargetAndroid(api_levels=None):
+    """Decorator to skip tests when the target is Android.
+
+    Arguments:
+        api_levels - The API levels for which the test should be skipped. If
+            it is None, then the test will be skipped for all API levels.
+    """
+    def myImpl(func):
         if isinstance(func, type) and issubclass(func, unittest2.TestCase):
-        raise Exception("@skipIfTargetAndroid can only be used to decorate a test method")
+            raise Exception("@skipIfTargetAndroid can only be used to "
+                            "decorate a test method")
         @wraps(func)
         def wrapper(*args, **kwargs):
             from unittest2 import case
             self = args[0]
             triple = self.dbg.GetSelectedPlatform().GetTriple()
             if re.match(".*-.*-.*-android", triple):
-            self.skipTest("skip on Android target")
+                if api_levels:
+                    device_api = android_device_api()
+                    if device_api and (device_api in api_levels):
+                        self.skipTest(
+                            "skip on Android target with API %d" % device_api)
                 else:
+                    self.skipTest("skip on Android target")
             func(*args, **kwargs)
         return wrapper
+    return myImpl
 
 def skipUnlessCompilerRt(func):
     """Decorate the item to skip tests if testing remotely."""
     if isinstance(func, type) and issubclass(func, unittest2.TestCase):
         raise Exception("@skipUnless can only be used to decorate a test method")
     @wraps(func)
     def wrapper(*args, **kwargs):
         from unittest2 import case

test/tools/lldb-server/inferior-crash/TestGdbRemoteAbort.py

class TestGdbRemoteAbort(gdbremote_testcase.GdbRemoteTestCaseBase):
     @dsym_test
     def test_inferior_abort_received_debugserver_dsym(self):
         self.init_debugserver_test()
         self.buildDsym()
         self.inferior_abort_received()
 
     @llgs_test
     @dwarf_test
+    @skipIfTargetAndroid(api_levels=[16])
     def test_inferior_abort_received_llgs_dwarf(self):
         self.init_llgs_test()
         self.buildDwarf()
         self.inferior_abort_received()