[Lldb-commits] [lldb] [lldb] Add missing quit to TestClangREPL.py (PR #201058)

[via lldb-commits]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Raphael Isemann (Teemperor)

<details>
<summary>Changes</summary>

Without the explicit quit, the test runs until the pexpect instance times out after 1 minute.

This patch reduces the time of this test from >1m to about 10s.

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


1 Files Affected:

- (modified) lldb/test/API/repl/clang/TestClangREPL.py (+2) 


``````````diff
diff --git a/lldb/test/API/repl/clang/TestClangREPL.py b/lldb/test/API/repl/clang/TestClangREPL.py
index 97d91d1b8f7a5..60d8c9f839098 100644
--- a/lldb/test/API/repl/clang/TestClangREPL.py
+++ b/lldb/test/API/repl/clang/TestClangREPL.py
@@ -68,3 +68,5 @@ def test_completion_with_space_only_line(self):
         self.child.send("   ")
         self.child.send("\t")
         self.expect_repl("3 + 3", substrs=["(int) $0 = 6"])
+
+        self.quit()

``````````

</details>


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