Skip to content

Commit

Permalink
Skip LaunchInTerminalTestCase test on remote systems
Browse files Browse the repository at this point in the history
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
  • Loading branch information
k15tfu committed Feb 27, 2015
1 parent aa069c2 commit d99b8f3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lldb/test/functionalities/tty/TestTerminal.py
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down

0 comments on commit d99b8f3

Please sign in to comment.