DavidSpickett created this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. DavidSpickett requested review of this revision. Herald added a subscriber: JDevlieghere.
init_llgs_test no longer takes an argument but these two were not updated. Also fix some mistakes in TestAutoInstallMainExecutable to get it passing again. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D91612 Files: lldb/test/API/commands/target/auto-install-main-executable/TestAutoInstallMainExecutable.py lldb/test/API/tools/lldb-server/platform-process-connect/TestPlatformProcessConnect.py Index: lldb/test/API/tools/lldb-server/platform-process-connect/TestPlatformProcessConnect.py =================================================================== --- lldb/test/API/tools/lldb-server/platform-process-connect/TestPlatformProcessConnect.py +++ lldb/test/API/tools/lldb-server/platform-process-connect/TestPlatformProcessConnect.py @@ -16,7 +16,7 @@ @expectedFailureAll(hostoslist=["windows"], triple='.*-android') def test_platform_process_connect(self): self.build() - self.init_llgs_test(False) + self.init_llgs_test() working_dir = lldb.remote_platform.GetWorkingDirectory() src = lldb.SBFileSpec(self.getBuildArtifact("a.out")) Index: lldb/test/API/commands/target/auto-install-main-executable/TestAutoInstallMainExecutable.py =================================================================== --- lldb/test/API/commands/target/auto-install-main-executable/TestAutoInstallMainExecutable.py +++ lldb/test/API/commands/target/auto-install-main-executable/TestAutoInstallMainExecutable.py @@ -19,7 +19,7 @@ @expectedFailureAll(hostoslist=["windows"], triple='.*-android') def test_target_auto_install_main_executable(self): self.build() - self.init_llgs_test(False) + self.init_llgs_test() # Manually install the modified binary. working_dir = lldb.remote_platform.GetWorkingDirectory() @@ -77,10 +77,10 @@ (os.path.join(working_dir,dest.GetFilename()), self.getBuildArtifact("a.out"))) - target = new_debugger.GetSelectedTarget() + target = self.dbg.GetSelectedTarget() breakpoint = target.BreakpointCreateByName("main") - launch_info = taget.GetLaunchInfo() + launch_info = target.GetLaunchInfo() error = lldb.SBError() process = target.Launch(launch_info, error) self.assertTrue(process, PROCESS_IS_VALID)
Index: lldb/test/API/tools/lldb-server/platform-process-connect/TestPlatformProcessConnect.py =================================================================== --- lldb/test/API/tools/lldb-server/platform-process-connect/TestPlatformProcessConnect.py +++ lldb/test/API/tools/lldb-server/platform-process-connect/TestPlatformProcessConnect.py @@ -16,7 +16,7 @@ @expectedFailureAll(hostoslist=["windows"], triple='.*-android') def test_platform_process_connect(self): self.build() - self.init_llgs_test(False) + self.init_llgs_test() working_dir = lldb.remote_platform.GetWorkingDirectory() src = lldb.SBFileSpec(self.getBuildArtifact("a.out")) Index: lldb/test/API/commands/target/auto-install-main-executable/TestAutoInstallMainExecutable.py =================================================================== --- lldb/test/API/commands/target/auto-install-main-executable/TestAutoInstallMainExecutable.py +++ lldb/test/API/commands/target/auto-install-main-executable/TestAutoInstallMainExecutable.py @@ -19,7 +19,7 @@ @expectedFailureAll(hostoslist=["windows"], triple='.*-android') def test_target_auto_install_main_executable(self): self.build() - self.init_llgs_test(False) + self.init_llgs_test() # Manually install the modified binary. working_dir = lldb.remote_platform.GetWorkingDirectory() @@ -77,10 +77,10 @@ (os.path.join(working_dir,dest.GetFilename()), self.getBuildArtifact("a.out"))) - target = new_debugger.GetSelectedTarget() + target = self.dbg.GetSelectedTarget() breakpoint = target.BreakpointCreateByName("main") - launch_info = taget.GetLaunchInfo() + launch_info = target.GetLaunchInfo() error = lldb.SBError() process = target.Launch(launch_info, error) self.assertTrue(process, PROCESS_IS_VALID)
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits