Skip to content

Commit efce7a1

Browse files
authored
[lldb-dap] Re-enable the lldb-dap tests (#138791)
Re-enable the lldb-dap tests. We've spent the last week improving the reliability of the test suite and the tests now pass reliably on macOS and Linux at desk. Let's see how things fare on the bots.
1 parent 92d3029 commit efce7a1

File tree

9 files changed

+6
-15
lines changed

9 files changed

+6
-15
lines changed

lldb/test/API/tools/lldb-dap/attach/TestDAP_attach.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ def spawn_and_wait(program, delay):
2424
process.wait()
2525

2626

27-
@skip
2827
class TestDAP_attach(lldbdap_testcase.DAPTestCaseBase):
2928
def set_and_hit_breakpoint(self, continueToExit=True):
3029
self.dap_server.wait_for_stopped()

lldb/test/API/tools/lldb-dap/attach/TestDAP_attachByPortNum.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
import socket
1919

2020

21+
@skip("https://github.com/llvm/llvm-project/issues/138803")
2122
class TestDAP_attachByPortNum(lldbdap_testcase.DAPTestCaseBase):
2223
default_timeout = 20
2324

lldb/test/API/tools/lldb-dap/breakpoint/TestDAP_breakpointLocations.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,7 @@
1111
import lldbdap_testcase
1212
import os
1313

14-
# DAP tests are flakey, see https://github.com/llvm/llvm-project/issues/137660.
15-
@skip
14+
1615
class TestDAP_breakpointLocations(lldbdap_testcase.DAPTestCaseBase):
1716
def setUp(self):
1817
lldbdap_testcase.DAPTestCaseBase.setUp(self)

lldb/test/API/tools/lldb-dap/breakpoint/TestDAP_setBreakpoints.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,7 @@
1111
import lldbdap_testcase
1212
import os
1313

14-
# DAP tests are flakey, see https://github.com/llvm/llvm-project/issues/137660.
15-
@skip
14+
1615
class TestDAP_setBreakpoints(lldbdap_testcase.DAPTestCaseBase):
1716
def setUp(self):
1817
lldbdap_testcase.DAPTestCaseBase.setUp(self)

lldb/test/API/tools/lldb-dap/commands/TestDAP_commands.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,6 @@
66
from lldbsuite.test.decorators import *
77

88

9-
# DAP tests are flakey, see https://github.com/llvm/llvm-project/issues/137660.
10-
@skip
119
class TestDAP_commands(lldbdap_testcase.DAPTestCaseBase):
1210
def test_command_directive_quiet_on_success(self):
1311
program = self.getBuildArtifact("a.out")

lldb/test/API/tools/lldb-dap/disassemble/TestDAP_disassemble.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,7 @@
1010
import lldbdap_testcase
1111
import os
1212

13-
# DAP tests are flakey, see https://github.com/llvm/llvm-project/issues/137660.
14-
@skip
13+
1514
class TestDAP_disassemble(lldbdap_testcase.DAPTestCaseBase):
1615
@skipIfWindows
1716
def test_disassemble(self):

lldb/test/API/tools/lldb-dap/evaluate/TestDAP_evaluate.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,6 @@
1111
from lldbsuite.test.lldbtest import *
1212

1313

14-
# DAP tests are flakey, see https://github.com/llvm/llvm-project/issues/137660.
15-
@skip
1614
class TestDAP_evaluate(lldbdap_testcase.DAPTestCaseBase):
1715
def assertEvaluate(self, expression, regex):
1816
self.assertRegex(

lldb/test/API/tools/lldb-dap/memory/TestDAP_memory.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,7 @@
1010
import lldbdap_testcase
1111
import os
1212

13-
# DAP tests are flakey, see https://github.com/llvm/llvm-project/issues/137660.
14-
@skip
13+
1514
class TestDAP_memory(lldbdap_testcase.DAPTestCaseBase):
1615
def test_memory_refs_variables(self):
1716
"""

lldb/test/API/tools/lldb-dap/variables/TestDAP_variables.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,7 @@ def make_buffer_verify_dict(start_idx, count, offset=0):
1717
verify_dict["[%i]" % (i)] = {"type": "int", "value": str(i + offset)}
1818
return verify_dict
1919

20-
# DAP tests are flakey, see https://github.com/llvm/llvm-project/issues/137660.
21-
@skip
20+
2221
class TestDAP_variables(lldbdap_testcase.DAPTestCaseBase):
2322
def verify_values(self, verify_dict, actual, varref_dict=None, expression=None):
2423
if "equals" in verify_dict:

0 commit comments

Comments
 (0)