1 files changed, 34 insertions, 13 deletions
diff --git a/packages/Python/lldbsuite/test/lang/cpp/gmodules/TestWithModuleDebugging.py b/packages/Python/lldbsuite/test/lang/cpp/gmodules/TestWithModuleDebugging.py
index 942149f6173b..dcc9206867b3 100644
--- a/packages/Python/lldbsuite/test/lang/cpp/gmodules/TestWithModuleDebugging.py
+++ b/packages/Python/lldbsuite/test/lang/cpp/gmodules/TestWithModuleDebugging.py
@@ -1,14 +1,15 @@
-import lldb, os
+import lldb
+import os
 from lldbsuite.test.decorators import *
 from lldbsuite.test.lldbtest import *
 from lldbsuite.test import lldbutil
 
+
 class TestWithGmodulesDebugInfo(TestBase):
 
     mydir = TestBase.compute_mydir(__file__)
 
     @add_test_categories(["gmodules"])
-    @expectedFailureAll(oslist=["macosx"], bugnumber="llvm.org/pr28156")
     def test_specialized_typedef_from_pch(self):
         self.build()
         cwd = os.getcwd()
@@ -18,33 +19,53 @@ class TestWithGmodulesDebugInfo(TestBase):
         self.assertTrue(src_file_spec.IsValid(), "breakpoint file")
 
         # Get the path of the executable
-        exe_path  = os.path.join(cwd, 'a.out')
+        exe_path = os.path.join(cwd, 'a.out')
 
         # Load the executable
         target = self.dbg.CreateTarget(exe_path)
         self.assertTrue(target.IsValid(), VALID_TARGET)
 
         # Break on interesting line
-        breakpoint = target.BreakpointCreateBySourceRegex("break here", src_file_spec)
-        self.assertTrue(breakpoint.IsValid() and breakpoint.GetNumLocations() >= 1, VALID_BREAKPOINT)
+        breakpoint = target.BreakpointCreateBySourceRegex(
+            "break here", src_file_spec)
+        self.assertTrue(
+            breakpoint.IsValid() and breakpoint.GetNumLocations() >= 1,
+            VALID_BREAKPOINT)
 
         # Launch the process
-        process = target.LaunchSimple(None, None, self.get_process_working_directory())
+        process = target.LaunchSimple(
+            None, None, self.get_process_working_directory())
         self.assertTrue(process.IsValid(), PROCESS_IS_VALID)
 
         # Get the thread of the process
         self.assertTrue(process.GetState() == lldb.eStateStopped)
-        thread = lldbutil.get_stopped_thread(process, lldb.eStopReasonBreakpoint)
-        self.assertTrue(thread.IsValid(), "There should be a thread stopped due to breakpoint condition")
+        thread = lldbutil.get_stopped_thread(
+            process, lldb.eStopReasonBreakpoint)
+        self.assertTrue(
+            thread.IsValid(),
+            "There should be a thread stopped due to breakpoint condition")
 
         # Get frame for current thread
         frame = thread.frames[0]
 
         testValue = frame.EvaluateExpression("test")
-        self.assertTrue(testValue.GetError().Success(), "Test expression value invalid: %s" % (testValue.GetError().GetCString()))
-        self.assertTrue(testValue.GetTypeName() == "IntContainer", "Test expression type incorrect")
+        self.assertTrue(
+            testValue.GetError().Success(),
+            "Test expression value invalid: %s" %
+            (testValue.GetError().GetCString()))
+        self.assertTrue(
+            testValue.GetTypeName() == "IntContainer",
+            "Test expression type incorrect")
 
         memberValue = testValue.GetChildMemberWithName("storage")
-        self.assertTrue(memberValue.GetError().Success(), "Member value missing or invalid: %s" % (testValue.GetError().GetCString()))
-        self.assertTrue(memberValue.GetTypeName() == "int", "Member type incorrect")
-        self.assertEqual(42, memberValue.GetValueAsSigned(), "Member value incorrect")
+        self.assertTrue(
+            memberValue.GetError().Success(),
+            "Member value missing or invalid: %s" %
+            (testValue.GetError().GetCString()))
+        self.assertTrue(
+            memberValue.GetTypeName() == "int",
+            "Member type incorrect")
+        self.assertEqual(
+            42,
+            memberValue.GetValueAsSigned(),
+            "Member value incorrect")