[Lldb-commits] [lldb] Fix flaky TestDAP_console test. (PR #94494)

via lldb-commits lldb-commits at lists.llvm.org
Wed Jun 5 09:06:36 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Miro Bucko (mbucko)

<details>
<summary>Changes</summary>

Test Plan:
llvm-lit llvm-project/lldb/test/API/tools/lldb-dap/console/TestDAP_console.py

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


1 Files Affected:

- (modified) lldb/test/API/tools/lldb-dap/console/TestDAP_console.py (+4) 


``````````diff
diff --git a/lldb/test/API/tools/lldb-dap/console/TestDAP_console.py b/lldb/test/API/tools/lldb-dap/console/TestDAP_console.py
index e6345818bf087..fad4c31bb7658 100644
--- a/lldb/test/API/tools/lldb-dap/console/TestDAP_console.py
+++ b/lldb/test/API/tools/lldb-dap/console/TestDAP_console.py
@@ -141,6 +141,10 @@ def test_exit_status_message_sigterm(self):
 
         # Get the console output
         console_output = self.collect_console(1.0)
+        n_reads = 0
+        while n_reads < 10 and "exited with status" not in console_output:
+            console_output += self.collect_console(1.0)
+            n_reads += 1
 
         # Verify the exit status message is printed.
         self.assertIn(

``````````

</details>


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


More information about the lldb-commits mailing list