[Lldb-commits] [lldb] 8bd895c - [lldb/test] Use shorter test case names in TestStandardUnwind

Jordan Rupprecht via lldb-commits lldb-commits at lists.llvm.org
Fri Aug 28 11:50:43 PDT 2020


Author: Jordan Rupprecht
Date: 2020-08-28T11:49:50-07:00
New Revision: 8bd895cac0cd4eaf76b9bb296a995e5ee485205b

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

LOG: [lldb/test] Use shorter test case names in TestStandardUnwind

TestStandardUnwind uses the full absolute path to a set of C/C++ files as the test case name, which in turn is used in the name of a log file. When the source file is long, and the directory where log files are stored is also long, this causes an OSError because the log filename is too long.

Reviewed By: JDevlieghere

Differential Revision: https://reviews.llvm.org/D86752

Added: 
    

Modified: 
    lldb/test/API/functionalities/unwind/standard/TestStandardUnwind.py

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/functionalities/unwind/standard/TestStandardUnwind.py b/lldb/test/API/functionalities/unwind/standard/TestStandardUnwind.py
index 032b9e1aa618..fdf488978cbd 100644
--- a/lldb/test/API/functionalities/unwind/standard/TestStandardUnwind.py
+++ b/lldb/test/API/functionalities/unwind/standard/TestStandardUnwind.py
@@ -164,7 +164,7 @@ def test_function_dwarf(self, f=f):
                 self.skipTest("Inferior not supported")
             self.standard_unwind_tests()
 
-        test_name = "test_unwind_" + str(f)
+        test_name = "test_unwind_" + str(os.path.basename(f))
         for c in ".=()/\\":
             test_name = test_name.replace(c, '_')
 


        


More information about the lldb-commits mailing list