From 5df19ebe00e88d443e062c6661f52b7069f486d7 Mon Sep 17 00:00:00 2001
From: Dylan Allbee <dylan@allbee.org>
Date: Thu, 23 Jan 2020 21:09:21 -0800
Subject: Use command passed into p.Parse(...) write methods

It is currently impossible to programatically write help and usage
messages for subcommands, due to parser.WriteHelp and parser.WriteUsage
not taking the state of the parser into account.

Check for the existence of p.lastCmd and use it for the writers when
available.

Enables ability to write unit tests for subcommand help.
---
 usage_test.go | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

(limited to 'usage_test.go')

diff --git a/usage_test.go b/usage_test.go
index d9d33f0..24846ae 100644
--- a/usage_test.go
+++ b/usage_test.go
@@ -266,8 +266,45 @@ Options:
 	p, err := NewParser(Config{}, &args)
 	require.NoError(t, err)
 
+	var help bytes.Buffer
+	p.WriteHelp(&help)
+	assert.Equal(t, expectedHelp, help.String())
+}
+
+func TestUsagWithNestedSubcommands(t *testing.T) {
+	expectedHelp := `Usage: example child nested [--enable] OUTPUT
+
+Positional arguments:
+  OUTPUT
+
+Options:
+  --enable
+
+Global options:
+  --values VALUES        Values
+  --verbose, -v          verbosity level
+  --help, -h             display this help and exit
+`
+
+	var args struct {
+		Verbose bool `arg:"-v" help:"verbosity level"`
+		Child   *struct {
+			Values []float64 `help:"Values"`
+			Nested *struct {
+				Enable bool
+				Output string `arg:"positional,required"`
+			} `arg:"subcommand:nested"`
+		} `arg:"subcommand:child"`
+	}
+
 	os.Args[0] = "example"
+	p, err := NewParser(Config{}, &args)
+	require.NoError(t, err)
+
+	err = p.Parse([]string{"child", "nested", "value"})
+
 	var help bytes.Buffer
 	p.WriteHelp(&help)
+	fmt.Println(help.String())
 	assert.Equal(t, expectedHelp, help.String())
 }
-- 
cgit v1.2.3


From c24567c12e7036f9f18c62ea13bb85b84e3d80ff Mon Sep 17 00:00:00 2001
From: Dylan Allbee <dylan@allbee.org>
Date: Thu, 23 Jan 2020 21:36:24 -0800
Subject: Fix lint warnings

---
 usage.go      | 4 +---
 usage_test.go | 2 +-
 2 files changed, 2 insertions(+), 4 deletions(-)

(limited to 'usage_test.go')

diff --git a/usage.go b/usage.go
index c13835e..db43dd1 100644
--- a/usage.go
+++ b/usage.go
@@ -163,9 +163,7 @@ func (p *Parser) writeHelpForCommand(w io.Writer, cmd *command) {
 	var globals []*spec
 	ancestor := cmd.parent
 	for ancestor != nil {
-		for _, spec := range ancestor.specs {
-			globals = append(globals, spec)
-		}
+		globals = append(globals, ancestor.specs...)
 		ancestor = ancestor.parent
 	}
 
diff --git a/usage_test.go b/usage_test.go
index 24846ae..31b439b 100644
--- a/usage_test.go
+++ b/usage_test.go
@@ -301,7 +301,7 @@ Global options:
 	p, err := NewParser(Config{}, &args)
 	require.NoError(t, err)
 
-	err = p.Parse([]string{"child", "nested", "value"})
+	_ = p.Parse([]string{"child", "nested", "value"})
 
 	var help bytes.Buffer
 	p.WriteHelp(&help)
-- 
cgit v1.2.3