changed format and added a test to run darwin

Author: Bar Soloveychik

lldb/source/API/SBModule.cpp

@@ -674,12 +674,8 @@ void SBModule::GarbageCollectAllocatedModules() {
 
 const char *SBModule::GetName() const {
   LLDB_INSTRUMENT_VA(this);
-  if (!m_opaque_sp) {
-    return nullptr;
-  }
-  auto mod_name = m_opaque_sp->GetObjectName();
-  if (!mod_name) {
-    return nullptr;
-  }
-  return mod_name.AsCString();
+
+  if (!m_opaque_sp)
+    return nullptr; 
+  return m_opaque_sp->GetObjectName().AsCString();
 }

lldb/test/API/python_api/sbmodule/TestSBModule.py

@@ -20,6 +20,17 @@ def tearDown(self):
 
     @skipUnlessDarwin
     @skipIfRemote
+    def test_getname(self):
+        """Test the SBModule::GetName() method"""
+        self.build()
+        target, _, _, _ = lldbutil.run_to_source_breakpoint(
+            self, "// break here", lldb.SBFileSpec("main.c")
+        )
+
+        self.assertGreater(target.GetNumModules(), 0)
+        module_names = {target.GetModuleAtIndex(i).GetName() for i in range(target.GetNumModules())}
+        self.assertIn("a.out", module_names)
+
     def test_module_is_file_backed(self):
         """Test the SBModule::IsFileBacked() method"""
         self.build()