Allow choices text to default to value

Signed-off-by: Mathieu Frenette <[email protected]>
Samasource · Mar 26, 2021 · a68f789 · a68f789
1 parent 22f3a27
commit a68f789
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/src/internal/spec/spec.go b/src/internal/spec/spec.go
@@ -338,11 +338,11 @@ func loadChoiceStep(_map yaml.Map) (exec.Executable, error) {
 		if !ok {
 			return nil, fmt.Errorf("items of %q property must be objects", "options")
 		}
-		text, err := getRequiredStringFromMap(childMap, "text")
+		value, err := getRequiredStringFromMap(childMap, "value")
 		if err != nil {
 			return nil, err
 		}
-		value, err := getRequiredStringFromMap(childMap, "value")
+		text, err := getOptionalStringFromMap(childMap, "text", value)
 		if err != nil {
 			return nil, err
 		}

diff --git a/src/internal/spec/spec_test.go b/src/internal/spec/spec_test.go
@@ -199,8 +199,7 @@ choice:
       value: Value 1
     - text: Text 2
       value: Value 2
-    - text: Text 3
-      value: Value 3`,
+    - value: Value 3`,
 			Expected: choice.Prompt{
 				Message: "Message",
 				Var:     "Variable",
@@ -215,7 +214,7 @@ choice:
 						Value: "Value 2",
 					},
 					{
-						Text:  "Text 3",
+						Text:  "Value 3",
 						Value: "Value 3",
 					},
 				},