diff options
| author | Enrico Granata <egranata@apple.com> | 2014-10-17 01:11:29 +0000 |
|---|---|---|
| committer | Enrico Granata <egranata@apple.com> | 2014-10-17 01:11:29 +0000 |
| commit | cf3ab58e47769a8e6039c306f4efa56a30ee5737 (patch) | |
| tree | bdb22b508d9f95f770e9f5b3be817c9d02f7c695 /lldb/test/functionalities/inferior-assert | |
| parent | 4b2f7456adbd1276aaa30ab5614a8c9389a7286d (diff) | |
| download | bcm5719-llvm-cf3ab58e47769a8e6039c306f4efa56a30ee5737.tar.gz bcm5719-llvm-cf3ab58e47769a8e6039c306f4efa56a30ee5737.zip | |
Making all @expectedFailure markers take an explicit bugnumber annotation. This used to be optional, but that makes it harder to track what tests are failing for what reason. So, make it mandatory, in the form of refusing to run the test suite if annotations are missing
llvm-svn: 220012
Diffstat (limited to 'lldb/test/functionalities/inferior-assert')
| -rw-r--r-- | lldb/test/functionalities/inferior-assert/TestInferiorAssert.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/lldb/test/functionalities/inferior-assert/TestInferiorAssert.py b/lldb/test/functionalities/inferior-assert/TestInferiorAssert.py index f8ed37249b9..7b57d9722dd 100644 --- a/lldb/test/functionalities/inferior-assert/TestInferiorAssert.py +++ b/lldb/test/functionalities/inferior-assert/TestInferiorAssert.py @@ -16,7 +16,7 @@ class AssertingInferiorTestCase(TestBase): self.buildDsym() self.inferior_asserting() - @expectedFailurei386 # llvm.org/pr17384: lldb needs to be aware of linux-vdso.so to unwind stacks properly' + @expectedFailurei386("llvm.org/pr17384: lldb needs to be aware of linux-vdso.so to unwind stacks properly") @expectedFailureDarwin("rdar://15367233") def test_inferior_asserting_dwarf(self): """Test that lldb reliably catches the inferior asserting (command).""" @@ -34,9 +34,9 @@ class AssertingInferiorTestCase(TestBase): self.buildDwarf() self.inferior_asserting_registers() - @expectedFailurei386 # llvm.org/pr17384: lldb needs to be aware of linux-vdso.so to unwind stacks properly - @expectedFailureFreeBSD('llvm.org/pr18533') # PC in __assert frame is outside of function - @expectedFailureLinux('') # PC in __GI___assert_fail frame is just after the function (this is a no-return so there is no epilogue afterwards) + @expectedFailurei386("llvm.org/pr17384: lldb needs to be aware of linux-vdso.so to unwind stacks properly") + @expectedFailureFreeBSD('llvm.org/pr18533 - PC in __assert frame is outside of function') + @expectedFailureLinux("PC in __GI___assert_fail frame is just after the function (this is a no-return so there is no epilogue afterwards)") def test_inferior_asserting_disassemble(self): """Test that lldb reliably disassembles frames after asserting (command).""" self.buildDefault() @@ -55,7 +55,7 @@ class AssertingInferiorTestCase(TestBase): self.buildDsym() self.inferior_asserting_expr() - @expectedFailurei386 # llvm.org/pr17384: lldb needs to be aware of linux-vdso.so to unwind stacks properly + @expectedFailurei386('llvm.org/pr17384: lldb needs to be aware of linux-vdso.so to unwind stacks properly') @unittest2.expectedFailure("rdar://15367233") def test_inferior_asserting_expr(self): """Test that the lldb expression interpreter can read from the inferior after asserting (command).""" @@ -69,7 +69,7 @@ class AssertingInferiorTestCase(TestBase): self.buildDsym() self.inferior_asserting_step() - @expectedFailurei386 # llvm.org/pr17384: lldb needs to be aware of linux-vdso.so to unwind stacks properly + @expectedFailurei386("llvm.org/pr17384: lldb needs to be aware of linux-vdso.so to unwind stacks properly") @expectedFailureDarwin("rdar://15367233") def test_inferior_asserting_step(self): """Test that lldb functions correctly after stepping through a call to assert().""" |
