[lldb][test] 3 - Add gmodules XFAIL test for lang/cpp/gmodules/templates

[Michael137]

Summary

Since we don't implicitly compile tests with gmodules now,
add a gmodules test category to this test explicitly.

[Michael137]

• Fix breakpoint and move docstring

lldb/test/API/lang/cpp/gmodules/templates/Makefile

+PCH_CXX_SOURCES = a.h b.h
+CXX_SOURCES    := main.cpp
+
+include Makefile.rules

lldb/test/API/lang/cpp/gmodules/templates/TestGModules.py

-import lldbsuite.test.lldbinline as lldbinline
+import lldb
 from lldbsuite.test.decorators import *
+from lldbsuite.test.lldbtest import *
+from lldbsuite.test import lldbutil
 
-lldbinline.MakeInlineTest(__file__, globals(), [
-    expectedFailureAll(oslist=["linux"], bugnumber="llvm.org/pr36107",
-        debug_info="gmodules")])
+class TestGModules(TestBase):
+
+    @add_test_categories(["gmodules"])
+    @expectedFailureAll
+    def test_gmodules(self):
+        self.build()
+        lldbutil.run_to_source_breakpoint(self, "Break here", lldb.SBFileSpec("main.cpp"))
+
+        name = lldbutil.frame().FindVariable('m') \
+                .GetChildAtIndex(0).GetChildAtIndex(0).GetChildAtIndex(0).GetName()
+
+        self.assertEqual(name, 'buffer', 'find template specializations in imported modules')

lldb/test/API/lang/cpp/gmodules/templates/main.cpp

 #include "b.h"
 
 int main(int argc, const char * argv[])
 {
     Module m;
     // Test that the type Module which contains a field that is a
     // template instantiation can be fully resolved.
-    return 0; //% self.assertTrue(self.frame().FindVariable('m').GetChildAtIndex(0).GetChildAtIndex(0).GetChildAtIndex(0).GetName() == 'buffer', 'find template specializations in imported modules')
+    return 0;
 }