Browse code
Windows: Disable TestAttachTTYWithoutStdin
Signed-off-by: John Howard <jhoward@microsoft.com>
John Howard authored on 2017/07/19 06:23:23Showing 1 changed files
| ... | ... |
@@ -88,6 +88,14 @@ func (s *DockerSuite) TestAttachMultipleAndRestart(c *check.C) {
|
| 88 | 88 |
} |
| 89 | 89 |
|
| 90 | 90 |
func (s *DockerSuite) TestAttachTTYWithoutStdin(c *check.C) {
|
| 91 |
+ // TODO @jhowardmsft. Figure out how to get this running again reliable on Windows. |
|
| 92 |
+ // It works by accident at the moment. Sometimes. I've gone back to v1.13.0 and see the same. |
|
| 93 |
+ // On Windows, docker run -d -ti busybox causes the container to exit immediately. |
|
| 94 |
+ // Obviously a year back when I updated the test, that was not the case. However, |
|
| 95 |
+ // with this, and the test racing with the tear-down which panic's, sometimes CI |
|
| 96 |
+ // will just fail and `MISS` all the other tests. For now, disabling it. Will |
|
| 97 |
+ // open an issue to track re-enabling this and root-causing the problem. |
|
| 98 |
+ testRequires(c, DaemonIsLinux) |
|
| 91 | 99 |
out, _ := dockerCmd(c, "run", "-d", "-ti", "busybox") |
| 92 | 100 |
|
| 93 | 101 |
id := strings.TrimSpace(out) |