error if both --build-id and --unversioned used (#991)

The CLI documentation for the `--build-id` and `--unversioned` flags for
the `temporal worker deployment set-current-version` command indicate
that these flags are mutually exclusive:

```
> ./temporal worker deployment set-current-version --help
<snip>
      --build-id string              Build ID of the Worker Deployment Version. Required unless --unversioned is specified.
      --deployment-name string       Name of the Worker Deployment. Required.
  -h, --help                         help for set-current-version
      --ignore-missing-task-queues   Override protection to accidentally remove task queues.
      --unversioned                  Set unversioned workers as the target version. Cannot be used with --build-id.
```

However, there was no error returned if both `--build-id` and
`--unversioned` were supplied on the command line.

This commit returns an error indicating that the flags are mutually
exclusive and adds a simple unit test verifying the behaviour.

Signed-off-by: Jay Pipes <jay.pipes@temporal.io>
Co-authored-by: Alex Stanfield <13949480+chaptersix@users.noreply.github.com>

Author: jaypipes

internal/temporalcli/commands.worker.deployment.go

@@ -832,6 +832,10 @@ func (c *TemporalWorkerDeploymentSetCurrentVersionCommand) run(cctx *CommandCont
 	}
 	defer cl.Close()
 
+	if c.BuildId != "" && c.Unversioned {
+		return fmt.Errorf("specify either --build-id or --unversioned, not both")
+	}
+
 	token, err := c.Parent.getConflictToken(cctx, &getDeploymentConflictTokenOptions{
 		safeMode:        !c.Yes,
 		safeModeMessage: "Current",

internal/temporalcli/commands.worker.deployment_test.go

@@ -186,6 +186,17 @@ func (s *SharedServerSuite) TestDeployment_Set_Current_Version() {
 	s.NoError(res.Err)
 	s.NoError(json.Unmarshal(res.Stdout.Bytes(), &jsonVersionOut))
 	s.Nil(jsonVersionOut.Metadata)
+
+	// --build-id and --unversioned are either/or. there should be an error if
+	// the user specifies both flags.
+	res = s.Execute(
+		"worker", "deployment", "set-current-version",
+		"--address", s.Address(),
+		"--deployment-name", version.DeploymentName, "--build-id", version.BuildID,
+		"--unversioned",
+	)
+	s.Error(res.Err)
+	s.ErrorContains(res.Err, "specify either --build-id or --unversioned")
 }
 
 func (s *SharedServerSuite) TestDeployment_Set_Current_Version_AllowNoPollers() {