use string for wantErr

Author: Carlos Flores

pkg/targetgroupbinding/networking_manager_test.go

@@ -1435,7 +1435,7 @@ func Test_defaultNetworkingManager_resolveEndpointSGForENI(t *testing.T) {
 		fields  fields
 		args    args
 		want    string
-		wantErr error
+		wantErr string
 	}{
 		{
 			name: "Only one security group in eniInfo returns early",
@@ -1470,7 +1470,7 @@ func Test_defaultNetworkingManager_resolveEndpointSGForENI(t *testing.T) {
 				},
 			},
 			want:    "",
-			wantErr: errors.New("expected exactly one securityGroup tagged with kubernetes.io/cluster/cluster-a for eni eni-a, got: [] (clusterName: cluster-a)"),
+			wantErr: "expected exactly one securityGroup tagged with kubernetes.io/cluster/cluster-a for eni eni-a, got: [] (clusterName: cluster-a)",
 		},
 		{
 			name: "A single security group with cluster name tag and no service target tags set",
@@ -1585,7 +1585,7 @@ func Test_defaultNetworkingManager_resolveEndpointSGForENI(t *testing.T) {
 				},
 			},
 			want:    "",
-			wantErr: errors.New("expected exactly one securityGroup tagged with kubernetes.io/cluster/cluster-a and map[keyA:valueNotA] for eni eni-a, got: [] (clusterName: cluster-a)"),
+			wantErr: "expected exactly one securityGroup tagged with kubernetes.io/cluster/cluster-a and map[keyA:valueNotA] for eni eni-a, got: [] (clusterName: cluster-a)",
 		},
 		{
 			name: "A single security group with cluster name tag and multiple service target tags set",
@@ -1666,7 +1666,7 @@ func Test_defaultNetworkingManager_resolveEndpointSGForENI(t *testing.T) {
 				},
 			},
 			want:    "",
-			wantErr: errors.New("expected exactly one securityGroup tagged with kubernetes.io/cluster/cluster-a and map[keyA:valueA keyB:valueNotB2] for eni eni-a, got: [] (clusterName: cluster-a)"),
+			wantErr: "expected exactly one securityGroup tagged with kubernetes.io/cluster/cluster-a and map[keyA:valueA keyB:valueNotB2] for eni eni-a, got: [] (clusterName: cluster-a)",
 		},
 		{
 			name: "A single security group with cluster name tag and a service target tags with an empty value",
@@ -1745,7 +1745,7 @@ func Test_defaultNetworkingManager_resolveEndpointSGForENI(t *testing.T) {
 				},
 			},
 			want:    "",
-			wantErr: errors.New("expected exactly one securityGroup tagged with kubernetes.io/cluster/cluster-a and map[keyE:] for eni eni-a, got: [] (clusterName: cluster-a)"),
+			wantErr: "expected exactly one securityGroup tagged with kubernetes.io/cluster/cluster-a and map[keyE:] for eni eni-a, got: [] (clusterName: cluster-a)",
 		},
 	}
 	for _, tt := range tests {
@@ -1764,8 +1764,8 @@ func Test_defaultNetworkingManager_resolveEndpointSGForENI(t *testing.T) {
 				serviceTargetENISGTags: tt.fields.serviceTargetENISGTags,
 			}
 			got, err := m.resolveEndpointSGForENI(tt.args.ctx, tt.args.eniInfo)
-			if tt.wantErr != nil {
-				assert.EqualError(t, err, tt.wantErr.Error())
+			if tt.wantErr != "" {
+				assert.EqualError(t, err, tt.wantErr)
 			} else {
 				assert.NoError(t, err)
 				assert.Equal(t, tt.want, got)