1
0
Fork 0
mirror of https://github.com/moby/moby.git synced 2022-11-09 12:21:53 -05:00

integration-cli: fix some bashism's in Dockerfiles

`TestBuildBuildTimeArgEnv` and `TestBuildBuildTimeArgEmptyValVariants` were
using non-standard comparisons.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This commit is contained in:
Sebastiaan van Stijn 2019-09-02 10:51:18 +02:00
parent 234b951483
commit dbde4786e4
No known key found for this signature in database
GPG key ID: 76698F39D527CE8C

View file

@ -4535,17 +4535,17 @@ func (s *DockerSuite) TestBuildBuildTimeArgEnv(c *testing.T) {
ARG FOO6 ARG FOO6
ARG FO10 ARG FO10
RUN env RUN env
RUN [ "$FOO1" == "fromcmd" ] RUN [ "$FOO1" = "fromcmd" ]
RUN [ "$FOO2" == "" ] RUN [ "$FOO2" = "" ]
RUN [ "$FOO3" == "fromenv" ] RUN [ "$FOO3" = "fromenv" ]
RUN [ "$FOO4" == "fromfile" ] RUN [ "$FOO4" = "fromfile" ]
RUN [ "$FOO5" == "fromcmd" ] RUN [ "$FOO5" = "fromcmd" ]
# The following should not exist at all in the env # The following should not exist at all in the env
RUN [ "$(env | grep FOO6)" == "" ] RUN [ "$(env | grep FOO6)" = "" ]
RUN [ "$(env | grep FOO7)" == "" ] RUN [ "$(env | grep FOO7)" = "" ]
RUN [ "$(env | grep FOO8)" == "" ] RUN [ "$(env | grep FOO8)" = "" ]
RUN [ "$(env | grep FOO9)" == "" ] RUN [ "$(env | grep FOO9)" = "" ]
RUN [ "$FO10" == "" ] RUN [ "$FO10" = "" ]
` `
result := buildImage("testbuildtimeargenv", result := buildImage("testbuildtimeargenv",
cli.WithFlags( cli.WithFlags(
@ -4615,9 +4615,9 @@ func (s *DockerSuite) TestBuildBuildTimeArgEmptyValVariants(c *testing.T) {
ARG %s= ARG %s=
ARG %s="" ARG %s=""
ARG %s='' ARG %s=''
RUN [ "$%s" == "$%s" ] RUN [ "$%s" = "$%s" ]
RUN [ "$%s" == "$%s" ] RUN [ "$%s" = "$%s" ]
RUN [ "$%s" == "$%s" ]`, envKey, envKey1, envKey2, envKey, envKey1, envKey1, envKey2, envKey, envKey2) RUN [ "$%s" = "$%s" ]`, envKey, envKey1, envKey2, envKey, envKey1, envKey1, envKey2, envKey, envKey2)
buildImageSuccessfully(c, imgName, build.WithDockerfile(dockerfile)) buildImageSuccessfully(c, imgName, build.WithDockerfile(dockerfile))
} }