From cff346e7fdf591433bf750890f2537b4545bdc60 Mon Sep 17 00:00:00 2001
From: Mariano Cano <mariano@smallstep.com>
Date: Thu, 13 Feb 2020 15:49:16 -0800
Subject: [PATCH] Skip test on travis, it won't fail because they have access
 to the KMS.

---
 kms/cloudkms/cloudkms_test.go | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/kms/cloudkms/cloudkms_test.go b/kms/cloudkms/cloudkms_test.go
index 1776ee17..b4f92fa6 100644
--- a/kms/cloudkms/cloudkms_test.go
+++ b/kms/cloudkms/cloudkms_test.go
@@ -5,6 +5,7 @@ import (
 	"crypto"
 	"fmt"
 	"io/ioutil"
+	"os"
 	"reflect"
 	"testing"
 
@@ -54,16 +55,21 @@ func TestNew(t *testing.T) {
 		opts apiv1.Options
 	}
 	tests := []struct {
-		name    string
-		args    args
-		want    *CloudKMS
-		wantErr bool
+		name     string
+		skipOnCI bool
+		args     args
+		want     *CloudKMS
+		wantErr  bool
 	}{
-		{"fail authentication", args{context.Background(), apiv1.Options{}}, nil, true},
-		{"fail credentials", args{context.Background(), apiv1.Options{CredentialsFile: "testdata/missing"}}, nil, true},
+		{"fail authentication", true, args{context.Background(), apiv1.Options{}}, nil, true},
+		{"fail credentials", false, args{context.Background(), apiv1.Options{CredentialsFile: "testdata/missing"}}, nil, true},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
+			if tt.skipOnCI && os.Getenv("CI") == "true" {
+				t.SkipNow()
+			}
+
 			got, err := New(tt.args.ctx, tt.args.opts)
 			if (err != nil) != tt.wantErr {
 				t.Errorf("New() error = %v, wantErr %v", err, tt.wantErr)