This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGd327108d175f: [lldb/test] Split Scripted Process test in 
multiple tests (NFC) (authored by mib).

Changed prior to commit:
  https://reviews.llvm.org/D118513?vs=407269&id=407276#toc

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118513/new/

https://reviews.llvm.org/D118513

Files:
  lldb/test/API/functionalities/scripted_process/TestScriptedProcess.py
  lldb/test/API/functionalities/scripted_process/TestStackCoreScriptedProcess.py

Index: lldb/test/API/functionalities/scripted_process/TestStackCoreScriptedProcess.py
===================================================================
--- lldb/test/API/functionalities/scripted_process/TestStackCoreScriptedProcess.py
+++ lldb/test/API/functionalities/scripted_process/TestStackCoreScriptedProcess.py
@@ -10,7 +10,9 @@
 from lldbsuite.test import lldbutil
 from lldbsuite.test import lldbtest
 
-class ScriptedProcesTestCase(TestBase):
+class StackCoreScriptedProcesTestCase(TestBase):
+
+    NO_DEBUG_INFO_TESTCASE = True
 
     mydir = TestBase.compute_mydir(__file__)
 
@@ -20,117 +22,6 @@
     def tearDown(self):
         TestBase.tearDown(self)
 
-    def test_python_plugin_package(self):
-        """Test that the lldb python module has a `plugins.scripted_process`
-        package."""
-        self.expect('script import lldb.plugins',
-                    substrs=["ModuleNotFoundError"], matching=False)
-
-        self.expect('script dir(lldb.plugins)',
-                    substrs=["scripted_process"])
-
-        self.expect('script import lldb.plugins.scripted_process',
-                    substrs=["ModuleNotFoundError"], matching=False)
-
-        self.expect('script dir(lldb.plugins.scripted_process)',
-                    substrs=["ScriptedProcess"])
-
-        self.expect('script from lldb.plugins.scripted_process import ScriptedProcess',
-                    substrs=["ImportError"], matching=False)
-
-        self.expect('script dir(ScriptedProcess)',
-                    substrs=["launch"])
-
-    @skipUnlessDarwin
-    def test_invalid_scripted_register_context(self):
-        """Test that we can launch an lldb scripted process with an invalid
-        Scripted Thread, with invalid register context."""
-        self.build()
-        target = self.dbg.CreateTarget(self.getBuildArtifact("a.out"))
-        self.assertTrue(target, VALID_TARGET)
-        log_file = self.getBuildArtifact('thread.log')
-        self.runCmd("log enable lldb thread -f " + log_file)
-        self.assertTrue(os.path.isfile(log_file))
-
-        os.environ['SKIP_SCRIPTED_PROCESS_LAUNCH'] = '1'
-        def cleanup():
-          del os.environ["SKIP_SCRIPTED_PROCESS_LAUNCH"]
-        self.addTearDownHook(cleanup)
-
-        scripted_process_example_relpath = 'invalid_scripted_process.py'
-        self.runCmd("command script import " + os.path.join(self.getSourceDir(),
-                                                            scripted_process_example_relpath))
-
-        launch_info = lldb.SBLaunchInfo(None)
-        launch_info.SetProcessPluginName("ScriptedProcess")
-        launch_info.SetScriptedProcessClassName("invalid_scripted_process.InvalidScriptedProcess")
-        error = lldb.SBError()
-
-        process = target.Launch(launch_info, error)
-
-        self.assertTrue(error.Success(), error.GetCString())
-        self.assertTrue(process, PROCESS_IS_VALID)
-        self.assertEqual(process.GetProcessID(), 666)
-        self.assertEqual(process.GetNumThreads(), 0)
-
-        with open(log_file, 'r') as f:
-            log = f.read()
-
-        self.assertIn("Failed to get scripted thread registers data.", log)
-
-    @skipIf(archs=no_match(['x86_64', 'arm64', 'arm64e']))
-    def test_scripted_process_and_scripted_thread(self):
-        """Test that we can launch an lldb scripted process using the SBAPI,
-        check its process ID, read string from memory, check scripted thread
-        id, name stop reason and register context.
-        """
-        self.build()
-        target = self.dbg.CreateTarget(self.getBuildArtifact("a.out"))
-        self.assertTrue(target, VALID_TARGET)
-
-        os.environ['SKIP_SCRIPTED_PROCESS_LAUNCH'] = '1'
-        def cleanup():
-          del os.environ["SKIP_SCRIPTED_PROCESS_LAUNCH"]
-        self.addTearDownHook(cleanup)
-
-        scripted_process_example_relpath = 'dummy_scripted_process.py'
-        self.runCmd("command script import " + os.path.join(self.getSourceDir(),
-                                                            scripted_process_example_relpath))
-
-        launch_info = lldb.SBLaunchInfo(None)
-        launch_info.SetProcessPluginName("ScriptedProcess")
-        launch_info.SetScriptedProcessClassName("dummy_scripted_process.DummyScriptedProcess")
-
-        error = lldb.SBError()
-        process = target.Launch(launch_info, error)
-        self.assertTrue(process and process.IsValid(), PROCESS_IS_VALID)
-        self.assertEqual(process.GetProcessID(), 42)
-
-        self.assertEqual(process.GetNumThreads(), 1)
-
-        thread = process.GetSelectedThread()
-        self.assertTrue(thread, "Invalid thread.")
-        self.assertEqual(thread.GetThreadID(), 0x19)
-        self.assertEqual(thread.GetName(), "DummyScriptedThread.thread-1")
-        self.assertEqual(thread.GetStopReason(), lldb.eStopReasonSignal)
-
-        self.assertGreater(thread.GetNumFrames(), 0)
-
-        frame = thread.GetFrameAtIndex(0)
-        GPRs = None
-        register_set = frame.registers # Returns an SBValueList.
-        for regs in register_set:
-            if 'general purpose' in regs.name.lower():
-                GPRs = regs
-                break
-
-        self.assertTrue(GPRs, "Invalid General Purpose Registers Set")
-        self.assertGreater(GPRs.GetNumChildren(), 0)
-        for idx, reg in enumerate(GPRs, start=1):
-            if idx > 21:
-                break
-            self.assertEqual(idx, int(reg.value, 16))
-
     def create_stack_skinny_corefile(self, file):
         self.build()
         target, process, thread, _ = lldbutil.run_to_source_breakpoint(self, "// break here",
@@ -142,7 +33,6 @@
         self.assertTrue(self.dbg.DeleteTarget(target), "Couldn't delete target")
 
     @skipUnlessDarwin
-    @skipIf(archs=no_match(['arm64', 'arm64e']))
     @skipIfOutOfTreeDebugserver
     def test_launch_scripted_process_stack_frames(self):
         """Test that we can launch an lldb scripted process from the command
@@ -192,7 +82,7 @@
         self.assertEqual(process.GetProcessID(), 42)
 
         self.assertEqual(process.GetNumThreads(), 3)
-        thread = process.GetThreadAtIndex(2)
+        thread = process.GetSelectedThread()
         self.assertTrue(thread, "Invalid thread.")
         self.assertEqual(thread.GetName(), "StackCoreScriptedThread.thread-2")
 
Index: lldb/test/API/functionalities/scripted_process/TestScriptedProcess.py
===================================================================
--- lldb/test/API/functionalities/scripted_process/TestScriptedProcess.py
+++ lldb/test/API/functionalities/scripted_process/TestScriptedProcess.py
@@ -2,7 +2,7 @@
 Test python scripted process in lldb
 """
 
-import os, json, tempfile
+import os
 
 import lldb
 from lldbsuite.test.decorators import *
@@ -12,6 +12,8 @@
 
 class ScriptedProcesTestCase(TestBase):
 
+    NO_DEBUG_INFO_TESTCASE = True
+
     mydir = TestBase.compute_mydir(__file__)
 
     def setUp(self):
@@ -131,90 +133,3 @@
                 break
             self.assertEqual(idx, int(reg.value, 16))
 
-    def create_stack_skinny_corefile(self, file):
-        self.build()
-        target, process, thread, _ = lldbutil.run_to_source_breakpoint(self, "// break here",
-                                                                       lldb.SBFileSpec("main.cpp"))
-        self.assertTrue(process.IsValid(), "Process is invalid.")
-        # FIXME: Use SBAPI to save the process corefile.
-        self.runCmd("process save-core -s stack  " + file)
-        self.assertTrue(os.path.exists(file), "No stack-only corefile found.")
-        self.assertTrue(self.dbg.DeleteTarget(target), "Couldn't delete target")
-
-    @skipUnlessDarwin
-    @skipIf(archs=no_match(['arm64', 'arm64e']))
-    @skipIfOutOfTreeDebugserver
-    def test_launch_scripted_process_stack_frames(self):
-        """Test that we can launch an lldb scripted process from the command
-        line, check its process ID and read string from memory."""
-        self.build()
-        target = self.dbg.CreateTarget(self.getBuildArtifact("a.out"))
-        self.assertTrue(target, VALID_TARGET)
-
-        for module in target.modules:
-            if 'a.out' in module.GetFileSpec().GetFilename():
-                main_module = module
-                break
-
-        self.assertTrue(main_module, "Invalid main module.")
-        error = target.SetModuleLoadAddress(main_module, 0)
-        self.assertTrue(error.Success(), "Reloading main module at offset 0 failed.")
-
-        os.environ['SKIP_SCRIPTED_PROCESS_LAUNCH'] = '1'
-        def cleanup():
-          del os.environ["SKIP_SCRIPTED_PROCESS_LAUNCH"]
-        self.addTearDownHook(cleanup)
-
-        scripted_process_example_relpath = 'stack_core_scripted_process.py'
-        self.runCmd("command script import " + os.path.join(self.getSourceDir(),
-                                                            scripted_process_example_relpath))
-
-        corefile_process = None
-        with tempfile.NamedTemporaryFile() as file:
-            self.create_stack_skinny_corefile(file.name)
-            corefile_target = self.dbg.CreateTarget(None)
-            corefile_process = corefile_target.LoadCore(self.getBuildArtifact(file.name))
-        self.assertTrue(corefile_process, PROCESS_IS_VALID)
-
-        structured_data = lldb.SBStructuredData()
-        structured_data.SetFromJSON(json.dumps({
-            "backing_target_idx" : self.dbg.GetIndexOfTarget(corefile_process.GetTarget())
-        }))
-        launch_info = lldb.SBLaunchInfo(None)
-        launch_info.SetProcessPluginName("ScriptedProcess")
-        launch_info.SetScriptedProcessClassName("stack_core_scripted_process.StackCoreScriptedProcess")
-        launch_info.SetScriptedProcessDictionary(structured_data)
-
-        error = lldb.SBError()
-        process = target.Launch(launch_info, error)
-        self.assertTrue(error.Success(), error.GetCString())
-        self.assertTrue(process, PROCESS_IS_VALID)
-        self.assertEqual(process.GetProcessID(), 42)
-
-        self.assertEqual(process.GetNumThreads(), 3)
-        thread = process.GetThreadAtIndex(2)
-        self.assertTrue(thread, "Invalid thread.")
-        self.assertEqual(thread.GetName(), "StackCoreScriptedThread.thread-2")
-
-        self.assertTrue(target.triple, "Invalid target triple")
-        arch = target.triple.split('-')[0]
-        supported_arch = ['x86_64', 'arm64', 'arm64e']
-        self.assertIn(arch, supported_arch)
-        # When creating a corefile of a arm process, lldb saves the exception
-        # that triggers the breakpoint in the LC_NOTES of the corefile, so they
-        # can be reloaded with the corefile on the next debug session.
-        if arch in 'arm64e':
-            self.assertTrue(thread.GetStopReason(), lldb.eStopReasonException)
-        # However, it's architecture specific, and corefiles made from intel
-        # process don't save any metadata to retrieve to stop reason.
-        # To mitigate this, the StackCoreScriptedProcess will report a
-        # eStopReasonSignal with a SIGTRAP, mimicking what debugserver does.
-        else:
-            self.assertTrue(thread.GetStopReason(), lldb.eStopReasonSignal)
-
-        self.assertEqual(thread.GetNumFrames(), 6)
-        frame = thread.GetSelectedFrame()
-        self.assertTrue(frame, "Invalid frame.")
-        self.assertIn("bar", frame.GetFunctionName())
-        self.assertEqual(int(frame.FindValue("i", lldb.eValueTypeVariableArgument).GetValue()), 42)
-        self.assertEqual(int(frame.FindValue("j", lldb.eValueTypeVariableLocal).GetValue()), 42 * 42)
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to