[Lldb-commits] [lldb] 090205f - [lldb] Fix TestStepOverWatchpoint

[Pavel Labath via lldb-commits]

Author: Pavel Labath
Date: 2023-03-21T10:45:12+01:00
New Revision: 090205fb57a1ca65bec6fada32232c2e975d0c48

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

LOG: [lldb] Fix TestStepOverWatchpoint

Added: 
    

Modified: 
    lldb/test/API/commands/watchpoints/step_over_watchpoint/TestStepOverWatchpoint.py

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/commands/watchpoints/step_over_watchpoint/TestStepOverWatchpoint.py b/lldb/test/API/commands/watchpoints/step_over_watchpoint/TestStepOverWatchpoint.py
index fd70bd692a216..f88428b872c00 100644
--- a/lldb/test/API/commands/watchpoints/step_over_watchpoint/TestStepOverWatchpoint.py
+++ b/lldb/test/API/commands/watchpoints/step_over_watchpoint/TestStepOverWatchpoint.py
@@ -34,13 +34,13 @@ def get_to_start(self, bkpt_text):
         # stepping off from the breakpoint:
         bkpt.SetEnabled(False)
         
-        return (target, process, thread, read_watchpoint)
+        return (target, process, thread, frame, read_watchpoint)
     
     # Read-write watchpoints not supported on SystemZ
     @expectedFailureAll(archs=['s390x'])
     @add_test_categories(["basic_process"])
     def test_step_over(self):
-        target, process, thread, wp = self.get_to_start("Set a breakpoint here")
+        target, process, thread, frame, wp = self.get_to_start("Set a breakpoint here")
     
         thread.StepOver()
         self.assertStopReason(thread.GetStopReason(), lldb.eStopReasonWatchpoint,
@@ -61,7 +61,7 @@ def test_step_over(self):
         bugnumber="<rdar://problem/34027183>")
     @add_test_categories(["basic_process"])
     def test_step_instruction(self):
-        target, process, thread, wp = self.get_to_start("Set breakpoint after call")
+        target, process, thread, frame, wp = self.get_to_start("Set breakpoint after call")
 
         self.step_inst_for_watchpoint(1)
 
@@ -75,6 +75,7 @@ def test_step_instruction(self):
         if re.match("^mips", arch) or re.match("powerpc64le", arch):
             self.runCmd("watchpoint delete 1")
 
+        error = lldb.SBError()
         # resolve_location=True, read=False, write=True
         write_watchpoint = write_value.Watch(True, False, True, error)
         self.assertTrue(write_watchpoint, "Failed to set write watchpoint.")