Tweak flags so we don't repeat ourselves too much

internal/cli/flags.go

@@ -15,6 +15,12 @@ type Flag struct {
 	IsRequired bool
 }
 
+func (f *Flag) Required() *Flag {
+	clone := *f
+	clone.IsRequired = true
+	return &clone
+}
+
 func (f *Flag) Ask(cmd *cobra.Command, value interface{}) error {
 	return askInput(cmd, f, value, false)
 }

internal/cli/flags_test.go

@@ -0,0 +1,18 @@
+package cli
+
+import "testing"
+
+func TestFlagsRequiredClone(t *testing.T) {
+	orig := &Flag{
+		Name: "some-flag",
+	}
+
+	clone := orig.Required()
+	if !clone.IsRequired {
+		t.Fatal("wanted flag to be required")
+	}
+
+	if orig.IsRequired {
+		t.Fatal("wanted original to be left intact")
+	}
+}

internal/cli/rules.go

@@ -11,14 +11,6 @@ import (
 )
 
 var (
-	ruleNameRequired = Flag{
-		Name:       "Name",
-		LongForm:   "name",
-		ShortForm:  "n",
-		Help:       "Name of the rule.",
-		IsRequired: true,
-	}
-
 	ruleName = Flag{
 		Name:      "Name",
 		LongForm:  "name",
@@ -113,7 +105,7 @@ auth0 rules create --name "My Rule" --template [empty-rule]"
 			prepareInteractivity(cmd)
 		},
 		RunE: func(cmd *cobra.Command, args []string) error {
-			if err := ruleNameRequired.Ask(cmd, &inputs.Name); err != nil {
+			if err := ruleName.Required().Ask(cmd, &inputs.Name); err != nil {
 				return err
 			}
 
@@ -151,7 +143,7 @@ auth0 rules create --name "My Rule" --template [empty-rule]"
 		},
 	}
 
-	ruleNameRequired.RegisterString(cmd, &inputs.Name, "")
+	ruleName.Required().RegisterString(cmd, &inputs.Name, "")
 	ruleTemplate.RegisterString(cmd, &inputs.Template, "")
 	ruleEnabled.RegisterBool(cmd, &inputs.Enabled, true)