]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/go: only run TestGoGetHTTPS404 where it works
authorIan Lance Taylor <iant@golang.org>
Thu, 9 Jun 2016 00:45:55 +0000 (17:45 -0700)
committerIan Lance Taylor <iant@golang.org>
Thu, 9 Jun 2016 03:48:34 +0000 (03:48 +0000)
The test TestGoGetHTTPS404 downloads a package that does not build on
every OS, so change it to only run where the package builds. It's not
great for the test to depend on an external package, but this is an
improvement on the current situation.

Fixes #15644.

Change-Id: I1679cee5ab1e61a5b26f4ad39dc8a397fbc0da69
Reviewed-on: https://go-review.googlesource.com/23920
Run-TryBot: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Andrew Gerrand <adg@golang.org>
src/cmd/go/go_test.go

index 1bee5e6609826d17baab5a18bb85dbee0af254c7..c46e0c7da59506ed916876673f49524edd8f9c67 100644 (file)
@@ -2358,6 +2358,11 @@ func TestGoGetRscIoToolstash(t *testing.T) {
 // Issue 13037: Was not parsing <meta> tags in 404 served over HTTPS
 func TestGoGetHTTPS404(t *testing.T) {
        testenv.MustHaveExternalNetwork(t)
+       switch runtime.GOOS {
+       case "darwin", "linux", "freebsd":
+       default:
+               t.Skipf("test case does not work on %s", runtime.GOOS)
+       }
 
        tg := testgo(t)
        defer tg.cleanup()