From 1fb1f857ac13798252e62980798de06d84fefae0 Mon Sep 17 00:00:00 2001 From: Alan Donovan Date: Fri, 8 Jan 2016 14:30:43 -0500 Subject: [PATCH] go/loader: use Skipf in test (late response to comment in review of CL 18452) Change-Id: I1fa80fd1c7fd22cec752c839d20a483e0732337c Reviewed-on: https://go-review.googlesource.com/18454 Reviewed-by: Alan Donovan --- go/loader/loader_test.go | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/go/loader/loader_test.go b/go/loader/loader_test.go index 197a66f1..1f05e183 100644 --- a/go/loader/loader_test.go +++ b/go/loader/loader_test.go @@ -399,9 +399,8 @@ func TestCwd(t *testing.T) { func TestLoad_vendor(t *testing.T) { if !go16 { - // Vendoring requires Go 1.6. // TODO(adonovan): delete in due course. - t.Skip() + t.Skipf("vendoring requires Go 1.6") } pkgs := map[string]string{ "a": `package a; import _ "x"`, @@ -437,9 +436,8 @@ func TestLoad_vendor(t *testing.T) { func TestVendorCwd(t *testing.T) { if !go16 { - // Vendoring requires Go 1.6. // TODO(adonovan): delete in due course. - t.Skip() + t.Skipf("vendoring requires Go 1.6") } // Test the interaction of cwd and vendor directories. ctxt := fakeContext(map[string]string{