diff --git a/integration-cli/build_tests/TestBuildAddTar/1/Dockerfile b/integration-cli/build_tests/TestBuildAddTar/1/Dockerfile deleted file mode 100644 index 2091b0e4d9..0000000000 --- a/integration-cli/build_tests/TestBuildAddTar/1/Dockerfile +++ /dev/null @@ -1,3 +0,0 @@ -FROM busybox -ADD test.tar /test.tar -RUN cat /test.tar/test/foo diff --git a/integration-cli/build_tests/TestBuildAddTar/2/Dockerfile b/integration-cli/build_tests/TestBuildAddTar/2/Dockerfile deleted file mode 100644 index 830e9ddbee..0000000000 --- a/integration-cli/build_tests/TestBuildAddTar/2/Dockerfile +++ /dev/null @@ -1,3 +0,0 @@ -FROM busybox -ADD test.tar / -RUN cat /test/foo diff --git a/integration-cli/build_tests/TestBuildAddTar/2/test.tar b/integration-cli/build_tests/TestBuildAddTar/2/test.tar deleted file mode 100644 index 33639c6476..0000000000 Binary files a/integration-cli/build_tests/TestBuildAddTar/2/test.tar and /dev/null differ diff --git a/integration-cli/build_tests/TestBuildAddTar/Dockerfile b/integration-cli/build_tests/TestBuildAddTar/Dockerfile new file mode 100644 index 0000000000..df4e9c9016 --- /dev/null +++ b/integration-cli/build_tests/TestBuildAddTar/Dockerfile @@ -0,0 +1,14 @@ +FROM busybox +ADD test.tar / +RUN cat /test/foo | grep Hi +ADD test.tar /test.tar +RUN cat /test.tar/test/foo | grep Hi +ADD test.tar /unlikely-to-exist +RUN cat /unlikely-to-exist/test/foo | grep Hi +ADD test.tar /unlikely-to-exist-trailing-slash/ +RUN cat /unlikely-to-exist-trailing-slash/test/foo | grep Hi +RUN mkdir /existing-directory +ADD test.tar /existing-directory +RUN cat /existing-directory/test/foo | grep Hi +ADD test.tar /existing-directory-trailing-slash/ +RUN cat /existing-directory-trailing-slash/test/foo | grep Hi diff --git a/integration-cli/build_tests/TestBuildAddTar/1/test.tar b/integration-cli/build_tests/TestBuildAddTar/test.tar similarity index 100% rename from integration-cli/build_tests/TestBuildAddTar/1/test.tar rename to integration-cli/build_tests/TestBuildAddTar/test.tar diff --git a/integration-cli/docker_cli_build_test.go b/integration-cli/docker_cli_build_test.go index 3b1c1c230e..66bbbf6dcd 100644 --- a/integration-cli/docker_cli_build_test.go +++ b/integration-cli/docker_cli_build_test.go @@ -1747,33 +1747,12 @@ RUN [ "$(cat /testfile)" = 'test!' ]` func TestBuildAddTar(t *testing.T) { name := "testbuildaddtar" defer deleteImages(name) - checkOutput := func(out string) { - n := -1 - x := "" - for i, line := range strings.Split(out, "\n") { - if strings.HasPrefix(line, "Step 2") { - n = i + 2 - x = line[strings.Index(line, "cat ")+4:] - } - if i == n { - if line != "Hi" { - t.Fatalf("Could not find contents of %s (expected 'Hi' got '%s'", x, line) - } - n = -2 - } - } - if n > -2 { - t.Fatalf("Could not find contents of %s in build output", x) - } - } - for _, n := range []string{"1", "2"} { - buildDirectory := filepath.Join(workingDirectory, "build_tests", "TestBuildAddTar", n) - buildCmd := exec.Command(dockerBinary, "build", "-t", name, ".") - buildCmd.Dir = buildDirectory - out, _, err := runCommandWithOutput(buildCmd) - errorOut(err, t, fmt.Sprintf("build failed to complete for TestBuildAddTar/%s: %v", n, err)) - checkOutput(out) - } + buildDirectory := filepath.Join(workingDirectory, "build_tests", "TestBuildAddTar") + buildCmd := exec.Command(dockerBinary, "build", "-t", name, ".") + buildCmd.Dir = buildDirectory + out, _, err := runCommandWithOutput(buildCmd) + errorOut(err, t, fmt.Sprintf("build failed to complete for TestBuildAddTar/%s: %v", n, err)) + logDone("build - ADD tar") }