19146: Remove unneeded special case checks, explain the needed one.

[arvados.git] / lib / install / deps_test.go

diff --git a/lib/install/deps_test.go b/lib/install/deps_test.go
index d3e63986f6cb639811174611cf6dba60a34912b7..993e779e5b7944e1da6d68cac34b4d8f00154b33 100644 (file)
--- a/lib/install/deps_test.go
+++ b/lib/install/deps_test.go
@@ -2,7 +2,11 @@
 //
 // SPDX-License-Identifier: AGPL-3.0
 
-// Skip this slow test unless invoked as "go test -tags docker":
+// Skip this slow test unless invoked as "go test -tags docker".
+// Depending on host/network speed, Go's default 10m test timeout
+// might be too short; recommend "go test -timeout 20m -tags docker".
+//
+//go:build docker
 // +build docker
 
 package install
@@ -26,15 +30,15 @@ func (*Suite) TestInstallDeps(c *check.C) {
        tmp := c.MkDir()
        script := `
 set -x
-export GOPATH=${GOPATH:-${HOME}/go}
 tmp="` + tmp + `"
 sourcepath="$(realpath ../..)"
 (cd ${sourcepath} && go build -o ${tmp} ./cmd/arvados-server)
 docker run -i --rm --workdir /arvados \
        -v ${tmp}/arvados-server:/arvados-server:ro \
        -v ${sourcepath}:/arvados:ro \
-       -v /arvados/services/api/.bundle \
        -v /arvados/apps/workbench/.bundle \
+       -v /arvados/services/api/.bundle \
+       -v /arvados/services/api/tmp \
        --env http_proxy \
        --env https_proxy \
        debian:10 \