Skip to content

Commit 60c53d1

Browse files
committed
r/aws_iam_role(test): remove and tidy expected no refresh errors
1 parent 82b7e07 commit 60c53d1

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

internal/service/iam/role_test.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1006,8 +1006,8 @@ func TestAccIAMRole_Identity_ExistingResource_NoRefresh(t *testing.T) {
10061006
Check: resource.ComposeAggregateTestCheckFunc(
10071007
testAccCheckRoleExists(ctx, resourceName, &conf),
10081008
),
1009-
// We DON'T want this error. A fix should cause this test to begin failing.
1010-
ExpectError: regexache.MustCompile(`Missing Resource Identity After Update`),
1009+
// No error indicates the identity interceptor properly identified
1010+
// a fully null identity on update and set values appropriately.
10111011
},
10121012
},
10131013
})
@@ -1047,7 +1047,9 @@ func TestAccIAMRole_Identity_ExistingResource_NoRefreshFailure(t *testing.T) {
10471047
Check: resource.ComposeAggregateTestCheckFunc(
10481048
testAccCheckRoleExists(ctx, resourceName, &conf),
10491049
),
1050-
// We DON'T want this error. A fix should cause this test to begin failing.
1050+
// On an update failure, the identity interceptor is not executed and both
1051+
// the MalformedPolicyDocument error and a missing resource identity
1052+
// error will be present.
10511053
ExpectError: regexache.MustCompile(`Missing Resource Identity After Update`),
10521054
},
10531055
},

0 commit comments

Comments
 (0)