This revision was automatically updated to reflect the committed changes.
Closed by commit rL250038: dotest.py: Remove useless AttributeError catches
(authored by labath).
Changed prior to commit:
http://reviews.llvm.org/D13583?vs=36933&id=37102#toc
Repository:
rL LLVM
http://reviews.llvm.org/D13583
Files:
lldb/trunk/test/lldbtest.py
Index: lldb/trunk/test/lldbtest.py
===================================================================
--- lldb/trunk/test/lldbtest.py
+++ lldb/trunk/test/lldbtest.py
@@ -475,11 +475,8 @@
raise Exception("@python_api_test can only be used to decorate a test method")
@wraps(func)
def wrapper(self, *args, **kwargs):
- try:
- if lldb.dont_do_python_api_test:
- self.skipTest("python api tests")
- except AttributeError:
- pass
+ if lldb.dont_do_python_api_test:
+ self.skipTest("python api tests")
return func(self, *args, **kwargs)
# Mark this function as such to separate them from lldb command line tests.
@@ -492,11 +489,8 @@
raise Exception("@lldbmi_test can only be used to decorate a test method")
@wraps(func)
def wrapper(self, *args, **kwargs):
- try:
- if lldb.dont_do_lldbmi_test:
- self.skipTest("lldb-mi tests")
- except AttributeError:
- pass
+ if lldb.dont_do_lldbmi_test:
+ self.skipTest("lldb-mi tests")
return func(self, *args, **kwargs)
# Mark this function as such to separate them from lldb command line tests.
@@ -509,11 +503,8 @@
raise Exception("@benchmarks_test can only be used to decorate a test method")
@wraps(func)
def wrapper(self, *args, **kwargs):
- try:
- if not lldb.just_do_benchmarks_test:
- self.skipTest("benchmarks tests")
- except AttributeError:
- pass
+ if not lldb.just_do_benchmarks_test:
+ self.skipTest("benchmarks tests")
return func(self, *args, **kwargs)
# Mark this function as such to separate them from the regular tests.
@@ -539,11 +530,8 @@
raise Exception("@dsym_test can only be used to decorate a test method")
@wraps(func)
def wrapper(self, *args, **kwargs):
- try:
- if lldb.dont_do_dsym_test:
- self.skipTest("dsym tests")
- except AttributeError:
- pass
+ if lldb.dont_do_dsym_test:
+ self.skipTest("dsym tests")
return func(self, *args, **kwargs)
# Mark this function as such to separate them from the regular tests.
@@ -556,11 +544,8 @@
raise Exception("@dwarf_test can only be used to decorate a test method")
@wraps(func)
def wrapper(self, *args, **kwargs):
- try:
- if lldb.dont_do_dwarf_test:
- self.skipTest("dwarf tests")
- except AttributeError:
- pass
+ if lldb.dont_do_dwarf_test:
+ self.skipTest("dwarf tests")
return func(self, *args, **kwargs)
# Mark this function as such to separate them from the regular tests.
@@ -573,11 +558,8 @@
raise Exception("@dwo_test can only be used to decorate a test method")
@wraps(func)
def wrapper(self, *args, **kwargs):
- try:
- if lldb.dont_do_dwo_test:
- self.skipTest("dwo tests")
- except AttributeError:
- pass
+ if lldb.dont_do_dwo_test:
+ self.skipTest("dwo tests")
return func(self, *args, **kwargs)
# Mark this function as such to separate them from the regular tests.
@@ -590,11 +572,8 @@
raise Exception("@debugserver_test can only be used to decorate a test method")
@wraps(func)
def wrapper(self, *args, **kwargs):
- try:
- if lldb.dont_do_debugserver_test:
- self.skipTest("debugserver tests")
- except AttributeError:
- pass
+ if lldb.dont_do_debugserver_test:
+ self.skipTest("debugserver tests")
return func(self, *args, **kwargs)
# Mark this function as such to separate them from the regular tests.
@@ -607,11 +586,8 @@
raise Exception("@llgs_test can only be used to decorate a test method")
@wraps(func)
def wrapper(self, *args, **kwargs):
- try:
- if lldb.dont_do_llgs_test:
- self.skipTest("llgs tests")
- except AttributeError:
- pass
+ if lldb.dont_do_llgs_test:
+ self.skipTest("llgs tests")
return func(self, *args, **kwargs)
# Mark this function as such to separate them from the regular tests.
@@ -624,11 +600,8 @@
raise Exception("@not_remote_testsuite_ready can only be used to decorate a test method")
@wraps(func)
def wrapper(self, *args, **kwargs):
- try:
- if lldb.lldbtest_remote_sandbox or lldb.remote_platform:
- self.skipTest("not ready for remote testsuite")
- except AttributeError:
- pass
+ if lldb.lldbtest_remote_sandbox or lldb.remote_platform:
+ self.skipTest("not ready for remote testsuite")
return func(self, *args, **kwargs)
# Mark this function as such to separate them from the regular tests.
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
