Skip LaunchInTerminalTestCase test on remote systems

k15tfu · k15tfu · commit d99b8f3ba8d1 · 2015-02-27T19:14:12.000Z
Summary: This ability was added by @jasonmolenda in [[ http://reviews.llvm.org/rL225748 | r225748 ]] but it was commented out because he hadn't test it. I tested it on OS X and now we can enable it legally. This change is made by @chying request. Reviewers: jasonmolenda, chying, clayborg Reviewed By: clayborg Subscribers: lldb-commits, chying, jasonmolenda, clayborg Differential Revision: http://reviews.llvm.org/D7930 llvm-svn: 230782
diff --git a/lldb/test/functionalities/tty/TestTerminal.py b/lldb/test/functionalities/tty/TestTerminal.py
@@ -22,9 +22,9 @@ class LaunchInTerminalTestCase(TestBase):
     # privilege so it can't open the socket to talk back to the test case
     @unittest2.skipUnless(os.geteuid() != 0, "test cannot be run as root")
 
-    # Do we need to disable this test if the testsuite is being run on a rmeote system?
+    # Do we need to disable this test if the testsuite is being run on a remote system?
     # This env var is only defined when the shell is running in a local mac terminal window
-    # @unittest2.skipUnless(os.environ.has_key('TERM_PROGRAM'), "test must be run on local system")
+    @unittest2.skipUnless(os.environ.has_key('TERM_PROGRAM'), "test must be run on local system")
 
     def test_launch_in_terminal (self):
         exe = "/bin/ls"
