test: add actionRun guard to short flag tests per review feedback

Author: lihan3238

command_test.go

@@ -5817,13 +5817,15 @@ func TestDefaultCommandWithSubcommandFlags(t *testing.T) {
 
 func TestDefaultCommandWithShortFlag(t *testing.T) {
 	// Covers the short-flag splitting path in parseFlags when DefaultCommand is set
+	actionRun := false
 	cmd := &Command{
 		DefaultCommand: "run",
 		Commands: []*Command{
 			{
 				Name:  "run",
 				Usage: "run the app",
 				Action: func(ctx context.Context, cmd *Command) error {
+					actionRun = true
 					if cmd.String("foo") != "baz" {
 						return fmt.Errorf("expected foo=baz, got %s", cmd.String("foo"))
 					}
@@ -5841,10 +5843,12 @@ func TestDefaultCommandWithShortFlag(t *testing.T) {
 
 	err := cmd.Run(buildTestContext(t), []string{"c", "-f", "baz"})
 	assert.NoError(t, err)
+	assert.True(t, actionRun, "expected run action to be executed")
 }
 
 func TestDefaultCommandWithShortFlagHandling(t *testing.T) {
 	// Covers the shortOptionHandling for-loop path when DefaultCommand is set
+	actionRun := false
 	cmd := &Command{
 		UseShortOptionHandling: true,
 		DefaultCommand:         "run",
@@ -5853,6 +5857,7 @@ func TestDefaultCommandWithShortFlagHandling(t *testing.T) {
 				Name:  "run",
 				Usage: "run the app",
 				Action: func(ctx context.Context, cmd *Command) error {
+					actionRun = true
 					if cmd.String("foo") != "baz" {
 						return fmt.Errorf("expected foo=baz, got %s", cmd.String("foo"))
 					}
@@ -5869,4 +5874,5 @@ func TestDefaultCommandWithShortFlagHandling(t *testing.T) {
 	}
 	err := cmd.Run(buildTestContext(t), []string{"c", "-f", "baz"})
 	assert.NoError(t, err)
+	assert.True(t, actionRun, "expected run action to be executed")
 }