Fix tests

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>

Author: antgamdia

cmd/kubeapps-apis/core/plugins/v1alpha1/plugins_test.go

@@ -360,18 +360,18 @@ func TestCreateConfigGetterWithParams(t *testing.T) {
 		{
 			name:           "it doesn't create the config and throws a grpc error when passing an invalid authorization metadata",
 			headers:        http.Header{"Authorization": []string{"Bla"}},
-			expectedErrMsg: connect.NewError(connect.CodeUnauthenticated, fmt.Errorf("invalid authorization metadata: malformed authorization metadata")),
+			expectedErrMsg: connect.NewError(connect.CodeUnauthenticated, fmt.Errorf("Invalid authorization metadata: malformed authorization metadata")),
 		},
 		{
 			name:            "it doesn't create the config and throws a grpc error for the default cluster when no authorization metadata is passed",
 			expectedAPIHost: DefaultK8sAPI,
-			expectedErrMsg:  connect.NewError(connect.CodeUnauthenticated, fmt.Errorf("invalid authorization metadata: missing authorization metadata")),
+			expectedErrMsg:  connect.NewError(connect.CodeUnauthenticated, fmt.Errorf("Invalid authorization metadata: missing authorization metadata")),
 		},
 		{
 			name:            "it doesn't create the config and throws a grpc error for the other cluster",
 			cluster:         OtherClusterName,
 			expectedAPIHost: OtherK8sAPI,
-			expectedErrMsg:  connect.NewError(connect.CodeUnauthenticated, fmt.Errorf("invalid authorization metadata: missing authorization metadata")),
+			expectedErrMsg:  connect.NewError(connect.CodeUnauthenticated, fmt.Errorf("Invalid authorization metadata: missing authorization metadata")),
 		},
 	}
 

cmd/kubeapps-apis/plugins/kapp_controller/packages/v1alpha1/server_test.go

@@ -6701,7 +6701,7 @@ func TestAddPackageRepository(t *testing.T) {
 				return request
 			},
 			expectedErrorCode:   connect.CodeInvalidArgument,
-			expectedErrorString: "secret name is not provided",
+			expectedErrorString: "Secret name is not provided",
 		},
 		{
 			name: "validate auth (user managed, secret does not exist)",
@@ -6717,7 +6717,7 @@ func TestAddPackageRepository(t *testing.T) {
 				return request
 			},
 			expectedErrorCode:   connect.CodeInvalidArgument,
-			expectedErrorString: "not found",
+			expectedErrorString: "Not found",
 		},
 		{
 			name: "validate auth (user managed, secret is incompatible)",
@@ -6739,7 +6739,7 @@ func TestAddPackageRepository(t *testing.T) {
 				return request
 			},
 			expectedErrorCode:   connect.CodeInvalidArgument,
-			expectedErrorString: "secret does not match",
+			expectedErrorString: "Secret does not match",
 		},
 		{
 			name: "validate auth (plugin managed, invalid config, basic auth)",
@@ -6750,7 +6750,7 @@ func TestAddPackageRepository(t *testing.T) {
 				return request
 			},
 			expectedErrorCode:   connect.CodeInvalidArgument,
-			expectedErrorString: "missing basic auth",
+			expectedErrorString: "Missing basic auth",
 		},
 		{
 			name: "validate auth (plugin managed, invalid config, docker)",
@@ -6764,7 +6764,7 @@ func TestAddPackageRepository(t *testing.T) {
 				return request
 			},
 			expectedErrorCode:   connect.CodeInvalidArgument,
-			expectedErrorString: "missing Docker Config auth",
+			expectedErrorString: "Missing Docker Config auth",
 		},
 		{
 			name: "validate auth (plugin managed, invalid config, ssh auth)",
@@ -6782,7 +6782,7 @@ func TestAddPackageRepository(t *testing.T) {
 				return request
 			},
 			expectedErrorCode:   connect.CodeInvalidArgument,
-			expectedErrorString: "unexpected REDACTED",
+			expectedErrorString: "Unexpected REDACTED",
 		},
 		{
 			name: "create with description",
@@ -7392,7 +7392,7 @@ func TestUpdatePackageRepository(t *testing.T) {
 				return request
 			},
 			expectedErrorCode:    connect.CodeInvalidArgument,
-			expectedStatusString: "missing basic auth",
+			expectedStatusString: "Missing basic auth",
 		},
 		{
 			name: "validate auth (plugin managed, invalid config, docker)",

cmd/kubeapps-apis/plugins/pkg/connecterror/connecterror_test.go

@@ -28,15 +28,15 @@ func TestErrorByStatus(t *testing.T) {
 			"my-resource",
 			"",
 			status.Errorf(codes.InvalidArgument, "boom!"),
-			connect.NewError(connect.CodeInternal, fmt.Errorf("unable to get the my-resource 'all' due to 'rpc error: code = InvalidArgument desc = boom!'")),
+			connect.NewError(connect.CodeInternal, fmt.Errorf("Unable to get the my-resource 'all' due to 'rpc error: code = InvalidArgument desc = boom!'")),
 		},
 		{
 			"error msg for a single resources ",
 			"get",
 			"my-resource",
 			"my-id",
 			status.Errorf(codes.InvalidArgument, "boom!"),
-			connect.NewError(connect.CodeInternal, fmt.Errorf("unable to get the my-resource 'my-id' due to 'rpc error: code = InvalidArgument desc = boom!'")),
+			connect.NewError(connect.CodeInternal, fmt.Errorf("Unable to get the my-resource 'my-id' due to 'rpc error: code = InvalidArgument desc = boom!'")),
 		},
 	}
 	for _, tt := range tests {