skip unit test instead of uncommenting it

Author: jeroenvervaeke

internal/validate/validate_test.go

@@ -141,8 +141,9 @@ func TestObjectID(t *testing.T) {
 	}
 }
 
-/*
 func TestCredentials(t *testing.T) {
+	t.Skip("Will reenable on ticket CLOUDP-333193")
+
 	t.Run("no credentials", func(t *testing.T) {
 		if err := Credentials(); err == nil {
 			t.Fatal("Credentials() expected an error\n")
@@ -169,26 +170,27 @@ func TestCredentials(t *testing.T) {
 		}
 	})
 }
-*/
 
 func TestNoAPIKeys(t *testing.T) {
 	t.Run("no credentials", func(t *testing.T) {
 		if err := NoAPIKeys(); err != nil {
 			t.Fatalf("NoAPIKeys() unexpected error %v\n", err)
 		}
 	})
-	/*
-		t.Run("with api key credentials", func(t *testing.T) {
-			// this function depends on the global config (globals are bad I know)
-			// the easiest way we have to test it is via ENV vars
-			viper.AutomaticEnv()
-			t.Setenv("PUBLIC_API_KEY", "test")
-			t.Setenv("PRIVATE_API_KEY", "test")
-			if err := NoAPIKeys(); err == nil {
-				t.Fatalf("NoAPIKeys() expected error\n")
-			}
-		})
-	*/
+
+	t.Run("with api key credentials", func(t *testing.T) {
+		t.Skip("Will reenable on ticket CLOUDP-333193")
+
+		// this function depends on the global config (globals are bad I know)
+		// the easiest way we have to test it is via ENV vars
+		viper.AutomaticEnv()
+		t.Setenv("PUBLIC_API_KEY", "test")
+		t.Setenv("PRIVATE_API_KEY", "test")
+		if err := NoAPIKeys(); err == nil {
+			t.Fatalf("NoAPIKeys() expected error\n")
+		}
+	})
+
 	t.Run("with auth token credentials", func(t *testing.T) {
 		// this function depends on the global config (globals are bad I know)
 		// the easiest way we have to test it is via ENV vars
@@ -217,18 +219,19 @@ func TestNoAccessToken(t *testing.T) {
 			t.Fatalf("NoAccessToken() unexpected error %v\n", err)
 		}
 	})
-	/*
-		t.Run("with auth token credentials", func(t *testing.T) {
-			// this function depends on the global config (globals are bad I know)
-			// the easiest way we have to test it is via ENV vars
-			viper.AutomaticEnv()
-			t.Setenv("ACCESS_TOKEN", "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6IkpvaG4gRG9lIiwiaWF0IjoxNTE2MjM5MDIyfQ.SflKxwRJSMeKKF2QT4fwpMeJf36POk6yJV_adQssw5c")
-			t.Setenv("REFRESH_TOKEN", "test")
-			if err := NoAccessToken(); err == nil {
-				t.Fatalf("NoAccessToken() expected error\n")
-			}
-		})
-	*/
+
+	t.Run("with auth token credentials", func(t *testing.T) {
+		t.Skip("Will reenable on ticket CLOUDP-333193")
+
+		// this function depends on the global config (globals are bad I know)
+		// the easiest way we have to test it is via ENV vars
+		viper.AutomaticEnv()
+		t.Setenv("ACCESS_TOKEN", "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6IkpvaG4gRG9lIiwiaWF0IjoxNTE2MjM5MDIyfQ.SflKxwRJSMeKKF2QT4fwpMeJf36POk6yJV_adQssw5c")
+		t.Setenv("REFRESH_TOKEN", "test")
+		if err := NoAccessToken(); err == nil {
+			t.Fatalf("NoAccessToken() expected error\n")
+		}
+	})
 }
 
 func TestFlagInSlice(t *testing.T) {