From 0deb18ab42094f897d9996aa23fabe3c6a76a67f Mon Sep 17 00:00:00 2001 From: Kir Kolyshkin Date: Sat, 27 Apr 2019 04:38:55 +0000 Subject: [PATCH] Dockerfile.e2e: copy test sources Package "gotest.tools/assert" uses source introspection to print more info in case of assertion failure. When source code is not available, it prints an error instead. In other words, before this commit: > --- SKIP: TestCgroupDriverSystemdMemoryLimit (0.00s) > cgroupdriver_systemd_test.go:32: failed to parse source file: /go/src/github.com/docker/docker/integration/system/cgroupdriver_systemd_test.go: open /go/src/github.com/docker/docker/integration/system/cgroupdriver_systemd_test.go: no such file or directory > cgroupdriver_systemd_test.go:32: and after: > --- SKIP: TestCgroupDriverSystemdMemoryLimit (0.09s) > cgroupdriver_systemd_test.go:32: !hasSystemd() This increases the resulting image size by about 2 MB on my system (from 758 to 760 MB). Signed-off-by: Kir Kolyshkin --- Dockerfile.e2e | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Dockerfile.e2e b/Dockerfile.e2e index d1bd9a37c1..a1b5ab7934 100644 --- a/Dockerfile.e2e +++ b/Dockerfile.e2e @@ -43,8 +43,11 @@ FROM base AS builder ARG DOCKER_GITCOMMIT ENV DOCKER_GITCOMMIT=${DOCKER_GITCOMMIT:-undefined} COPY . . +# Copy test sources tests that use assert can print errors +RUN mkdir -p /build${PWD} && find integration integration-cli -name \*_test.go -exec cp --parents '{}' /build${PWD} \; +# Build and install test binaries RUN hack/make.sh build-integration-test-binary -RUN mkdir -p /build/ && find . -name test.main -exec cp --parents '{}' /build \; +RUN mkdir -p /build/tests && find . -name test.main -exec cp --parents '{}' /build/tests \; ## Generate testing image FROM alpine:3.9 as runner @@ -77,4 +80,4 @@ COPY integration-cli/fixtures /tests/integration-cli/fixtures COPY --from=frozen-images /build/ /docker-frozen-images COPY --from=dockercli /build/ /usr/bin/ COPY --from=contrib /build/ /tests/contrib/ -COPY --from=builder /build/ /tests/ +COPY --from=builder /build/ /