Merge pull request #76 from mitchellh/b-default

Ignore default command for autocomplete init

cli.go

@@ -419,6 +419,11 @@ func (c *CLI) initAutocomplete() {
 func (c *CLI) initAutocompleteSub(prefix string) complete.Command {
 	var cmd complete.Command
 	walkFn := func(k string, raw interface{}) bool {
+		// Ignore the empty key which can be present for default commands.
+		if k == "" {
+			return false
+		}
+
 		// Keep track of the full key so that we can nest further if necessary
 		fullKey := k
 

cli_test.go

@@ -205,6 +205,32 @@ func TestCLIRun_default(t *testing.T) {
 	}
 }
 
+// GH-74: When using NewCLI with a default command only, Run would
+// stack overflow and crash.
+func TestCLIRun_defaultFromNew(t *testing.T) {
+	commandBar := new(MockCommand)
+
+	cli := NewCLI("test", "0.1.0")
+	cli.Commands = map[string]CommandFactory{
+		"": func() (Command, error) {
+			return commandBar, nil
+		},
+	}
+
+	exitCode, err := cli.Run()
+	if err != nil {
+		t.Fatalf("err: %s", err)
+	}
+
+	if exitCode != commandBar.RunResult {
+		t.Fatalf("bad: %d", exitCode)
+	}
+
+	if !commandBar.RunCalled {
+		t.Fatalf("run should be called")
+	}
+}
+
 func TestCLIRun_helpNested(t *testing.T) {
 	helpCalled := false
 	buf := new(bytes.Buffer)