Skip to content

Conversation

@da-viper
Copy link
Contributor

The issue was fixed in #163821

@llvmbot
Copy link
Member

llvmbot commented Dec 10, 2025

@llvm/pr-subscribers-lldb

Author: Ebuka Ezike (da-viper)

Changes

The issue was fixed in #163821


Full diff: https://github.com/llvm/llvm-project/pull/171584.diff

1 Files Affected:

  • (modified) lldb/test/API/tools/lldb-dap/module/TestDAP_module.py (-2)
diff --git a/lldb/test/API/tools/lldb-dap/module/TestDAP_module.py b/lldb/test/API/tools/lldb-dap/module/TestDAP_module.py
index 0ed53dac5d869..0bf271a13ba5e 100644
--- a/lldb/test/API/tools/lldb-dap/module/TestDAP_module.py
+++ b/lldb/test/API/tools/lldb-dap/module/TestDAP_module.py
@@ -7,8 +7,6 @@
 import lldbdap_testcase
 import re
 
-# Flakey in Github CI runs, see https://github.com/llvm/llvm-project/issues/137660.
-@skipIfLinux
 class TestDAP_module(lldbdap_testcase.DAPTestCaseBase):
     def run_test(self, symbol_basename, expect_debug_info_size):
         program_basename = "a.out.stripped"

Copy link
Collaborator

@DavidSpickett DavidSpickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm assuming the general stability work fixed something here, and it's not the test case itself that was wrong as such. No harm trying it again.

LGTM.

@da-viper da-viper merged commit 36865a1 into llvm:main Dec 10, 2025
12 checks passed
@da-viper da-viper deleted the enable-module-test-linux branch December 10, 2025 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants