]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/go: convert TestIssue6844 to a script test
authorBryan C. Mills <bcmills@google.com>
Mon, 18 Nov 2019 19:28:15 +0000 (14:28 -0500)
committerBryan C. Mills <bcmills@google.com>
Tue, 19 Nov 2019 00:08:30 +0000 (00:08 +0000)
It was attempting to write a test binary to the working directory.

Updates #28387
Updates #30316

Change-Id: I82eca3a8a3e019dc6dacbe1f02a0583577694b93
Reviewed-on: https://go-review.googlesource.com/c/go/+/207614
Run-TryBot: Bryan C. Mills <bcmills@google.com>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/cmd/go/go_test.go
src/cmd/go/testdata/dep_test.go [deleted file]
src/cmd/go/testdata/script/test_rebuildall.txt [new file with mode: 0644]

index 0a5021e459fdf044c99b6db7d37dd9d0c33ef687..aaeb42d98c8b1be950c21114fa5cc9ef9195a887 100644 (file)
@@ -2711,19 +2711,6 @@ func TestIssue7108(t *testing.T) {
        tg.runFail("test", "notest")
 }
 
-// cmd/go: go test -a foo does not rebuild regexp.
-func TestIssue6844(t *testing.T) {
-       if testing.Short() {
-               t.Skip("don't rebuild the standard library in short mode")
-       }
-
-       tg := testgo(t)
-       defer tg.cleanup()
-       tg.creatingTemp("deps.test" + exeSuffix)
-       tg.run("test", "-x", "-a", "-c", "testdata/dep_test.go")
-       tg.grepStderr("regexp", "go test -x -a -c testdata/dep-test.go did not rebuild regexp")
-}
-
 func TestGoBuildTestOnly(t *testing.T) {
        tg := testgo(t)
        defer tg.cleanup()
diff --git a/src/cmd/go/testdata/dep_test.go b/src/cmd/go/testdata/dep_test.go
deleted file mode 100644 (file)
index ac39a5b..0000000
+++ /dev/null
@@ -1,7 +0,0 @@
-// Copyright 2014 The Go Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style
-// license that can be found in the LICENSE file.
-
-package deps
-
-import _ "testing"
diff --git a/src/cmd/go/testdata/script/test_rebuildall.txt b/src/cmd/go/testdata/script/test_rebuildall.txt
new file mode 100644 (file)
index 0000000..38233c1
--- /dev/null
@@ -0,0 +1,14 @@
+env GO111MODULE=off
+
+# Regression test for golang.org/issue/6844:
+# 'go test -a' should force dependencies in the standard library to be rebuilt.
+
+[short] skip
+
+go test -x -a -c testdata/dep_test.go
+stderr '^.*[/\\]compile'$GOEXE'["]? (.* )?regexp .*[/\\]regexp\.go'
+
+-- testdata/dep_test.go --
+package deps
+
+import _ "testing"