Fix TestAttachDenied.py remote execution on Linux.

[ovyalov]

Summary

TestAttachDenied should create the pipe on a target machine in case of remote execution - use SBPlatform::Run to create/read/remove a pipe either on local or remote machine.

[ovyalov]

Please let me know if you have any suggestions here - otherwise will be submitting it soon.

[vharron]

test/functionalities/process_attach/attach_denied/TestAttachDenied.py

31

We want to be able to run this test from Windows against remote targets that support cat/mkfifo/rm. We need a new test decorator skipIfWindowsTarget (meaning Windows host is okay). Doesn't need to be in this CL.

[ovyalov]

test/functionalities/process_attach/attach_denied/TestAttachDenied.py

31

OK, makes sense - let me get to this once I'll setup Windows station.

[ovyalov]

Committed - http://reviews.llvm.org/rL229587

test/functionalities/process_attach/attach_denied/TestAttachDenied.py

 """
 
 import os
-import shutil
-import tempfile
+import time
 import unittest2
 import lldb
 from lldbtest import *

 
     mydir = TestBase.compute_mydir(__file__)
 
+    def run_platform_command(self, cmd):
+        platform = self.dbg.GetSelectedPlatform()
+        shell_command = lldb.SBPlatformShellCommand(cmd)
+        err = platform.Run(shell_command)
+        return (err, shell_command.GetOutput())
+
     @skipIfWindows
     def test_attach_to_process_by_id_denied(self):
         """Test attach by process id denied"""

         self.buildDefault()
         exe = os.path.join(os.getcwd(), exe_name)
 
-        temp_dir = tempfile.mkdtemp()
-        self.addTearDownHook(lambda: shutil.rmtree(temp_dir))
-
         # Use named pipe as a synchronization point between test and inferior.
-        pid_pipe_path = os.path.join(temp_dir, "pid_pipe")
-        os.mkfifo(pid_pipe_path)
+        pid_pipe_path = os.path.join(self.get_process_working_directory(),
+                                     "pid_pipe_%d" % (int(time.time())))
+
+        err, _ = self.run_platform_command("mkfifo %s" % (pid_pipe_path))
+        self.assertTrue(err.Success(), "Failed to create FIFO %s: %s" % (pid_pipe_path, err.GetCString()))
+
+        self.addTearDownHook(lambda: self.run_platform_command("rm %s" % (pid_pipe_path)))
 
         # Spawn a new process
         popen = self.spawnSubprocess(exe, [pid_pipe_path])
         self.addTearDownHook(self.cleanupSubprocesses)
 
-        pid_pipe = open(pid_pipe_path, 'r')
-        self.addTearDownHook(lambda: pid_pipe.close())
-        pid = pid_pipe.read()
+        err, pid = self.run_platform_command("cat %s" % (pid_pipe_path))
+        self.assertTrue(err.Success(), "Failed to read FIFO %s: %s" % (pid_pipe_path, err.GetCString()))
 
         self.expect('process attach -p ' + pid,
                     startstr = 'error: attach failed:',