From: Bryan C. Mills <bcmills@google.com>
Date: Fri, 11 Oct 2019 12:58:52 +0000 (-0400)
Subject: internal/reflectlite: skip TestMirrorWithReflect reflect package source is unavailable
X-Git-Tag: go1.14beta1~785
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=d189bdd6846f1b7569d71cdcd5eeb77931f5c570;p=gostls13.git

internal/reflectlite: skip TestMirrorWithReflect reflect package source is unavailable

Fixes #34847

Change-Id: Ibb58c8820cdab8b9f3755fcfba754ad0c9ca982f
Reviewed-on: https://go-review.googlesource.com/c/go/+/200657
Run-TryBot: Bryan C. Mills <bcmills@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Elias Naur <mail@eliasnaur.com>
---

diff --git a/src/internal/reflectlite/reflect_mirror_test.go b/src/internal/reflectlite/reflect_mirror_test.go
index 59d3ec407e..fbb6fb397e 100644
--- a/src/internal/reflectlite/reflect_mirror_test.go
+++ b/src/internal/reflectlite/reflect_mirror_test.go
@@ -87,6 +87,13 @@ func loadTypes(path, pkgName string, v visitor) {
 }
 
 func TestMirrorWithReflect(t *testing.T) {
+	reflectDir := filepath.Join(runtime.GOROOT(), "src", "reflect")
+	if _, err := os.Stat(reflectDir); os.IsNotExist(err) {
+		// On some mobile builders, the test binary executes on a machine without a
+		// complete GOROOT source tree.
+		t.Skipf("GOROOT source not present")
+	}
+
 	var wg sync.WaitGroup
 	rl, r := newVisitor(), newVisitor()
 
@@ -95,7 +102,7 @@ func TestMirrorWithReflect(t *testing.T) {
 		v         visitor
 	}{
 		{".", "reflectlite", rl},
-		{filepath.Join(runtime.GOROOT(), "src", "reflect"), "reflect", r},
+		{reflectDir, "reflect", r},
 	} {
 		tc := tc
 		wg.Add(1)