Author: Med Ismail Bennani Date: 2022-01-24T21:48:31+01:00 New Revision: c3ca2c6b14f91f8232525373c4f5b1dc504a39a1
URL: https://github.com/llvm/llvm-project/commit/c3ca2c6b14f91f8232525373c4f5b1dc504a39a1 DIFF: https://github.com/llvm/llvm-project/commit/c3ca2c6b14f91f8232525373c4f5b1dc504a39a1.diff LOG: [lldb/test] Fix `TestScriptedProcess.test_scripted_process_and_scripted_thread` This patch updates `dummy_scripted_process.py` to report the dummy thread correctly to reflect the changes introduced by `d3e0f7e`. Signed-off-by: Med Ismail Bennani <medismail.benn...@gmail.com> Added: Modified: lldb/test/API/functionalities/scripted_process/TestScriptedProcess.py lldb/test/API/functionalities/scripted_process/dummy_scripted_process.py Removed: ################################################################################ diff --git a/lldb/test/API/functionalities/scripted_process/TestScriptedProcess.py b/lldb/test/API/functionalities/scripted_process/TestScriptedProcess.py index 4831d48a0b5a..0c215f082c5d 100644 --- a/lldb/test/API/functionalities/scripted_process/TestScriptedProcess.py +++ b/lldb/test/API/functionalities/scripted_process/TestScriptedProcess.py @@ -41,6 +41,7 @@ def test_python_plugin_package(self): 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.""" @@ -77,7 +78,7 @@ def cleanup(): self.assertIn("Failed to get scripted thread registers data.", log) - @skipIf(archs=no_match(['x86_64'])) + @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 @@ -124,8 +125,10 @@ def cleanup(): break self.assertTrue(GPRs, "Invalid General Purpose Registers Set") - self.assertEqual(GPRs.GetNumChildren(), 21) + 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): diff --git a/lldb/test/API/functionalities/scripted_process/dummy_scripted_process.py b/lldb/test/API/functionalities/scripted_process/dummy_scripted_process.py index d7f428d40845..67850cf57a73 100644 --- a/lldb/test/API/functionalities/scripted_process/dummy_scripted_process.py +++ b/lldb/test/API/functionalities/scripted_process/dummy_scripted_process.py @@ -9,6 +9,7 @@ class DummyScriptedProcess(ScriptedProcess): def __init__(self, target: lldb.SBTarget, args : lldb.SBStructuredData): super().__init__(target, args) + self.threads[0] = DummyScriptedThread(self, None) def get_memory_region_containing_address(self, addr: int) -> lldb.SBMemoryRegionInfo: return None _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
