[Lldb-commits] [lldb] r115826 - /lldb/trunk/test/hello_world/TestHelloWorld.py

[Johnny Chen]

Author: johnny
Date: Wed Oct  6 15:13:28 2010
New Revision: 115826

URL: http://llvm.org/viewvc/llvm-project?rev=115826&view=rev
Log:
Remove the @expectedFailure decorator since the SBTarget.LaunchProcess() race
condition has been fixed.

Modified:
    lldb/trunk/test/hello_world/TestHelloWorld.py

Modified: lldb/trunk/test/hello_world/TestHelloWorld.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/test/hello_world/TestHelloWorld.py?rev=115826&r1=115825&r2=115826&view=diff
==============================================================================
--- lldb/trunk/test/hello_world/TestHelloWorld.py (original)
+++ lldb/trunk/test/hello_world/TestHelloWorld.py Wed Oct  6 15:13:28 2010
@@ -18,7 +18,6 @@
         self.buildDsym()
         self.hello_world_python(useLaunchAPI = False)
 
-    @unittest2.expectedFailure
     def test_with_dwarf_and_process_launch_api(self):
         """Create target, breakpoint, launch a process, and then kill it.
 
@@ -53,9 +52,11 @@
 
         if useLaunchAPI:
             process = target.LaunchProcess([''], [''], os.ctermid(), 0, False)
+            # The following isn't needed anymore, rdar://8364687 is fixed.
+            #
             # Apply some dances after LaunchProcess() in order to break at "main".
             # It only works sometimes.
-            self.breakAfterLaunch(process, "main")
+            #self.breakAfterLaunch(process, "main")
         else:
             # On the other hand, the following line of code are more reliable.
             self.runCmd("run", setCookie=False)