From 58f592c0d97d312308d0aac45ce48225aa91dff7 Mon Sep 17 00:00:00 2001 From: Eli Uriegas Date: Wed, 4 Oct 2017 10:43:27 -0700 Subject: [PATCH] Add note to test changes for rmi Signed-off-by: Eli Uriegas --- components/engine/integration-cli/docker_cli_rmi_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/components/engine/integration-cli/docker_cli_rmi_test.go b/components/engine/integration-cli/docker_cli_rmi_test.go index 2c6829925a..1b7c8cf585 100644 --- a/components/engine/integration-cli/docker_cli_rmi_test.go +++ b/components/engine/integration-cli/docker_cli_rmi_test.go @@ -152,7 +152,7 @@ func (s *DockerSuite) TestRmiImageIDForceWithRunningContainersAndMultipleTags(c out, _, err := dockerCmdWithError("rmi", "-f", imgID) // rmi -f should not delete image with running containers - c.Assert(err, checker.IsNil) + c.Assert(err, checker.IsNil) // NOTE: rmi -f fails silently on a real error see: https://github.com/docker/cli/issues/394 c.Assert(out, checker.Contains, "(cannot be forced) - image is being used by running container") } @@ -245,7 +245,7 @@ func (s *DockerSuite) TestRmiContainerImageNotFound(c *check.C) { // Try to remove the image of the running container and see if it fails as expected. out, _, err := dockerCmdWithError("rmi", "-f", imageIds[0]) // The image of the running container should not be removed. - c.Assert(err, checker.IsNil) + c.Assert(err, checker.IsNil) // NOTE: rmi -f fails silently on a real error see: https://github.com/docker/cli/issues/394 c.Assert(out, checker.Contains, "image is being used by running container", check.Commentf("out: %s", out)) }