2023-04-10 15:36:59 +00:00
|
|
|
// Copyright (c) HashiCorp, Inc.
|
|
|
|
// SPDX-License-Identifier: MPL-2.0
|
|
|
|
|
2022-08-12 07:52:32 +00:00
|
|
|
package command
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"testing"
|
|
|
|
|
|
|
|
"github.com/hashicorp/nomad/ci"
|
|
|
|
"github.com/hashicorp/nomad/command/agent"
|
|
|
|
"github.com/hashicorp/nomad/helper/uuid"
|
|
|
|
"github.com/hashicorp/nomad/nomad/structs"
|
|
|
|
"github.com/hashicorp/nomad/testutil"
|
|
|
|
"github.com/mitchellh/cli"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
|
|
)
|
|
|
|
|
|
|
|
func TestACLRoleUpdateCommand_Run(t *testing.T) {
|
|
|
|
ci.Parallel(t)
|
|
|
|
|
|
|
|
// Build a test server with ACLs enabled.
|
|
|
|
srv, _, url := testServer(t, false, func(c *agent.Config) {
|
|
|
|
c.ACL.Enabled = true
|
|
|
|
})
|
|
|
|
defer srv.Shutdown()
|
|
|
|
|
|
|
|
// Wait for the server to start fully and ensure we have a bootstrap token.
|
|
|
|
testutil.WaitForLeader(t, srv.Agent.RPC)
|
|
|
|
rootACLToken := srv.RootToken
|
|
|
|
require.NotNil(t, rootACLToken)
|
|
|
|
|
|
|
|
ui := cli.NewMockUi()
|
|
|
|
cmd := &ACLRoleUpdateCommand{
|
|
|
|
Meta: Meta{
|
|
|
|
Ui: ui,
|
|
|
|
flagAddress: url,
|
|
|
|
},
|
|
|
|
}
|
|
|
|
|
|
|
|
// Try calling the command without setting an ACL Role ID arg.
|
|
|
|
require.Equal(t, 1, cmd.Run([]string{"-address=" + url}))
|
|
|
|
require.Contains(t, ui.ErrorWriter.String(), "This command takes one argument")
|
|
|
|
|
|
|
|
ui.OutputWriter.Reset()
|
|
|
|
ui.ErrorWriter.Reset()
|
|
|
|
|
|
|
|
// Try calling the command with an ACL role ID that does not exist.
|
|
|
|
code := cmd.Run([]string{"-address=" + url, "-token=" + rootACLToken.SecretID, "catch-me-if-you-can"})
|
|
|
|
require.Equal(t, 1, code)
|
|
|
|
require.Contains(t, ui.ErrorWriter.String(), "ACL role not found")
|
|
|
|
|
|
|
|
ui.OutputWriter.Reset()
|
|
|
|
ui.ErrorWriter.Reset()
|
|
|
|
|
|
|
|
// Create an ACL policy that can be referenced within the ACL role.
|
|
|
|
aclPolicy := structs.ACLPolicy{
|
|
|
|
Name: "acl-role-cli-test-policy",
|
|
|
|
Rules: `namespace "default" {
|
|
|
|
policy = "read"
|
|
|
|
}
|
|
|
|
`,
|
|
|
|
}
|
|
|
|
err := srv.Agent.Server().State().UpsertACLPolicies(
|
|
|
|
structs.MsgTypeTestSetup, 10, []*structs.ACLPolicy{&aclPolicy})
|
|
|
|
require.NoError(t, err)
|
|
|
|
|
|
|
|
// Create an ACL role that can be used for updating.
|
|
|
|
aclRole := structs.ACLRole{
|
|
|
|
ID: uuid.Generate(),
|
|
|
|
Name: "acl-role-cli-test",
|
|
|
|
Description: "my-lovely-role",
|
|
|
|
Policies: []*structs.ACLRolePolicyLink{{Name: aclPolicy.Name}},
|
|
|
|
}
|
|
|
|
|
|
|
|
err = srv.Agent.Server().State().UpsertACLRoles(
|
2022-08-22 06:54:07 +00:00
|
|
|
structs.MsgTypeTestSetup, 20, []*structs.ACLRole{&aclRole}, false)
|
2022-08-12 07:52:32 +00:00
|
|
|
require.NoError(t, err)
|
|
|
|
|
|
|
|
// Try a merge update without setting any parameters to update.
|
|
|
|
code = cmd.Run([]string{"-address=" + url, "-token=" + rootACLToken.SecretID, aclRole.ID})
|
|
|
|
require.Equal(t, 1, code)
|
|
|
|
require.Contains(t, ui.ErrorWriter.String(), "Please provide at least one flag to update the ACL role")
|
|
|
|
|
|
|
|
ui.OutputWriter.Reset()
|
|
|
|
ui.ErrorWriter.Reset()
|
|
|
|
|
|
|
|
// Update the description using the merge method.
|
|
|
|
code = cmd.Run([]string{
|
|
|
|
"-address=" + url, "-token=" + rootACLToken.SecretID, "-description=badger-badger-badger", aclRole.ID})
|
|
|
|
require.Equal(t, 0, code)
|
|
|
|
s := ui.OutputWriter.String()
|
|
|
|
require.Contains(t, s, fmt.Sprintf("ID = %s", aclRole.ID))
|
|
|
|
require.Contains(t, s, "Name = acl-role-cli-test")
|
|
|
|
require.Contains(t, s, "Description = badger-badger-badger")
|
|
|
|
require.Contains(t, s, "Policies = acl-role-cli-test-policy")
|
|
|
|
|
|
|
|
ui.OutputWriter.Reset()
|
|
|
|
ui.ErrorWriter.Reset()
|
|
|
|
|
|
|
|
// Try updating the role using no-merge without setting the required flags.
|
|
|
|
code = cmd.Run([]string{"-address=" + url, "-token=" + rootACLToken.SecretID, "-no-merge", aclRole.ID})
|
|
|
|
require.Equal(t, 1, code)
|
|
|
|
require.Contains(t, ui.ErrorWriter.String(), "ACL role name must be specified using the -name flag")
|
|
|
|
|
|
|
|
ui.OutputWriter.Reset()
|
|
|
|
ui.ErrorWriter.Reset()
|
|
|
|
|
|
|
|
code = cmd.Run([]string{
|
|
|
|
"-address=" + url, "-token=" + rootACLToken.SecretID, "-no-merge", "-name=update-role-name", aclRole.ID})
|
|
|
|
require.Equal(t, 1, code)
|
2022-08-24 14:15:02 +00:00
|
|
|
require.Contains(t, ui.ErrorWriter.String(), "At least one policy name must be specified using the -policy flag")
|
2022-08-12 07:52:32 +00:00
|
|
|
|
|
|
|
// Update the role using no-merge with all required flags set.
|
|
|
|
code = cmd.Run([]string{
|
|
|
|
"-address=" + url, "-token=" + rootACLToken.SecretID, "-no-merge", "-name=update-role-name",
|
2022-08-24 14:15:02 +00:00
|
|
|
"-description=updated-description", "-policy=acl-role-cli-test-policy", aclRole.ID})
|
2022-08-12 07:52:32 +00:00
|
|
|
require.Equal(t, 0, code)
|
|
|
|
s = ui.OutputWriter.String()
|
|
|
|
require.Contains(t, s, fmt.Sprintf("ID = %s", aclRole.ID))
|
|
|
|
require.Contains(t, s, "Name = update-role-name")
|
|
|
|
require.Contains(t, s, "Description = updated-description")
|
|
|
|
require.Contains(t, s, "Policies = acl-role-cli-test-policy")
|
|
|
|
|
|
|
|
ui.OutputWriter.Reset()
|
|
|
|
ui.ErrorWriter.Reset()
|
|
|
|
}
|