diff options
| author | Bryan C. Mills <bcmills@google.com> | 2019-10-11 08:58:52 -0400 |
|---|---|---|
| committer | Bryan C. Mills <bcmills@google.com> | 2019-10-11 13:25:59 +0000 |
| commit | d189bdd6846f1b7569d71cdcd5eeb77931f5c570 (patch) | |
| tree | 52fcaddc6434c9ae72ff66a97da95caf3b7c5357 /src | |
| parent | d0f10a6e68b3cd1a0bb8536317cae4e850f38ba5 (diff) | |
| download | go-d189bdd6846f1b7569d71cdcd5eeb77931f5c570.tar.xz | |
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>
Diffstat (limited to 'src')
| -rw-r--r-- | src/internal/reflectlite/reflect_mirror_test.go | 9 |
1 files changed, 8 insertions, 1 deletions
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) |