kemko/nomad
1
0
Fork 0
mirror of https://github.com/kemko/nomad.git synced 2026-01-01 16:05:42 +03:00
Code Issues Packages Projects Releases Wiki Activity

cli: support json and t on acl binding-rule info command. (#16357)

Browse Source
This commit is contained in:
James Rasell
2023-03-07 18:27:02 +01:00
committed by GitHub
parent 03d6a8c70a
commit 003a567d10
3 changed files with 33 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
.changelog/16357.txt Normal file
View File

@@ -0,0 +1,3 @@
```release-note:bug
cli: Fixed a bug where the `-json` and `-t` flags were not respected on the `acl binding-rule info` command
```

11
command/acl_binding_rule_info.go
View File

@@ -95,6 +95,17 @@ func (a *ACLBindingRuleInfoCommand) Run(args []string) int {
return 1
}
if a.json || len(a.tmpl) > 0 {
out, err := Format(a.json, a.tmpl, aclBindingRule)
if err != nil {
a.Ui.Error(err.Error())
return 1
}
a.Ui.Output(out)
return 0
}
// Format the output.
a.Ui.Output(formatACLBindingRule(aclBindingRule))
return 0

19
command/acl_binding_rule_info_test.go
View File

@@ -1,6 +1,7 @@
package command
import (
"fmt"
"testing"
"github.com/hashicorp/nomad/ci"
@@ -68,4 +69,22 @@ func TestACLBindingRuleInfoCommand_Run(t *testing.T) {
ui.OutputWriter.Reset()
ui.ErrorWriter.Reset()
// Test that the JSON flag works in return a string that has JSON markers.
must.Eq(t, 0, cmd.Run([]string{"-address=" + url, "-token=" + rootACLToken.SecretID, "-json", aclBindingRule.ID}))
s = ui.OutputWriter.String()
must.StrContains(t, s, "{")
must.StrContains(t, s, "}")
ui.OutputWriter.Reset()
ui.ErrorWriter.Reset()
// Test that we can pass in a custom go template to format the output.
must.Eq(t, 0, cmd.Run([]string{
"-address=" + url, "-token=" + rootACLToken.SecretID, "-t", "Custom: {{ .ID }}", aclBindingRule.ID}))
s = ui.OutputWriter.String()
must.StrContains(t, s, fmt.Sprintf("Custom: %s", aclBindingRule.ID))
ui.OutputWriter.Reset()
ui.ErrorWriter.Reset()
}