[Lldb-commits] [lldb] [lldb] Fix permission issue in API test on lldb-x86_64-win (PR #187021)

[via lldb-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Stefan Gränitz (weliveindetail)

<details>
<summary>Changes</summary>

Deleting the executable at the end of this API test-case fails with a permission error, likely because lldb still holds a reference to the EXE. Exit explicitly to avoid that.

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


1 Files Affected:

- (modified) lldb/test/API/symstore/TestSymStoreLocal.py (+3-1) 


``````````diff
diff --git a/lldb/test/API/symstore/TestSymStoreLocal.py b/lldb/test/API/symstore/TestSymStoreLocal.py
index cedc76ac4354c..c8e900a1473dc 100644
--- a/lldb/test/API/symstore/TestSymStoreLocal.py
+++ b/lldb/test/API/symstore/TestSymStoreLocal.py
@@ -1,6 +1,5 @@
 import os
 import shutil
-import tempfile
 
 import lldb
 from lldbsuite.test.decorators import *
@@ -97,6 +96,7 @@ def test_no_symstore_url(self):
         exe, sym = self.build_inferior()
         with MockedSymStore(self, exe, sym):
             self.try_breakpoint(exe, should_have_loc=False)
+            self.runCmd("exit")
 
     def test_external_lookup_off(self):
         """
@@ -108,6 +108,7 @@ def test_external_lookup_off(self):
                 f"settings set plugin.symbol-locator.symstore.urls {symstore_dir}"
             )
             self.try_breakpoint(exe, ext_lookup=False, should_have_loc=False)
+            self.runCmd("exit")
 
     def test_local_dir(self):
         """
@@ -119,3 +120,4 @@ def test_local_dir(self):
                 f"settings set plugin.symbol-locator.symstore.urls {symstore_dir}"
             )
             self.try_breakpoint(exe, should_have_loc=True)
+            self.runCmd("exit")

``````````

</details>


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