[Lldb-commits] [lldb] 88c183e - [lldb] Fix TestDataFormatterGenericList

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Wed Dec 8 04:43:58 PST 2021


Author: Pavel Labath
Date: 2021-12-08T13:38:35+01:00
New Revision: 88c183e978ed0aeaaf9d10d5b2554ba0244b372f

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

LOG: [lldb] Fix TestDataFormatterGenericList

Test is using "next" commands to make progress in the process. D115137
added an additional statement to the program, without adding a command
to step over it. This only seemed to matter for the libc++ flavour of
the test, possibly because libstdc++ list is "empty" in its
uninitialized state.

Since moving with step commands is a treacherous, this patch adds a
run-to-breakpoint command to the test. It only does this for the
affected step, but one may consider doing it elsewhere too.

Added: 
    

Modified: 
    lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/list/TestDataFormatterGenericList.py

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/list/TestDataFormatterGenericList.py b/lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/list/TestDataFormatterGenericList.py
index d4303515c6f93..82fe84efc7b45 100644
--- a/lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/list/TestDataFormatterGenericList.py
+++ b/lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/list/TestDataFormatterGenericList.py
@@ -161,7 +161,9 @@ def cleanup():
 
         self.runCmd("type format delete int")
 
-        self.runCmd("n")
+        lldbutil.run_break_set_by_file_and_line(self, "main.cpp",
+                self.optional_line)
+        self.runCmd("continue")
 
         self.expect("frame variable text_list",
                     substrs=['size=0',
@@ -259,4 +261,4 @@ def test_with_run_command_libcpp(self):
 
     @add_test_categories(["libc++"])
     def test_ptr_and_ref_libcpp(self):
-        self.do_test_ptr_and_ref(USE_LIBCPP)
\ No newline at end of file
+        self.do_test_ptr_and_ref(USE_LIBCPP)


        


More information about the lldb-commits mailing list