Skip to content

Commit 2c8fda5

Browse files
committed
aws/permissions: add create/delete base permission unit tests.
1 parent ad0c51d commit 2c8fda5

File tree

1 file changed

+29
-0
lines changed

1 file changed

+29
-0
lines changed

pkg/asset/installconfig/aws/permissions_test.go

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -530,3 +530,32 @@ func TestPublicIPv4PoolPermissions(t *testing.T) {
530530
assert.NotContains(t, requiredPerms, PermissionPublicIpv4Pool)
531531
})
532532
}
533+
534+
func TestBasePermissions(t *testing.T) {
535+
t.Run("Should include", func(t *testing.T) {
536+
t.Run("base create permissions", func(t *testing.T) {
537+
t.Run("on standard regions", func(t *testing.T) {
538+
ic := validInstallConfig()
539+
requiredPerms := RequiredPermissionGroups(ic)
540+
assert.Contains(t, requiredPerms, PermissionCreateBase)
541+
})
542+
t.Run("on secret regions", func(t *testing.T) {
543+
ic := validInstallConfig()
544+
ic.AWS.Region = "us-iso-east-1"
545+
requiredPerms := RequiredPermissionGroups(ic)
546+
assert.Contains(t, requiredPerms, PermissionCreateBase)
547+
})
548+
})
549+
t.Run("base delete permissions on standard regions", func(t *testing.T) {
550+
ic := validInstallConfig()
551+
requiredPerms := RequiredPermissionGroups(ic)
552+
assert.Contains(t, requiredPerms, PermissionDeleteBase)
553+
})
554+
})
555+
t.Run("Should not include base delete permissions on secret regions", func(t *testing.T) {
556+
ic := validInstallConfig()
557+
ic.AWS.Region = "us-iso-east-1"
558+
requiredPerms := RequiredPermissionGroups(ic)
559+
assert.NotContains(t, requiredPerms, PermissionDeleteBase)
560+
})
561+
}

0 commit comments

Comments
 (0)