[InlineTest] getRerunArgs returns an empty string if the test was skipped.

[sivachandra]

Summary

The main issue that this patch is trying to address is that the current
implementation of getRerunArgs of InlineTest relies on the attribute
'using_dsym' which could be absent if the test was skipped altogether.
[That is, if both dsym and dwarf tests were skipped.]

While at it, the use of deprecated Python module 'new' is eliminated.

[clayborg]

Looks good.

test/lldbinline.py

 import lldb
 from lldbtest import *
 import lldbutil
 import os
-import new
 import unittest2
 import sys
 
 def source_type(filename):
     _, extension = os.path.splitext(filename)
     return {
         '.c' : 'C_SOURCES',
         '.cpp' : 'CXX_SOURCES',
def handle_breakpoint(self, test, breakpoint_id):
                 return
 
 class InlineTest(TestBase):
     # Internal implementation
 
     def getRerunArgs(self):
         # The -N option says to NOT run a if it matches the option argument, so
         # if we are using dSYM we say to NOT run dwarf (-N dwarf) and vice versa.
-        if self.using_dsym:
+        if self.using_dsym is None:
+            # The test was skipped altogether.
+            return ""
+        elif self.using_dsym:
             return "-N dwarf %s" % (self.mydir)
         else:
             return "-N dsym %s" % (self.mydir)
         
     def BuildMakefile(self):
         if os.path.exists("Makefile"):
             return
 

 
 def MakeInlineTest(__file, __globals, decorators=None):
     # Derive the test name from the current file name
     file_basename = os.path.basename(__file)
     InlineTest.mydir = TestBase.compute_mydir(__file)
 
     test_name, _ = os.path.splitext(file_basename)
     # Build the test case 
-    test = new.classobj(test_name, (InlineTest,), {})
+    test = type(test_name, (InlineTest,), {'using_dsym': None})
     test.name = test_name
 
     test.test_with_dsym = ApplyDecoratorsToFunction(test._InlineTest__test_with_dsym, decorators)
     test.test_with_dwarf = ApplyDecoratorsToFunction(test._InlineTest__test_with_dwarf, decorators)
 
     # Add the test case to the globals, and hide InlineTest
     __globals.update({test_name : test})
     
     return test