Merge pull request #101 from barrettj12/no-options

[documentation command] don't print options table if no flags

documentation.go

@@ -417,6 +417,9 @@ func (d *documentationCommand) formatFlags(c Command, info *Info) string {
 	f.VisitAll(func(f *gnuflag.Flag) {
 		flags[f.Value] = append(flags[f.Value], f)
 	})
+	if len(flags) == 0 {
+		return ""
+	}
 
 	// sort the output flags by shortest name for each group.
 	var byName flagsByName

documentation_test.go

@@ -19,6 +19,7 @@ func (s *documentationSuite) TestFormatCommand(c *gc.C) {
 		title    bool
 		expected string
 	}{{
+		// "smoke test" - just a regular command
 		command: &docTestCommand{
 			info: &cmd.Info{
 				Name:     "add-cloud",
@@ -56,6 +57,36 @@ details for add-cloud...
 
 ---
 
+`)[1:],
+	}, {
+		// no flags - don't print "Options" table
+		command: &docTestCommand{
+			info: &cmd.Info{
+				Name:     "foo",
+				Args:     "<args>",
+				Purpose:  "insert summary here...",
+				Doc:      "insert details here...",
+				Examples: "insert examples here...",
+			},
+			flags: []string{},
+		},
+		title: false,
+		expected: (`
+
+## Summary
+insert summary here...
+
+## Usage
+` + "```" + `juju foo [options] <args>` + "```" + `
+
+## Examples
+insert examples here...
+
+## Details
+insert details here...
+
+---
+
 `)[1:],
 	}}