fix: remove showing app name twice in errors

alecthomas · Jul 21, 2023 · 5a2e694 · 5a2e694
1 parent a9be85c
commit 5a2e694
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/context.go b/context.go
@@ -201,15 +201,19 @@ func (c *Context) Validate() error { // nolint: gocyclo
 
 		case *Application:
 			value = node.Target
-			desc = node.Name
+			desc = ""
 
 		case *Node:
 			value = node.Target
 			desc = node.Path()
 		}
 		if validate := isValidatable(value); validate != nil {
 			if err := validate.Validate(); err != nil {
-				return fmt.Errorf("%s: %w", desc, err)
+				if desc != "" {
+					return fmt.Errorf("%s: %w", desc, err)
+				} else {
+					return err
+				}
 			}
 		}
 	}

diff --git a/kong_test.go b/kong_test.go
@@ -1214,7 +1214,7 @@ func TestValidateApp(t *testing.T) {
 	cli := validateCli{}
 	p := mustNew(t, &cli)
 	_, err := p.Parse([]string{})
-	assert.EqualError(t, err, "test: app error")
+	assert.EqualError(t, err, "app error")
 }
 
 func TestValidateCmd(t *testing.T) {