Skip to content

Commit 2d51408

Browse files
ekristencorybekk
authored andcommitted
test(kms-key): fix tests for new alias property and list aliases call
1 parent b7d6c78 commit 2d51408

File tree

1 file changed

+18
-0
lines changed

1 file changed

+18
-0
lines changed

resources/kms-key_mock_test.go

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,14 @@ func Test_Mock_KMSKey_List(t *testing.T) {
5656
},
5757
)
5858

59+
mockKMS.EXPECT().ListAliases(&kms.ListAliasesInput{
60+
KeyId: aws.String("test-key-id"),
61+
}).Return(&kms.ListAliasesOutput{
62+
Aliases: []*kms.AliasListEntry{
63+
{AliasName: aws.String("alias/test-key-id")},
64+
},
65+
}, nil)
66+
5967
lister := KMSKeyLister{
6068
mockSvc: mockKMS,
6169
}
@@ -123,6 +131,14 @@ func Test_Mock_KMSKey_List_WithAccessDenied(t *testing.T) {
123131
},
124132
)
125133

134+
mockKMS.EXPECT().ListAliases(&kms.ListAliasesInput{
135+
KeyId: aws.String("test-key-id-1"),
136+
}).Return(&kms.ListAliasesOutput{
137+
Aliases: []*kms.AliasListEntry{
138+
{AliasName: aws.String("alias/test-key-id-1")},
139+
},
140+
}, nil)
141+
126142
lister := KMSKeyLister{
127143
mockSvc: mockKMS,
128144
}
@@ -180,6 +196,7 @@ func Test_Mock_KMSKey_Properties(t *testing.T) {
180196
ID: ptr.String("test-key-id"),
181197
State: ptr.String(kms.KeyStateEnabled),
182198
Manager: ptr.String(kms.KeyManagerTypeCustomer),
199+
Alias: ptr.String("alias/test-key-id"),
183200
Tags: []*kms.Tag{
184201
{TagKey: aws.String("Environment"), TagValue: aws.String("Test")},
185202
},
@@ -189,6 +206,7 @@ func Test_Mock_KMSKey_Properties(t *testing.T) {
189206
assert.Equal(t, kms.KeyStateEnabled, kmsKey.Properties().Get("State"))
190207
assert.Equal(t, kms.KeyManagerTypeCustomer, kmsKey.Properties().Get("Manager"))
191208
assert.Equal(t, "Test", kmsKey.Properties().Get("tag:Environment"))
209+
assert.Equal(t, "alias/test-key-id", kmsKey.Properties().Get("Alias"))
192210
}
193211

194212
func Test_Mock_KMSKey_Remove(t *testing.T) {

0 commit comments

Comments
 (0)