Skip to content

Commit

Permalink
gofmt, govet, run those in Travis, add 1.7
Browse files Browse the repository at this point in the history
  • Loading branch information
broady committed Aug 23, 2016
1 parent 1f47171 commit d1371ec
Show file tree
Hide file tree
Showing 5 changed files with 21 additions and 14 deletions.
18 changes: 13 additions & 5 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,9 +1,13 @@
language: go
go:
- 1.4.3
- 1.5.4
- 1.6.3
- tip

matrix:
include:
- go: 1.4.3
env: NOVET=true # No bundled vet.
- go: 1.5.4
- go: 1.6.3
- go: 1.7
- go: tip

matrix:
allow_failures:
Expand All @@ -16,3 +20,7 @@ before_install:
script:
- PATH=$PATH:$PWD/bin go test -v ./...
- go build
- diff -u <(echo -n) <(gofmt -d -s .)
- if [ -z $NOVET ]; then
diff -u <(echo -n) <(go tool vet . 2>&1 | grep -vE 'ExampleCommand|bash_completions.*Fprint');
fi
2 changes: 1 addition & 1 deletion bash_completions_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ func TestBashCompletionHiddenFlag(t *testing.T) {
cmdTrue.GenBashCompletion(out)
bashCompletion := out.String()
if strings.Contains(bashCompletion, flagName) {
t.Error("expected completion to not include %q flag: Got %v", flagName, bashCompletion)
t.Errorf("expected completion to not include %q flag: Got %v", flagName, bashCompletion)
}
}

Expand Down
5 changes: 2 additions & 3 deletions cobra_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -916,15 +916,14 @@ func TestRootUnknownCommand(t *testing.T) {

func TestRootUnknownCommandSilenced(t *testing.T) {
r := noRRSetupTestSilenced("bogus")
s := "Run 'cobra-test --help' for usage.\n"

if r.Output != "" {
t.Errorf("Unexpected response.\nExpecting to be:\n %q\nGot:\n %q\n", s, r.Output)
t.Errorf("Unexpected response.\nExpecting to be: \n\"\"\n Got:\n %q\n", r.Output)
}

r = noRRSetupTestSilenced("--strtwo=a bogus")
if r.Output != "" {
t.Errorf("Unexpected response.\nExpecting to be:\n %q\nGot:\n %q\n", s, r.Output)
t.Errorf("Unexpected response.\nExpecting to be:\n\"\"\nGot:\n %q\n", r.Output)
}
}

Expand Down
8 changes: 4 additions & 4 deletions command_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -142,11 +142,11 @@ func TestCommandsAreSorted(t *testing.T) {

var tmpCommand = &Command{Use: "tmp"}

for _, name := range(originalNames) {
for _, name := range originalNames {
tmpCommand.AddCommand(&Command{Use: name})
}

for i, c := range(tmpCommand.Commands()) {
for i, c := range tmpCommand.Commands() {
if expectedNames[i] != c.Name() {
t.Errorf("expected: %s, got: %s", expectedNames[i], c.Name())
}
Expand All @@ -162,11 +162,11 @@ func TestEnableCommandSortingIsDisabled(t *testing.T) {

var tmpCommand = &Command{Use: "tmp"}

for _, name := range(originalNames) {
for _, name := range originalNames {
tmpCommand.AddCommand(&Command{Use: name})
}

for i, c := range(tmpCommand.Commands()) {
for i, c := range tmpCommand.Commands() {
if originalNames[i] != c.Name() {
t.Errorf("expected: %s, got: %s", originalNames[i], c.Name())
}
Expand Down
2 changes: 1 addition & 1 deletion doc/man_docs.go
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ func GenManTreeFromOpts(cmd *cobra.Command, opts GenManTreeOptions) error {
separator = opts.CommandSeparator
}
basename := strings.Replace(cmd.CommandPath(), " ", separator, -1)
filename := filepath.Join(opts.Path, basename + "." + section)
filename := filepath.Join(opts.Path, basename+"."+section)
f, err := os.Create(filename)
if err != nil {
return err
Expand Down

0 comments on commit d1371ec

Please sign in to comment.