[Lldb-commits] [lldb] afe497d - [lldb][test] Fix cleanup in test_save_linux_minidump_one_region (#231809)

[via lldb-commits]

Author: David Spickett
Date: 2026-10-14T10:03:37+01:00
New Revision: afe497d2d937fd9ada4df6be6c7545baaf3f6247

URL: https://github.com/llvm/llvm-project/commit/afe497d2d937fd9ada4df6be6c7545baaf3f6247
DIFF: https://github.com/llvm/llvm-project/commit/afe497d2d937fd9ada4df6be6c7545baaf3f6247.diff

LOG: [lldb][test] Fix cleanup in test_save_linux_minidump_one_region (#231809)

On Windows you cannot unlink a file that is open
in something else. So we must delete lldb's target before trying to
unlink the minidump file.

Another follow up to #231448. Same thing I did to the other tests but I
missed this one.

Added: 
    

Modified: 
    lldb/test/API/functionalities/process_save_core_minidump/TestProcessSaveCoreMinidump.py

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/functionalities/process_save_core_minidump/TestProcessSaveCoreMinidump.py b/lldb/test/API/functionalities/process_save_core_minidump/TestProcessSaveCoreMinidump.py
index 34de0f7ad1e38..431e4db9e9b70 100644
--- a/lldb/test/API/functionalities/process_save_core_minidump/TestProcessSaveCoreMinidump.py
+++ b/lldb/test/API/functionalities/process_save_core_minidump/TestProcessSaveCoreMinidump.py
@@ -406,6 +406,8 @@ def test_save_linux_minidump_one_region(self):
         self.build()
         exe = self.getBuildArtifact("a.out")
         one_region_file = self.getBuildArtifact("core.one_region.dmp")
+        target = None
+        core_target = None
         try:
             target = self.dbg.CreateTarget(exe)
             process = target.LaunchSimple(
@@ -452,7 +454,10 @@ def test_save_linux_minidump_one_region(self):
             self.assertTrue(err.Fail(), "Should fail to read memory")
 
         finally:
-            self.assertTrue(self.dbg.DeleteTarget(target))
+            if target:
+                self.assertTrue(self.dbg.DeleteTarget(target))
+            if core_target:
+                self.assertTrue(self.dbg.DeleteTarget(core_target))
             if os.path.isfile(one_region_file):
                 os.unlink(one_region_file)