2018-10-19 16:04:07 +00:00
|
|
|
package tokenupdate
|
|
|
|
|
|
|
|
import (
|
2020-02-15 15:06:05 +00:00
|
|
|
"encoding/json"
|
2018-10-19 16:04:07 +00:00
|
|
|
"strings"
|
|
|
|
"testing"
|
|
|
|
|
|
|
|
"github.com/mitchellh/cli"
|
|
|
|
"github.com/stretchr/testify/assert"
|
2020-06-16 16:54:27 +00:00
|
|
|
"github.com/stretchr/testify/require"
|
2021-07-20 22:38:23 +00:00
|
|
|
|
|
|
|
"github.com/hashicorp/consul/agent"
|
|
|
|
"github.com/hashicorp/consul/api"
|
|
|
|
"github.com/hashicorp/consul/testrpc"
|
2018-10-19 16:04:07 +00:00
|
|
|
)
|
|
|
|
|
|
|
|
func TestTokenUpdateCommand_noTabs(t *testing.T) {
|
|
|
|
t.Parallel()
|
|
|
|
|
|
|
|
if strings.ContainsRune(New(cli.NewMockUi()).Help(), '\t') {
|
|
|
|
t.Fatal("help has tabs")
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func TestTokenUpdateCommand(t *testing.T) {
|
2020-12-07 18:42:55 +00:00
|
|
|
if testing.Short() {
|
|
|
|
t.Skip("too slow for testing.Short")
|
|
|
|
}
|
|
|
|
|
2018-10-19 16:04:07 +00:00
|
|
|
t.Parallel()
|
|
|
|
|
2020-03-31 19:59:56 +00:00
|
|
|
a := agent.NewTestAgent(t, `
|
2018-10-19 16:04:07 +00:00
|
|
|
primary_datacenter = "dc1"
|
|
|
|
acl {
|
|
|
|
enabled = true
|
|
|
|
tokens {
|
2021-12-07 12:48:50 +00:00
|
|
|
initial_management = "root"
|
2018-10-19 16:04:07 +00:00
|
|
|
}
|
|
|
|
}`)
|
|
|
|
|
|
|
|
defer a.Shutdown()
|
|
|
|
testrpc.WaitForLeader(t, a.RPC, "dc1")
|
|
|
|
|
|
|
|
// Create a policy
|
|
|
|
client := a.Client()
|
|
|
|
|
|
|
|
policy, _, err := client.ACL().PolicyCreate(
|
|
|
|
&api.ACLPolicy{Name: "test-policy"},
|
|
|
|
&api.WriteOptions{Token: "root"},
|
|
|
|
)
|
2020-06-16 16:54:27 +00:00
|
|
|
require.NoError(t, err)
|
2018-10-19 16:04:07 +00:00
|
|
|
|
|
|
|
// create a token
|
|
|
|
token, _, err := client.ACL().TokenCreate(
|
|
|
|
&api.ACLToken{Description: "test"},
|
|
|
|
&api.WriteOptions{Token: "root"},
|
|
|
|
)
|
2020-06-16 16:54:27 +00:00
|
|
|
require.NoError(t, err)
|
2018-11-05 14:32:09 +00:00
|
|
|
|
2020-06-16 16:54:27 +00:00
|
|
|
run := func(t *testing.T, args []string) *api.ACLToken {
|
|
|
|
ui := cli.NewMockUi()
|
2018-11-05 14:32:09 +00:00
|
|
|
cmd := New(ui)
|
2020-06-16 16:54:27 +00:00
|
|
|
|
|
|
|
code := cmd.Run(append(args, "-format=json"))
|
|
|
|
require.Equal(t, 0, code)
|
|
|
|
require.Empty(t, ui.ErrorWriter.String())
|
|
|
|
|
|
|
|
var token api.ACLToken
|
|
|
|
require.NoError(t, json.Unmarshal(ui.OutputWriter.Bytes(), &token))
|
|
|
|
return &token
|
|
|
|
}
|
|
|
|
|
|
|
|
// update with node identity
|
|
|
|
t.Run("node-identity", func(t *testing.T) {
|
|
|
|
token := run(t, []string{
|
2018-10-19 16:04:07 +00:00
|
|
|
"-http-addr=" + a.HTTPAddr(),
|
|
|
|
"-id=" + token.AccessorID,
|
|
|
|
"-token=root",
|
2020-06-16 16:54:27 +00:00
|
|
|
"-node-identity=foo:bar",
|
2018-10-19 16:04:07 +00:00
|
|
|
"-description=test token",
|
2020-06-16 16:54:27 +00:00
|
|
|
})
|
|
|
|
|
|
|
|
require.Len(t, token.NodeIdentities, 1)
|
|
|
|
require.Equal(t, "foo", token.NodeIdentities[0].NodeName)
|
|
|
|
require.Equal(t, "bar", token.NodeIdentities[0].Datacenter)
|
|
|
|
})
|
|
|
|
|
|
|
|
t.Run("node-identity-merge", func(t *testing.T) {
|
|
|
|
token := run(t, []string{
|
|
|
|
"-http-addr=" + a.HTTPAddr(),
|
|
|
|
"-id=" + token.AccessorID,
|
|
|
|
"-token=root",
|
|
|
|
"-node-identity=bar:baz",
|
|
|
|
"-description=test token",
|
|
|
|
"-merge-node-identities",
|
|
|
|
})
|
|
|
|
|
|
|
|
require.Len(t, token.NodeIdentities, 2)
|
|
|
|
expected := []*api.ACLNodeIdentity{
|
2020-06-16 17:19:31 +00:00
|
|
|
{
|
2020-06-16 16:54:27 +00:00
|
|
|
NodeName: "foo",
|
|
|
|
Datacenter: "bar",
|
|
|
|
},
|
2020-06-16 17:19:31 +00:00
|
|
|
{
|
2020-06-16 16:54:27 +00:00
|
|
|
NodeName: "bar",
|
|
|
|
Datacenter: "baz",
|
|
|
|
},
|
2018-10-19 16:04:07 +00:00
|
|
|
}
|
2020-06-16 16:54:27 +00:00
|
|
|
require.ElementsMatch(t, expected, token.NodeIdentities)
|
|
|
|
})
|
2018-10-19 16:04:07 +00:00
|
|
|
|
2020-06-16 16:54:27 +00:00
|
|
|
// update with policy by name
|
|
|
|
t.Run("policy-name", func(t *testing.T) {
|
|
|
|
token := run(t, []string{
|
|
|
|
"-http-addr=" + a.HTTPAddr(),
|
|
|
|
"-id=" + token.AccessorID,
|
|
|
|
"-token=root",
|
|
|
|
"-policy-name=" + policy.Name,
|
|
|
|
"-description=test token",
|
|
|
|
})
|
2018-10-19 16:04:07 +00:00
|
|
|
|
2020-06-16 16:54:27 +00:00
|
|
|
require.Len(t, token.Policies, 1)
|
|
|
|
})
|
2018-10-19 16:04:07 +00:00
|
|
|
|
|
|
|
// update with policy by id
|
2020-06-16 16:54:27 +00:00
|
|
|
t.Run("policy-id", func(t *testing.T) {
|
|
|
|
token := run(t, []string{
|
2018-10-19 16:04:07 +00:00
|
|
|
"-http-addr=" + a.HTTPAddr(),
|
|
|
|
"-id=" + token.AccessorID,
|
|
|
|
"-token=root",
|
|
|
|
"-policy-id=" + policy.ID,
|
|
|
|
"-description=test token",
|
2020-06-16 16:54:27 +00:00
|
|
|
})
|
2018-10-19 16:04:07 +00:00
|
|
|
|
2020-06-16 16:54:27 +00:00
|
|
|
require.Len(t, token.Policies, 1)
|
|
|
|
})
|
2018-11-05 14:32:09 +00:00
|
|
|
|
|
|
|
// update with no description shouldn't delete the current description
|
2020-06-16 16:54:27 +00:00
|
|
|
t.Run("merge-description", func(t *testing.T) {
|
|
|
|
token := run(t, []string{
|
2018-11-05 14:32:09 +00:00
|
|
|
"-http-addr=" + a.HTTPAddr(),
|
|
|
|
"-id=" + token.AccessorID,
|
|
|
|
"-token=root",
|
|
|
|
"-policy-name=" + policy.Name,
|
2020-06-16 16:54:27 +00:00
|
|
|
})
|
2018-11-05 14:32:09 +00:00
|
|
|
|
2020-06-16 16:54:27 +00:00
|
|
|
require.Equal(t, "test token", token.Description)
|
|
|
|
})
|
2018-10-19 16:04:07 +00:00
|
|
|
}
|
2020-02-15 15:06:05 +00:00
|
|
|
|
|
|
|
func TestTokenUpdateCommand_JSON(t *testing.T) {
|
2020-12-07 18:42:55 +00:00
|
|
|
if testing.Short() {
|
|
|
|
t.Skip("too slow for testing.Short")
|
|
|
|
}
|
|
|
|
|
2020-02-15 15:06:05 +00:00
|
|
|
t.Parallel()
|
|
|
|
|
2020-03-31 19:59:56 +00:00
|
|
|
a := agent.NewTestAgent(t, `
|
2020-02-15 15:06:05 +00:00
|
|
|
primary_datacenter = "dc1"
|
|
|
|
acl {
|
|
|
|
enabled = true
|
|
|
|
tokens {
|
2021-12-07 12:48:50 +00:00
|
|
|
initial_management = "root"
|
2020-02-15 15:06:05 +00:00
|
|
|
}
|
|
|
|
}`)
|
|
|
|
|
|
|
|
defer a.Shutdown()
|
|
|
|
testrpc.WaitForLeader(t, a.RPC, "dc1")
|
|
|
|
|
|
|
|
ui := cli.NewMockUi()
|
|
|
|
|
|
|
|
// Create a policy
|
|
|
|
client := a.Client()
|
|
|
|
|
|
|
|
policy, _, err := client.ACL().PolicyCreate(
|
|
|
|
&api.ACLPolicy{Name: "test-policy"},
|
|
|
|
&api.WriteOptions{Token: "root"},
|
|
|
|
)
|
2022-01-20 16:45:56 +00:00
|
|
|
require.NoError(t, err)
|
2020-02-15 15:06:05 +00:00
|
|
|
|
|
|
|
// create a token
|
|
|
|
token, _, err := client.ACL().TokenCreate(
|
|
|
|
&api.ACLToken{Description: "test"},
|
|
|
|
&api.WriteOptions{Token: "root"},
|
|
|
|
)
|
2022-01-20 16:45:56 +00:00
|
|
|
require.NoError(t, err)
|
2020-02-15 15:06:05 +00:00
|
|
|
|
|
|
|
t.Run("update with policy by name", func(t *testing.T) {
|
|
|
|
cmd := New(ui)
|
|
|
|
args := []string{
|
|
|
|
"-http-addr=" + a.HTTPAddr(),
|
|
|
|
"-id=" + token.AccessorID,
|
|
|
|
"-token=root",
|
|
|
|
"-policy-name=" + policy.Name,
|
|
|
|
"-description=test token",
|
|
|
|
"-format=json",
|
|
|
|
}
|
|
|
|
|
|
|
|
code := cmd.Run(args)
|
bulk rewrite using this script
set -euo pipefail
unset CDPATH
cd "$(dirname "$0")"
for f in $(git grep '\brequire := require\.New(' | cut -d':' -f1 | sort -u); do
echo "=== require: $f ==="
sed -i '/require := require.New(t)/d' $f
# require.XXX(blah) but not require.XXX(tblah) or require.XXX(rblah)
sed -i 's/\brequire\.\([a-zA-Z0-9_]*\)(\([^tr]\)/require.\1(t,\2/g' $f
# require.XXX(tblah) but not require.XXX(t, blah)
sed -i 's/\brequire\.\([a-zA-Z0-9_]*\)(\(t[^,]\)/require.\1(t,\2/g' $f
# require.XXX(rblah) but not require.XXX(r, blah)
sed -i 's/\brequire\.\([a-zA-Z0-9_]*\)(\(r[^,]\)/require.\1(t,\2/g' $f
gofmt -s -w $f
done
for f in $(git grep '\bassert := assert\.New(' | cut -d':' -f1 | sort -u); do
echo "=== assert: $f ==="
sed -i '/assert := assert.New(t)/d' $f
# assert.XXX(blah) but not assert.XXX(tblah) or assert.XXX(rblah)
sed -i 's/\bassert\.\([a-zA-Z0-9_]*\)(\([^tr]\)/assert.\1(t,\2/g' $f
# assert.XXX(tblah) but not assert.XXX(t, blah)
sed -i 's/\bassert\.\([a-zA-Z0-9_]*\)(\(t[^,]\)/assert.\1(t,\2/g' $f
# assert.XXX(rblah) but not assert.XXX(r, blah)
sed -i 's/\bassert\.\([a-zA-Z0-9_]*\)(\(r[^,]\)/assert.\1(t,\2/g' $f
gofmt -s -w $f
done
2022-01-20 16:46:23 +00:00
|
|
|
assert.Equal(t, code, 0)
|
|
|
|
assert.Empty(t, ui.ErrorWriter.String())
|
2020-02-15 15:06:05 +00:00
|
|
|
|
|
|
|
var jsonOutput json.RawMessage
|
|
|
|
err := json.Unmarshal([]byte(ui.OutputWriter.String()), &jsonOutput)
|
|
|
|
require.NoError(t, err, "token unmarshalling error")
|
|
|
|
})
|
|
|
|
}
|