[TestEvents] Replace expectedFailureLinux with skipIfLinux
Summary: If we do not mark them as skip, they are still executed, which in turn is leading to an assertion failure. The change also adds skipIfLinux to a testlet which was not previously marked with skipIfLinux. This is because running even that test let leads to an assertion failure. Test Plan: dotest.py -C clang -p TestEvents.py Reviewers: vharron Reviewed By: vharron Subscribers: lldb-commits Differential Revision: http://reviews.llvm.org/D6715 llvm-svn: 224644
This commit is contained in:
parent
5ab2e07231
commit
aaa3d47863
|
@ -20,6 +20,9 @@ class EventAPITestCase(TestBase):
|
|||
self.buildDsym()
|
||||
self.do_listen_for_and_print_event()
|
||||
|
||||
@unittest2.skipUnless((sys.platform.startswith("darwin") or
|
||||
sys.platform.startswith("freebsd")),
|
||||
"requires Darwin or FreeBSD")
|
||||
@python_api_test
|
||||
@dwarf_test
|
||||
def test_listen_for_and_print_event_with_dwarf(self):
|
||||
|
@ -35,7 +38,7 @@ class EventAPITestCase(TestBase):
|
|||
self.buildDsym()
|
||||
self.do_wait_for_event()
|
||||
|
||||
@expectedFailureLinux # non-core functionality, need to reenable and fix later (DES 2014.11.07)
|
||||
@skipIfLinux # non-core functionality, need to reenable and fix later (DES 2014.11.07)
|
||||
@python_api_test
|
||||
@dwarf_test
|
||||
def test_wait_for_event_with_dwarf(self):
|
||||
|
@ -52,7 +55,7 @@ class EventAPITestCase(TestBase):
|
|||
self.do_add_listener_to_broadcaster()
|
||||
|
||||
@skipIfFreeBSD # llvm.org/pr21325
|
||||
@expectedFailureLinux # non-core functionality, need to reenable and fix later (DES 2014.11.07)
|
||||
@skipIfLinux # non-core functionality, need to reenable and fix later (DES 2014.11.07)
|
||||
@python_api_test
|
||||
@dwarf_test
|
||||
def test_add_listener_to_broadcaster_with_dwarf(self):
|
||||
|
|
Loading…
Reference in New Issue