From b37c1d899349fef9e7af38940ae0c2d507c433a9 Mon Sep 17 00:00:00 2001 From: Ophestra Date: Fri, 7 Aug 2026 18:57:01 +0900 Subject: command: use NUL for magic usage string This change also improves documentation. Signed-off-by: Ophestra --- command/builder_test.go | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) (limited to 'command/builder_test.go') diff --git a/command/builder_test.go b/command/builder_test.go index 7b9a1701..05e29358 100644 --- a/command/builder_test.go +++ b/command/builder_test.go @@ -18,8 +18,14 @@ func TestBuild(t *testing.T) { t.Run("direct zero length", func(t *testing.T) { wantPanic := "invalid subcommand" - t.Run("zero length name", func(t *testing.T) { defer checkRecover(t, "Command", wantPanic); c.Command("", "usage", stubHandler) }) - t.Run("zero length usage", func(t *testing.T) { defer checkRecover(t, "Command", wantPanic); c.Command("name", "", stubHandler) }) + t.Run("zero length name", func(t *testing.T) { + defer checkRecover(t, "Command", wantPanic) + c.Command("", "usage", stubHandler) + }) + t.Run("zero length usage", func(t *testing.T) { + defer checkRecover(t, "Command", wantPanic) + c.Command("name", "", stubHandler) + }) }) t.Run("direct adopt unique names", func(t *testing.T) { @@ -34,8 +40,14 @@ func TestBuild(t *testing.T) { t.Run("zero length", func(t *testing.T) { wantPanic := "invalid subcommand tree" - t.Run("zero length name", func(t *testing.T) { defer checkRecover(t, "New", wantPanic); c.New("", "usage") }) - t.Run("zero length usage", func(t *testing.T) { defer checkRecover(t, "New", wantPanic); c.New("name", "") }) + t.Run("zero length name", func(t *testing.T) { + defer checkRecover(t, "New", wantPanic) + c.New("", "usage") + }) + t.Run("zero length usage", func(t *testing.T) { + defer checkRecover(t, "New", wantPanic) + c.New("name", "") + }) }) t.Run("direct adopt unique names", func(t *testing.T) { -- cgit v1.3.1