diff --git a/docs/stackit_beta_network-area_route.md b/docs/stackit_beta_network-area_route.md index 39efe6da8..767afefe3 100644 --- a/docs/stackit_beta_network-area_route.md +++ b/docs/stackit_beta_network-area_route.md @@ -33,4 +33,5 @@ stackit beta network-area route [flags] * [stackit beta network-area route delete](./stackit_beta_network-area_route_delete.md) - Deletes a static route in a STACKIT Network Area (SNA) * [stackit beta network-area route describe](./stackit_beta_network-area_route_describe.md) - Shows details of a static route in a STACKIT Network Area (SNA) * [stackit beta network-area route list](./stackit_beta_network-area_route_list.md) - Lists all static routes in a STACKIT Network Area (SNA) +* [stackit beta network-area route update](./stackit_beta_network-area_route_update.md) - Updates a static route in a STACKIT Network Area (SNA) diff --git a/docs/stackit_beta_network-area_route_create.md b/docs/stackit_beta_network-area_route_create.md index f9fca71f2..41a7d30c1 100644 --- a/docs/stackit_beta_network-area_route_create.md +++ b/docs/stackit_beta_network-area_route_create.md @@ -17,12 +17,16 @@ stackit beta network-area route create [flags] ``` Create a static route with prefix "1.1.1.0/24" and next hop "1.1.1.1" in a STACKIT Network Area with ID "xxx" in organization with ID "yyy" $ stackit beta network-area route create --organization-id yyy --network-area-id xxx --prefix 1.1.1.0/24 --next-hop 1.1.1.1 + + Create a static route with labels "key:value" and "foo:bar" with prefix "1.1.1.0/24" and next hop "1.1.1.1" in a STACKIT Network Area with ID "xxx" in organization with ID "yyy" + $ stackit beta network-area route create --labels key=value,foo=bar --organization-id yyy --network-area-id xxx --prefix 1.1.1.0/24 --next-hop 1.1.1.1 ``` ### Options ``` -h, --help Help for "stackit beta network-area route create" + --labels stringToString Labels are key-value string pairs which can be attached to a route. A label can be provided with the format key=value and the flag can be used multiple times to provide a list of labels (default []) --network-area-id string STACKIT Network Area ID --next-hop string Next hop IP address. Must be a valid IPv4 --organization-id string Organization ID diff --git a/docs/stackit_beta_network-area_route_update.md b/docs/stackit_beta_network-area_route_update.md new file mode 100644 index 000000000..e52a914ad --- /dev/null +++ b/docs/stackit_beta_network-area_route_update.md @@ -0,0 +1,44 @@ +## stackit beta network-area route update + +Updates a static route in a STACKIT Network Area (SNA) + +### Synopsis + +Updates a static route in a STACKIT Network Area (SNA). +This command is currently asynchonous only due to limitations in the waiting functionality of the SDK. This will be updated in a future release. + + +``` +stackit beta network-area route update [flags] +``` + +### Examples + +``` + Updates the label(s) of a static route with ID "xxx" in a STACKIT Network Area with ID "yyy" in organization with ID "zzz" + $ stackit beta network-area route update xxx --labels key=value,foo=bar --organization-id yyy --network-area-id zzz +``` + +### Options + +``` + -h, --help Help for "stackit beta network-area route update" + --labels stringToString Labels are key-value string pairs which can be attached to a route. A label can be provided with the format key=value and the flag can be used multiple times to provide a list of labels (default []) + --network-area-id string STACKIT Network Area ID + --organization-id string Organization ID +``` + +### Options inherited from parent commands + +``` + -y, --assume-yes If set, skips all confirmation prompts + --async If set, runs the command asynchronously + -o, --output-format string Output format, one of ["json" "pretty" "none" "yaml"] + -p, --project-id string Project ID + --verbosity string Verbosity of the CLI, one of ["debug" "info" "warning" "error"] (default "info") +``` + +### SEE ALSO + +* [stackit beta network-area route](./stackit_beta_network-area_route.md) - Provides functionality for static routes in STACKIT Network Areas + diff --git a/internal/cmd/beta/network-area/route/create/create.go b/internal/cmd/beta/network-area/route/create/create.go index b354d7d91..8c2673236 100644 --- a/internal/cmd/beta/network-area/route/create/create.go +++ b/internal/cmd/beta/network-area/route/create/create.go @@ -12,7 +12,8 @@ import ( "github.com/stackitcloud/stackit-cli/internal/pkg/globalflags" "github.com/stackitcloud/stackit-cli/internal/pkg/print" "github.com/stackitcloud/stackit-cli/internal/pkg/services/iaas/client" - "github.com/stackitcloud/stackit-cli/internal/pkg/services/iaas/utils" + iaasUtils "github.com/stackitcloud/stackit-cli/internal/pkg/services/iaas/utils" + "github.com/stackitcloud/stackit-cli/internal/pkg/utils" "github.com/stackitcloud/stackit-sdk-go/services/iaas" "github.com/spf13/cobra" @@ -23,6 +24,7 @@ const ( networkAreaIdFlag = "network-area-id" prefixFlag = "prefix" nexthopFlag = "next-hop" + labelFlag = "labels" ) type inputModel struct { @@ -31,6 +33,7 @@ type inputModel struct { NetworkAreaId *string Prefix *string Nexthop *string + Labels *map[string]string } func NewCmd(p *print.Printer) *cobra.Command { @@ -47,6 +50,10 @@ func NewCmd(p *print.Printer) *cobra.Command { `Create a static route with prefix "1.1.1.0/24" and next hop "1.1.1.1" in a STACKIT Network Area with ID "xxx" in organization with ID "yyy"`, "$ stackit beta network-area route create --organization-id yyy --network-area-id xxx --prefix 1.1.1.0/24 --next-hop 1.1.1.1", ), + examples.NewExample( + `Create a static route with labels "key:value" and "foo:bar" with prefix "1.1.1.0/24" and next hop "1.1.1.1" in a STACKIT Network Area with ID "xxx" in organization with ID "yyy"`, + "$ stackit beta network-area route create --labels key=value,foo=bar --organization-id yyy --network-area-id xxx --prefix 1.1.1.0/24 --next-hop 1.1.1.1", + ), ), RunE: func(cmd *cobra.Command, args []string) error { ctx := context.Background() @@ -62,7 +69,7 @@ func NewCmd(p *print.Printer) *cobra.Command { } // Get network area label - networkAreaLabel, err := utils.GetNetworkAreaName(ctx, apiClient, *model.OrganizationId, *model.NetworkAreaId) + networkAreaLabel, err := iaasUtils.GetNetworkAreaName(ctx, apiClient, *model.OrganizationId, *model.NetworkAreaId) if err != nil { p.Debug(print.ErrorLevel, "get network area name: %v", err) networkAreaLabel = *model.NetworkAreaId @@ -87,7 +94,7 @@ func NewCmd(p *print.Printer) *cobra.Command { return fmt.Errorf("empty response from API") } - route, err := utils.GetRouteFromAPIResponse(*model.Prefix, *model.Nexthop, resp.Items) + route, err := iaasUtils.GetRouteFromAPIResponse(*model.Prefix, *model.Nexthop, resp.Items) if err != nil { return err } @@ -104,6 +111,7 @@ func configureFlags(cmd *cobra.Command) { cmd.Flags().Var(flags.UUIDFlag(), networkAreaIdFlag, "STACKIT Network Area ID") cmd.Flags().Var(flags.CIDRFlag(), prefixFlag, "Static route prefix") cmd.Flags().String(nexthopFlag, "", "Next hop IP address. Must be a valid IPv4") + cmd.Flags().StringToString(labelFlag, nil, "Labels are key-value string pairs which can be attached to a route. A label can be provided with the format key=value and the flag can be used multiple times to provide a list of labels") err := flags.MarkFlagsRequired(cmd, organizationIdFlag, networkAreaIdFlag, prefixFlag, nexthopFlag) cobra.CheckErr(err) @@ -118,6 +126,7 @@ func parseInput(p *print.Printer, cmd *cobra.Command) (*inputModel, error) { NetworkAreaId: flags.FlagToStringPointer(p, cmd, networkAreaIdFlag), Prefix: flags.FlagToStringPointer(p, cmd, prefixFlag), Nexthop: flags.FlagToStringPointer(p, cmd, nexthopFlag), + Labels: flags.FlagToStringToStringPointer(p, cmd, labelFlag), } if p.IsVerbosityDebug() { @@ -134,11 +143,22 @@ func parseInput(p *print.Printer, cmd *cobra.Command) (*inputModel, error) { func buildRequest(ctx context.Context, model *inputModel, apiClient *iaas.APIClient) iaas.ApiCreateNetworkAreaRouteRequest { req := apiClient.CreateNetworkAreaRoute(ctx, *model.OrganizationId, *model.NetworkAreaId) + + var labelsMap *map[string]interface{} + if model.Labels != nil && len(*model.Labels) > 0 { + // convert map[string]string to map[string]interface{} + labelsMap = utils.Ptr(map[string]interface{}{}) + for k, v := range *model.Labels { + (*labelsMap)[k] = v + } + } + payload := iaas.CreateNetworkAreaRoutePayload{ Ipv4: &[]iaas.Route{ { Prefix: model.Prefix, Nexthop: model.Nexthop, + Labels: labelsMap, }, }, } diff --git a/internal/cmd/beta/network-area/route/create/create_test.go b/internal/cmd/beta/network-area/route/create/create_test.go index 03a3a7bb0..b106a937b 100644 --- a/internal/cmd/beta/network-area/route/create/create_test.go +++ b/internal/cmd/beta/network-area/route/create/create_test.go @@ -165,6 +165,16 @@ func TestParseInput(t *testing.T) { }), isValid: false, }, + { + description: "optional labels is provided", + flagValues: fixtureFlagValues(func(flagValues map[string]string) { + flagValues[labelFlag] = "key=value" + }), + expectedModel: fixtureInputModel(func(model *inputModel) { + model.Labels = utils.Ptr(map[string]string{"key": "value"}) + }), + isValid: true, + }, } for _, tt := range tests { @@ -224,6 +234,17 @@ func TestBuildRequest(t *testing.T) { model: fixtureInputModel(), expectedRequest: fixtureRequest(), }, + { + description: "optional labels provided", + model: fixtureInputModel(func(model *inputModel) { + model.Labels = utils.Ptr(map[string]string{"key": "value"}) + }), + expectedRequest: fixtureRequest(func(request *iaas.ApiCreateNetworkAreaRouteRequest) { + *request = request.CreateNetworkAreaRoutePayload(fixturePayload(func(payload *iaas.CreateNetworkAreaRoutePayload) { + (*payload.Ipv4)[0].Labels = utils.Ptr(map[string]interface{}{"key": "value"}) + })) + }), + }, } for _, tt := range tests { diff --git a/internal/cmd/beta/network-area/route/describe/describe.go b/internal/cmd/beta/network-area/route/describe/describe.go index 793d9b96f..f20fced9d 100644 --- a/internal/cmd/beta/network-area/route/describe/describe.go +++ b/internal/cmd/beta/network-area/route/describe/describe.go @@ -4,6 +4,7 @@ import ( "context" "encoding/json" "fmt" + "strings" "github.com/goccy/go-yaml" "github.com/stackitcloud/stackit-cli/internal/pkg/args" @@ -134,9 +135,17 @@ func outputResult(p *print.Printer, outputFormat string, route *iaas.Route) erro table := tables.NewTable() table.AddRow("ID", *route.RouteId) table.AddSeparator() - table.AddRow("Prefix", *route.Prefix) + table.AddRow("PREFIX", *route.Prefix) table.AddSeparator() - table.AddRow("Nexthop", *route.Nexthop) + table.AddRow("NEXTHOP", *route.Nexthop) + if route.Labels != nil && len(*route.Labels) > 0 { + labels := []string{} + for key, value := range *route.Labels { + labels = append(labels, fmt.Sprintf("%s: %s", key, value)) + } + table.AddSeparator() + table.AddRow("LABELS", strings.Join(labels, "\n")) + } err := table.Display(p) if err != nil { diff --git a/internal/cmd/beta/network-area/route/routes.go b/internal/cmd/beta/network-area/route/routes.go index f4b5bfac9..1125d4d70 100644 --- a/internal/cmd/beta/network-area/route/routes.go +++ b/internal/cmd/beta/network-area/route/routes.go @@ -5,6 +5,7 @@ import ( "github.com/stackitcloud/stackit-cli/internal/cmd/beta/network-area/route/delete" "github.com/stackitcloud/stackit-cli/internal/cmd/beta/network-area/route/describe" "github.com/stackitcloud/stackit-cli/internal/cmd/beta/network-area/route/list" + "github.com/stackitcloud/stackit-cli/internal/cmd/beta/network-area/route/update" "github.com/stackitcloud/stackit-cli/internal/pkg/args" "github.com/stackitcloud/stackit-cli/internal/pkg/print" "github.com/stackitcloud/stackit-cli/internal/pkg/utils" @@ -29,4 +30,5 @@ func addSubcommands(cmd *cobra.Command, p *print.Printer) { cmd.AddCommand(delete.NewCmd(p)) cmd.AddCommand(describe.NewCmd(p)) cmd.AddCommand(list.NewCmd(p)) + cmd.AddCommand(update.NewCmd(p)) } diff --git a/internal/cmd/beta/network-area/route/update/update.go b/internal/cmd/beta/network-area/route/update/update.go new file mode 100644 index 000000000..4a04ca654 --- /dev/null +++ b/internal/cmd/beta/network-area/route/update/update.go @@ -0,0 +1,166 @@ +package update + +import ( + "context" + "encoding/json" + "fmt" + + "github.com/goccy/go-yaml" + "github.com/spf13/cobra" + "github.com/stackitcloud/stackit-cli/internal/pkg/args" + "github.com/stackitcloud/stackit-cli/internal/pkg/errors" + "github.com/stackitcloud/stackit-cli/internal/pkg/examples" + "github.com/stackitcloud/stackit-cli/internal/pkg/flags" + "github.com/stackitcloud/stackit-cli/internal/pkg/globalflags" + "github.com/stackitcloud/stackit-cli/internal/pkg/print" + "github.com/stackitcloud/stackit-cli/internal/pkg/services/iaas/client" + iaasUtils "github.com/stackitcloud/stackit-cli/internal/pkg/services/iaas/utils" + "github.com/stackitcloud/stackit-cli/internal/pkg/utils" + "github.com/stackitcloud/stackit-sdk-go/services/iaas" +) + +const ( + routeIdArg = "ROUTE_ID" + + organizationIdFlag = "organization-id" + networkAreaIdFlag = "network-area-id" + + labelFlag = "labels" +) + +type inputModel struct { + *globalflags.GlobalFlagModel + OrganizationId *string + NetworkAreaId *string + RouteId string + Labels *map[string]string +} + +func NewCmd(p *print.Printer) *cobra.Command { + cmd := &cobra.Command{ + Use: "update", + Short: "Updates a static route in a STACKIT Network Area (SNA)", + Long: fmt.Sprintf("%s\n%s\n", + "Updates a static route in a STACKIT Network Area (SNA).", + "This command is currently asynchonous only due to limitations in the waiting functionality of the SDK. This will be updated in a future release.", + ), + Args: args.SingleArg(routeIdArg, utils.ValidateUUID), + Example: examples.Build( + examples.NewExample( + `Updates the label(s) of a static route with ID "xxx" in a STACKIT Network Area with ID "yyy" in organization with ID "zzz"`, + "$ stackit beta network-area route update xxx --labels key=value,foo=bar --organization-id yyy --network-area-id zzz", + ), + ), + RunE: func(cmd *cobra.Command, args []string) error { + ctx := context.Background() + model, err := parseInput(p, cmd, args) + if err != nil { + return err + } + + // Configure API client + apiClient, err := client.ConfigureClient(p) + if err != nil { + return err + } + + // Get network area label + networkAreaLabel, err := iaasUtils.GetNetworkAreaName(ctx, apiClient, *model.OrganizationId, *model.NetworkAreaId) + if err != nil { + p.Debug(print.ErrorLevel, "get network area name: %v", err) + networkAreaLabel = *model.NetworkAreaId + } + + // Call API + req := buildRequest(ctx, model, apiClient) + resp, err := req.Execute() + if err != nil { + return fmt.Errorf("create static route: %w", err) + } + + return outputResult(p, model, networkAreaLabel, *resp) + }, + } + configureFlags(cmd) + return cmd +} + +func configureFlags(cmd *cobra.Command) { + cmd.Flags().Var(flags.UUIDFlag(), organizationIdFlag, "Organization ID") + cmd.Flags().Var(flags.UUIDFlag(), networkAreaIdFlag, "STACKIT Network Area ID") + cmd.Flags().StringToString(labelFlag, nil, "Labels are key-value string pairs which can be attached to a route. A label can be provided with the format key=value and the flag can be used multiple times to provide a list of labels") + + err := flags.MarkFlagsRequired(cmd, organizationIdFlag, networkAreaIdFlag) + cobra.CheckErr(err) +} + +func parseInput(p *print.Printer, cmd *cobra.Command, inputArgs []string) (*inputModel, error) { + routeId := inputArgs[0] + globalFlags := globalflags.Parse(p, cmd) + + labels := flags.FlagToStringToStringPointer(p, cmd, labelFlag) + + if labels == nil { + return nil, &errors.EmptyUpdateError{} + } + + model := inputModel{ + GlobalFlagModel: globalFlags, + OrganizationId: flags.FlagToStringPointer(p, cmd, organizationIdFlag), + NetworkAreaId: flags.FlagToStringPointer(p, cmd, networkAreaIdFlag), + RouteId: routeId, + Labels: labels, + } + + if p.IsVerbosityDebug() { + modelStr, err := print.BuildDebugStrFromInputModel(model) + if err != nil { + p.Debug(print.ErrorLevel, "convert model to string for debugging: %v", err) + } else { + p.Debug(print.DebugLevel, "parsed input values: %s", modelStr) + } + } + + return &model, nil +} + +func buildRequest(ctx context.Context, model *inputModel, apiClient *iaas.APIClient) iaas.ApiUpdateNetworkAreaRouteRequest { + req := apiClient.UpdateNetworkAreaRoute(ctx, *model.OrganizationId, *model.NetworkAreaId, model.RouteId) + + // convert map[string]string to map[string]interface{} + labelsMap := make(map[string]interface{}) + for k, v := range *model.Labels { + labelsMap[k] = v + } + + payload := iaas.UpdateNetworkAreaRoutePayload{ + Labels: &labelsMap, + } + req = req.UpdateNetworkAreaRoutePayload(payload) + + return req +} + +func outputResult(p *print.Printer, model *inputModel, networkAreaLabel string, route iaas.Route) error { + switch model.OutputFormat { + case print.JSONOutputFormat: + details, err := json.MarshalIndent(route, "", " ") + if err != nil { + return fmt.Errorf("marshal static route: %w", err) + } + p.Outputln(string(details)) + + return nil + case print.YAMLOutputFormat: + details, err := yaml.MarshalWithOptions(route, yaml.IndentSequence(true)) + if err != nil { + return fmt.Errorf("marshal static route: %w", err) + } + p.Outputln(string(details)) + + return nil + default: + p.Outputf("Updated static route for SNA %q.\nStatic route ID: %s\n", networkAreaLabel, *route.RouteId) + return nil + } +} diff --git a/internal/cmd/beta/network-area/route/update/update_test.go b/internal/cmd/beta/network-area/route/update/update_test.go new file mode 100644 index 000000000..f9b1ddd06 --- /dev/null +++ b/internal/cmd/beta/network-area/route/update/update_test.go @@ -0,0 +1,269 @@ +package update + +import ( + "context" + "testing" + + "github.com/google/go-cmp/cmp" + "github.com/google/go-cmp/cmp/cmpopts" + "github.com/google/uuid" + "github.com/stackitcloud/stackit-cli/internal/pkg/globalflags" + "github.com/stackitcloud/stackit-cli/internal/pkg/print" + "github.com/stackitcloud/stackit-cli/internal/pkg/utils" + "github.com/stackitcloud/stackit-sdk-go/services/iaas" +) + +type testCtxKey struct{} + +var testCtx = context.WithValue(context.Background(), testCtxKey{}, "foo") +var testClient = &iaas.APIClient{} + +var testOrgId = uuid.NewString() +var testNetworkAreaId = uuid.NewString() +var testRouteId = uuid.NewString() + +func fixtureArgValues(mods ...func(argValues []string)) []string { + argValues := []string{ + testRouteId, + } + for _, mod := range mods { + mod(argValues) + } + return argValues +} + +func fixtureFlagValues(mods ...func(flagValues map[string]string)) map[string]string { + flagValues := map[string]string{ + organizationIdFlag: testOrgId, + networkAreaIdFlag: testNetworkAreaId, + labelFlag: "value=key", + } + for _, mod := range mods { + mod(flagValues) + } + return flagValues +} + +func fixturePayload(mods ...func(payload *iaas.UpdateNetworkAreaRoutePayload)) iaas.UpdateNetworkAreaRoutePayload { + payload := iaas.UpdateNetworkAreaRoutePayload{ + Labels: &map[string]interface{}{ + "value": "key", + }, + } + + for _, mod := range mods { + mod(&payload) + } + return payload +} + +func fixturePayloadAsStringMap() map[string]string { + payload := fixturePayload() + labelsMap := make(map[string]string) + for k, v := range *payload.Labels { + if value, ok := v.(string); ok { + labelsMap[k] = value + } + } + return labelsMap +} + +func fixtureInputModel(mods ...func(model *inputModel)) *inputModel { + payload := fixturePayloadAsStringMap() + model := &inputModel{ + GlobalFlagModel: &globalflags.GlobalFlagModel{ + Verbosity: globalflags.VerbosityDefault, + }, + OrganizationId: utils.Ptr(testOrgId), + NetworkAreaId: utils.Ptr(testNetworkAreaId), + RouteId: testRouteId, + Labels: utils.Ptr(payload), + } + for _, mod := range mods { + mod(model) + } + return model +} + +func fixtureRequest(mods ...func(request *iaas.ApiUpdateNetworkAreaRouteRequest)) iaas.ApiUpdateNetworkAreaRouteRequest { + request := testClient.UpdateNetworkAreaRoute(testCtx, testOrgId, testNetworkAreaId, testRouteId) + request = request.UpdateNetworkAreaRoutePayload(fixturePayload()) + for _, mod := range mods { + mod(&request) + } + return request +} + +func TestParseInput(t *testing.T) { + tests := []struct { + description string + argValues []string + flagValues map[string]string + isValid bool + expectedModel *inputModel + }{ + { + description: "base", + argValues: fixtureArgValues(), + flagValues: fixtureFlagValues(), + isValid: true, + expectedModel: fixtureInputModel(), + }, + { + description: "no values", + flagValues: map[string]string{}, + isValid: false, + }, + { + description: "org id missing", + flagValues: fixtureFlagValues(func(flagValues map[string]string) { + delete(flagValues, organizationIdFlag) + }), + isValid: false, + }, + { + description: "org id invalid 1", + flagValues: fixtureFlagValues(func(flagValues map[string]string) { + flagValues[organizationIdFlag] = "" + }), + isValid: false, + }, + { + description: "org area id invalid 2", + flagValues: fixtureFlagValues(func(flagValues map[string]string) { + flagValues[organizationIdFlag] = "invalid-uuid" + }), + isValid: false, + }, + { + description: "network area id missing", + flagValues: fixtureFlagValues(func(flagValues map[string]string) { + delete(flagValues, networkAreaIdFlag) + }), + isValid: false, + }, + { + description: "network area id invalid 1", + flagValues: fixtureFlagValues(func(flagValues map[string]string) { + flagValues[networkAreaIdFlag] = "" + }), + isValid: false, + }, + { + description: "network area id invalid 2", + flagValues: fixtureFlagValues(func(flagValues map[string]string) { + flagValues[networkAreaIdFlag] = "invalid-uuid" + }), + isValid: false, + }, + { + description: "route id missing", + flagValues: fixtureFlagValues(func(flagValues map[string]string) { + delete(flagValues, routeIdArg) + }), + isValid: false, + }, + { + description: "route id invalid 1", + flagValues: fixtureFlagValues(func(flagValues map[string]string) { + flagValues[routeIdArg] = "" + }), + isValid: false, + }, + { + description: "route id invalid 2", + flagValues: fixtureFlagValues(func(flagValues map[string]string) { + flagValues[routeIdArg] = "invalid-uuid" + }), + isValid: false, + }, + { + description: "labels missing", + flagValues: fixtureFlagValues(func(flagValues map[string]string) { + delete(flagValues, labelFlag) + }), + isValid: false, + }, + } + + for _, tt := range tests { + t.Run(tt.description, func(t *testing.T) { + p := print.NewPrinter() + cmd := NewCmd(p) + err := globalflags.Configure(cmd.Flags()) + if err != nil { + t.Fatalf("configure global flags: %v", err) + } + + for flag, value := range tt.flagValues { + err := cmd.Flags().Set(flag, value) + if err != nil { + if !tt.isValid { + return + } + t.Fatalf("setting flag --%s=%s: %v", flag, value, err) + } + } + + err = cmd.ValidateArgs(tt.argValues) + if err != nil { + if !tt.isValid { + return + } + t.Fatalf("error parsing args: %v", err) + } + + err = cmd.ValidateRequiredFlags() + if err != nil { + if !tt.isValid { + return + } + t.Fatalf("error validating flags: %v", err) + } + + model, err := parseInput(p, cmd, tt.argValues) + if err != nil { + if !tt.isValid { + return + } + t.Fatalf("error parsing flags: %v", err) + } + + if !tt.isValid { + t.Fatalf("did not fail on invalid input") + } + diff := cmp.Diff(model, tt.expectedModel) + if diff != "" { + t.Fatalf("Data does not match: %s", diff) + } + }) + } +} + +func TestBuildRequest(t *testing.T) { + tests := []struct { + description string + model *inputModel + expectedRequest iaas.ApiUpdateNetworkAreaRouteRequest + }{ + { + description: "base", + model: fixtureInputModel(), + expectedRequest: fixtureRequest(), + }, + } + + for _, tt := range tests { + t.Run(tt.description, func(t *testing.T) { + request := buildRequest(testCtx, tt.model, testClient) + + diff := cmp.Diff(request, tt.expectedRequest, + cmp.AllowUnexported(tt.expectedRequest), + cmpopts.EquateComparable(testCtx), + ) + if diff != "" { + t.Fatalf("Data does not match: %s", diff) + } + }) + } +}