chore(serviceaccount): remove redundant test cases

Author: s-inter

stackit/internal/services/serviceaccount/account/resource_test.go

@@ -79,30 +79,6 @@ func TestMapFields(t *testing.T) {
 			Model{},
 			false,
 		},
-		{
-			"nil_response_2",
-			&serviceaccount.ServiceAccount{},
-			Model{},
-			false,
-		},
-		{
-			"no_email",
-			&serviceaccount.ServiceAccount{
-				ProjectId: "pid",
-				Id:        "550e8400-e29b-41d4-a716-446655440000",
-			},
-			Model{},
-			false,
-		},
-		{
-			"no_id",
-			&serviceaccount.ServiceAccount{
-				ProjectId: "pid",
-				Email:     "mail",
-			},
-			Model{},
-			false,
-		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.description, func(t *testing.T) {

stackit/internal/services/serviceaccount/accounts/datasource_test.go

@@ -122,26 +122,6 @@ func TestMapDataSourceFields(t *testing.T) {
 			},
 			isValid: true,
 		},
-		{
-			description: "skip_nil_email",
-			apiItems: []serviceaccount.ServiceAccount{
-				{Email: emailA, Id: idA},
-				{Email: ""}, // Should be skipped
-			},
-			initialModel: ServiceAccountsModel{
-				ProjectId:     types.StringValue(projectId),
-				SortAscending: types.BoolValue(true),
-			},
-			expectedModel: ServiceAccountsModel{
-				Id:            types.StringValue(projectId),
-				ProjectId:     types.StringValue(projectId),
-				SortAscending: types.BoolValue(true),
-				Items: []ServiceAccountItem{
-					{Email: types.StringValue(emailA), Name: types.StringValue(nameA), ServiceAccountId: types.StringValue(idA)},
-				},
-			},
-			isValid: true,
-		},
 		{
 			description:  "nil_model",
 			apiItems:     []serviceaccount.ServiceAccount{},

stackit/internal/services/serviceaccount/key/resource_test.go

@@ -79,20 +79,6 @@ func TestMapResponse(t *testing.T) {
 			expected:    Model{},
 			isValid:     false,
 		},
-		{
-			description: "nil_response_2",
-			input:       &serviceaccount.CreateServiceAccountKeyResponse{},
-			expected:    Model{},
-			isValid:     false,
-		},
-		{
-			description: "no_id",
-			input: &serviceaccount.CreateServiceAccountKeyResponse{
-				Active: true,
-			},
-			expected: Model{},
-			isValid:  false,
-		},
 	}
 
 	for _, tt := range tests {