Fix vcl condition --comment flag

The --comment flag was setting the statement instead of the
comment, and was missing from the no-arguments validation check.

Author: jedisct1

CHANGELOG.md

@@ -6,6 +6,8 @@
 
 ### Bug Fixes:
 
+- fix(vcl/condition): `--comment` flag in `condition update` now correctly sets the comment instead of overwriting the statement
+
 ### Enhancements:
 
 ### Dependencies:

pkg/commands/service/vcl/condition/condition_test.go

@@ -86,7 +86,7 @@ func TestConditionUpdate(t *testing.T) {
 				CloneVersionFn:    testutil.CloneVersionResult(4),
 				UpdateConditionFn: updateConditionOK,
 			},
-			WantError: "error parsing arguments: must provide either --new-name, --statement, --type or --priority to update condition",
+			WantError: "error parsing arguments: must provide either --new-name, --statement, --type, --priority or --comment to update condition",
 		},
 		{
 			Args: "--service-id 123 --version 1 --name always_false --new-name false_always --autoclone",

pkg/commands/service/vcl/condition/update.go

@@ -97,8 +97,8 @@ func (c *UpdateCommand) Exec(_ io.Reader, out io.Writer) error {
 	c.input.ServiceVersion = fastly.ToValue(serviceVersion.Number)
 
 	// If no argument are provided, error with useful message.
-	if !c.newName.WasSet && !c.priority.WasSet && !c.statement.WasSet && !c.conditionType.WasSet {
-		return fmt.Errorf("error parsing arguments: must provide either --new-name, --statement, --type or --priority to update condition")
+	if !c.newName.WasSet && !c.priority.WasSet && !c.statement.WasSet && !c.conditionType.WasSet && !c.comment.WasSet {
+		return fmt.Errorf("error parsing arguments: must provide either --new-name, --statement, --type, --priority or --comment to update condition")
 	}
 
 	if c.newName.WasSet {
@@ -114,7 +114,7 @@ func (c *UpdateCommand) Exec(_ io.Reader, out io.Writer) error {
 		c.input.Statement = &c.statement.Value
 	}
 	if c.comment.WasSet {
-		c.input.Statement = &c.comment.Value
+		c.input.Comment = &c.comment.Value
 	}
 
 	r, err := c.Globals.APIClient.UpdateCondition(context.TODO(), &c.input)