[Lldb-commits] [lldb] [lldb] fix SourceManagerTestCase freeze (PR #118095)

via lldb-commits lldb-commits at lists.llvm.org
Fri Nov 29 05:49:33 PST 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: None (dlav-sc)

<details>
<summary>Changes</summary>

Currently lldb testsuite requires some interactive input:

```
test_artificial_source_location (TestSourceManager.SourceManagerTestCase)
There is a running process, kill it and restart?: [Y/n]
```

This patch sets lldb auto-confirm flag to true before SourceManagerTestCase start, so the test doesn't freeze now.

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


1 Files Affected:

- (modified) lldb/test/API/source-manager/TestSourceManager.py (+3) 


``````````diff
diff --git a/lldb/test/API/source-manager/TestSourceManager.py b/lldb/test/API/source-manager/TestSourceManager.py
index ad7c85aac70eaf..6c15f69118a194 100644
--- a/lldb/test/API/source-manager/TestSourceManager.py
+++ b/lldb/test/API/source-manager/TestSourceManager.py
@@ -37,6 +37,9 @@ def setUp(self):
         # Find the line number to break inside main().
         self.file = self.getBuildArtifact("main-copy.c")
         self.line = line_number("main.c", "// Set break point at this line.")
+        # disable "There is a running process, kill it and restart?" prompt
+        self.runCmd("settings set auto-confirm true")
+        self.addTearDownHook(lambda: self.runCmd("settings clear auto-confirm"))
 
     def modify_content(self):
         # Read the main.c file content.

``````````

</details>


https://github.com/llvm/llvm-project/pull/118095


More information about the lldb-commits mailing list