[libc++] Make sure the cin/wcin tests run on remote hosts
authorLouis Dionne <ldionne@apple.com>
Wed, 6 May 2020 14:42:02 +0000 (10:42 -0400)
committerLouis Dionne <ldionne@apple.com>
Wed, 6 May 2020 15:33:13 +0000 (11:33 -0400)
When running on remote hosts, we need the whole `echo 123 | %t.exe` command
to run on the remote host. Thus, we need to escape the pipe to make sure
the command is treated as `{ echo 123 | %t.exe } > %t.out` instead of
`{ echo 123 } | %t.exe > %t.out`m where only `echo 123` is run on the
remote host.

libcxx/test/libcxx/selftest/newformat/shell-escape-pipes.sh.cpp [new file with mode: 0644]
libcxx/test/std/input.output/iostream.objects/narrow.stream.objects/cin.sh.cpp
libcxx/test/std/input.output/iostream.objects/wide.stream.objects/wcin.sh.cpp

diff --git a/libcxx/test/libcxx/selftest/newformat/shell-escape-pipes.sh.cpp b/libcxx/test/libcxx/selftest/newformat/shell-escape-pipes.sh.cpp
new file mode 100644 (file)
index 0000000..3badc8c
--- /dev/null
@@ -0,0 +1,22 @@
+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+// Make sure manually escaped pipes are handled properly by the shell.
+// Specifically, we want to make sure that if we escape a pipe after %{exec},
+// it gets executed by the %{exec} substitution, as opposed to the result of
+// the %{exec} substitution being piped into the following command.
+//
+// This is a bit tricky to test. To test this, we basically want to ensure
+// that both sides of the pipe are executed inside %{exec}. When we're inside
+// %{exec}, the one difference we can rely on is that we're in a temporary
+// directory with all file dependencies satisfied, so that's what we use.
+
+// FILE_DEPENDENCIES: %t.foobar
+// RUN: touch %t.foobar
+// RUN: %{exec} echo \| ls > %t.out
+// RUN: grep -e ".foobar" %t.out
index 386dbbd..4cadde6 100644 (file)
@@ -14,7 +14,7 @@
 
 // FILE_DEPENDENCIES: %t.exe
 // RUN: %{build}
-// RUN: %{exec} echo "123" | %t.exe > %t.out
+// RUN: %{exec} echo "123" \| %t.exe > %t.out
 // RUN: grep -e 'The number is 123!' %t.out
 
 #include <iostream>
index 9d24a37..0d0ab7e 100644 (file)
@@ -14,7 +14,7 @@
 
 // FILE_DEPENDENCIES: %t.exe
 // RUN: %{build}
-// RUN: %{exec} echo "123" | %t.exe > %t.out
+// RUN: %{exec} echo "123" \| %t.exe > %t.out
 // RUN: grep -e 'The number is 123!' %t.out
 
 #include <iostream>