[Lldb-commits] [lldb] 74338bf - A test was changing directory and then incorrectly restoring the directory

[Jim Ingham via lldb-commits]

Author: Jim Ingham
Date: 2023-09-19T16:46:08-07:00
New Revision: 74338bfe0cfec8b8db24af131cdeb664e346a1b5

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

LOG: A test was changing directory and then incorrectly restoring the directory
to the "testdir" which is the build directory for that test, not the
original source directory.  That caused subsequent tests to fail.

Added: 
    

Modified: 
    lldb/test/API/commands/process/attach/TestProcessAttach.py

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/commands/process/attach/TestProcessAttach.py b/lldb/test/API/commands/process/attach/TestProcessAttach.py
index 67bb134ec0148a0..99ff3a4ec4db816 100644
--- a/lldb/test/API/commands/process/attach/TestProcessAttach.py
+++ b/lldb/test/API/commands/process/attach/TestProcessAttach.py
@@ -74,7 +74,8 @@ def test_attach_to_process_from_
diff erent_dir_by_id(self):
         popen = self.spawnSubprocess(exe)
 
         os.chdir(newdir)
-        self.addTearDownHook(lambda: os.chdir(testdir))
+        sourcedir = self.getSourceDir()
+        self.addTearDownHook(lambda: os.chdir(sourcedir))
         self.runCmd("process attach -p " + str(popen.pid))
 
         target = self.dbg.GetSelectedTarget()