DockerCLI/e2e/container
Kir Kolyshkin 149f69ca06
TestSigProxyWithTTY: fix
exec.CombinedOutput should not be used here because:
 - it redirects cmd Stdout and Stderr and we want it to be the tty
 - it calls cmd.Run which we already did

While at it
 - use pty.Start() as it is cleaner
 - make sure we don't leave a zombie running, by calling Wait() in defer
 - use test.Name() for containerName

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
(cherry picked from commit bc4ed69a23)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2019-10-29 15:20:53 +01:00
..
testdata Fix e2e test by using a pinned version of alpine 2017-09-14 11:38:46 -04:00
attach_test.go Update tests to use gotest.tools 👼 2018-06-08 18:24:26 +02:00
create_test.go Add missing test 2018-11-26 14:02:16 +00:00
kill_test.go Update tests to use gotest.tools 👼 2018-06-08 18:24:26 +02:00
main_test.go Add an end-to-end test for container run 2017-09-06 17:07:38 -04:00
proxy_signal_test.go TestSigProxyWithTTY: fix 2019-10-29 15:20:53 +01:00
run_test.go Update tests to use gotest.tools 👼 2018-06-08 18:24:26 +02:00