fix backup create test

Author: Benjosh95

internal/cmd/volume/backup/create/create.go

@@ -118,12 +118,7 @@ func NewCmd(params *params.CmdParams) *cobra.Command {
 				s.Stop()
 			}
 
-			if model.Async {
-				params.Printer.Info("Triggered backup of %q in %q. Backup ID: %s\n", sourceLabel, projectLabel, *resp.Id)
-			} else {
-				params.Printer.Info("Created backup of %q in %q. Backup ID: %s\n", sourceLabel, projectLabel, *resp.Id)
-			}
-			return nil
+			return outputResult(params.Printer, model.OutputFormat, model.Async, sourceLabel, projectLabel, resp)
 		},
 	}
 
@@ -203,8 +198,7 @@ func buildRequest(ctx context.Context, model *inputModel, apiClient *iaas.APICli
 		Type: &model.SourceType,
 	}
 
-	req = req.CreateBackupPayload(*createPayload)
-	return req
+	return req.CreateBackupPayload(*createPayload)
 }
 
 func outputResult(p *print.Printer, outputFormat string, async bool, sourceLabel, projectLabel string, resp *iaas.Backup) error {

internal/cmd/volume/backup/create/create_test.go

@@ -17,12 +17,13 @@ import (
 type testCtxKey struct{}
 
 var (
-	testCtx       = context.WithValue(context.Background(), testCtxKey{}, "foo")
-	testClient    = &iaas.APIClient{}
-	testProjectId = uuid.NewString()
-	testSourceId  = uuid.NewString()
-	testName      = "my-backup"
-	testLabels    = map[string]string{"key1": "value1"}
+	testCtx        = context.WithValue(context.Background(), testCtxKey{}, "foo")
+	testClient     = &iaas.APIClient{}
+	testProjectId  = uuid.NewString()
+	testSourceId   = uuid.NewString()
+	testName       = "my-backup"
+	testLabels     = map[string]string{"key1": "value1"}
+	testSourceType = "volume"
 )
 
 func fixtureFlagValues(mods ...func(flagValues map[string]string)) map[string]string {
@@ -58,6 +59,18 @@ func fixtureInputModel(mods ...func(model *inputModel)) *inputModel {
 
 func fixtureRequest(mods ...func(request *iaas.ApiCreateBackupRequest)) iaas.ApiCreateBackupRequest {
 	request := testClient.CreateBackup(testCtx, testProjectId)
+
+	createPayload := iaas.NewCreateBackupPayloadWithDefaults()
+	createPayload.Name = &testName
+	createPayload.Labels = &map[string]interface{}{
+		"key1": "value1",
+	}
+	createPayload.Source = &iaas.BackupSource{
+		Id:   &testSourceId,
+		Type: &testSourceType,
+	}
+
+	request = request.CreateBackupPayload(*createPayload)
 	for _, mod := range mods {
 		mod(&request)
 	}