[Lldb-commits] [lldb] 902814a - [lldb][test] Modernize TestModulesAutoImport (#194357)

[via lldb-commits]

Author: Raphael Isemann
Date: 2026-04-27T13:45:32+01:00
New Revision: 902814afe11799e2b90a9879658e1356b3ff6aff

URL: https://github.com/llvm/llvm-project/commit/902814afe11799e2b90a9879658e1356b3ff6aff
DIFF: https://github.com/llvm/llvm-project/commit/902814afe11799e2b90a9879658e1356b3ff6aff.diff

LOG: [lldb][test] Modernize TestModulesAutoImport (#194357)

This replaces all the custom test setup logic with the newer test
utilities. Not technically NFC as the newer checks are more strict.

Added: 
    

Modified: 
    lldb/test/API/lang/objc/modules-auto-import/TestModulesAutoImport.py
    lldb/test/API/lang/objc/modules-auto-import/main.m

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/lang/objc/modules-auto-import/TestModulesAutoImport.py b/lldb/test/API/lang/objc/modules-auto-import/TestModulesAutoImport.py
index f3558f62d51f8..9c8ba88bc6d83 100644
--- a/lldb/test/API/lang/objc/modules-auto-import/TestModulesAutoImport.py
+++ b/lldb/test/API/lang/objc/modules-auto-import/TestModulesAutoImport.py
@@ -1,6 +1,5 @@
 """Test that importing modules in Objective-C works as expected."""
 
-
 import lldb
 
 from lldbsuite.test.decorators import *
@@ -9,38 +8,14 @@
 
 
 class ObjCModulesAutoImportTestCase(TestBase):
-    def setUp(self):
-        # Call super's setUp().
-        TestBase.setUp(self)
-        # Find the line number to break inside main().
-        self.line = line_number("main.m", "// Set breakpoint 0 here.")
 
     @skipIf(macos_version=["<", "10.12"])
     @skipIf(compiler="clang", compiler_version=["<", "19.0"])
     def test_expr(self):
         self.build()
-        exe = self.getBuildArtifact("a.out")
-        self.runCmd("file " + exe, CURRENT_EXECUTABLE_SET)
-
-        # Break inside the foo function which takes a bar_ptr argument.
-        lldbutil.run_break_set_by_file_and_line(
-            self, "main.m", self.line, num_expected_locations=1, loc_exact=True
-        )
-
-        self.runCmd("run", RUN_SUCCEEDED)
-
-        # The stop reason of the thread should be breakpoint.
-        self.expect(
-            "thread list",
-            STOPPED_DUE_TO_BREAKPOINT,
-            substrs=["stopped", "stop reason = breakpoint"],
+        lldbutil.run_to_source_breakpoint(
+            self, "// break here", lldb.SBFileSpec("main.m", False)
         )
 
-        # The breakpoint should have a hit count of 1.
-        lldbutil.check_breakpoint(self, bpno=1, expected_hit_count=1)
-
         self.runCmd("settings set target.auto-import-clang-modules true")
-
-        self.expect(
-            "expression getpid()", VARIABLES_DISPLAYED_CORRECTLY, substrs=["pid_t"]
-        )
+        self.expect_expr("getpid()", result_type="pid_t")

diff  --git a/lldb/test/API/lang/objc/modules-auto-import/main.m b/lldb/test/API/lang/objc/modules-auto-import/main.m
index 5452ffd9bd15e..11508a0cd2029 100644
--- a/lldb/test/API/lang/objc/modules-auto-import/main.m
+++ b/lldb/test/API/lang/objc/modules-auto-import/main.m
@@ -1,7 +1,5 @@
 @import Darwin;
 
-int main()
-{
-    size_t ret = printf("Stop here\n"); // Set breakpoint 0 here.
-    return ret;
+int main() {
+    printf("Stop here\n"); // break here
 }