fix: Add nil checks for values which are printed to the console

internal/cmd/beta/image/create/create.go

@@ -395,7 +395,7 @@ func outputResult(p *print.Printer, model *inputModel, resp *iaas.ImageCreateRes
 
 		return nil
 	default:
-		p.Outputf("Created image %q with id %s\n", model.Name, *model.Id)
+		p.Outputf("Created image %q with id %s\n", model.Name, utils.PtrString(model.Id))
 		return nil
 	}
 }

internal/cmd/beta/key-pair/create/create.go

@@ -82,7 +82,7 @@ func NewCmd(p *print.Printer) *cobra.Command {
 				return fmt.Errorf("create key pair: %w", err)
 			}
 
-			return outputResult(p, model, resp)
+			return outputResult(p, model.GlobalFlagModel.OutputFormat, resp)
 		},
 	}
 	configureFlags(cmd)
@@ -141,8 +141,12 @@ func buildRequest(ctx context.Context, model *inputModel, apiClient *iaas.APICli
 	return req.CreateKeyPairPayload(payload)
 }
 
-func outputResult(p *print.Printer, model *inputModel, item *iaas.Keypair) error {
-	switch model.OutputFormat {
+func outputResult(p *print.Printer, outputFormat string, item *iaas.Keypair) error {
+	if item == nil {
+		return fmt.Errorf("no key pair found")
+	}
+
+	switch outputFormat {
 	case print.JSONOutputFormat:
 		details, err := json.MarshalIndent(item, "", "  ")
 		if err != nil {
@@ -156,7 +160,10 @@ func outputResult(p *print.Printer, model *inputModel, item *iaas.Keypair) error
 		}
 		p.Outputln(string(details))
 	default:
-		p.Outputf("Created key pair %q.\nkey pair Fingerprint: %q\n", *item.Name, *item.Fingerprint)
+		p.Outputf("Created key pair %q.\nkey pair Fingerprint: %q\n",
+			utils.PtrString(item.Name),
+			utils.PtrString(item.Fingerprint),
+		)
 	}
 	return nil
 }

internal/cmd/beta/key-pair/create/create_test.go

@@ -194,3 +194,42 @@ func TestBuildRequest(t *testing.T) {
 		})
 	}
 }
+
+func Test_outputResult(t *testing.T) {
+	type args struct {
+		item         *iaas.Keypair
+		outputFormat string
+	}
+	tests := []struct {
+		name    string
+		args    args
+		wantErr bool
+	}{
+		{
+			name: "empty",
+			args: args{
+				item:         nil,
+				outputFormat: "",
+			},
+			wantErr: true,
+		},
+		{
+			name: "base",
+			args: args{
+				item:         &iaas.Keypair{},
+				outputFormat: "",
+			},
+			wantErr: false,
+		},
+	}
+
+	p := print.NewPrinter()
+	p.Cmd = NewCmd(p)
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			if err := outputResult(p, tt.args.outputFormat, tt.args.item); (err != nil) != tt.wantErr {
+				t.Errorf("outputResult() error = %v, wantErr %v", err, tt.wantErr)
+			}
+		})
+	}
+}

internal/cmd/beta/key-pair/describe/describe.go

@@ -6,6 +6,8 @@ import (
 	"fmt"
 	"strings"
 
+	"github.com/stackitcloud/stackit-cli/internal/pkg/utils"
+
 	"github.com/stackitcloud/stackit-cli/internal/pkg/args"
 	"github.com/stackitcloud/stackit-cli/internal/pkg/examples"
 	"github.com/stackitcloud/stackit-cli/internal/pkg/flags"
@@ -108,6 +110,10 @@ func buildRequest(ctx context.Context, model *inputModel, apiClient *iaas.APICli
 }
 
 func outputResult(p *print.Printer, outputFormat string, showOnlyPublicKey bool, keyPair *iaas.Keypair) error {
+	if keyPair == nil {
+		p.Outputln("No keypair found.")
+		return nil
+	}
 	switch outputFormat {
 	case print.JSONOutputFormat:
 		details, err := json.MarshalIndent(keyPair, "", "  ")
@@ -145,10 +151,10 @@ func outputResult(p *print.Printer, outputFormat string, showOnlyPublicKey bool,
 			return nil
 		}
 		table := tables.NewTable()
-		table.AddRow("KEY PAIR NAME", *keyPair.Name)
+		table.AddRow("KEY PAIR NAME", utils.PtrString(keyPair.Name))
 		table.AddSeparator()
 
-		if *keyPair.Labels != nil && len(*keyPair.Labels) > 0 {
+		if keyPair.Labels != nil && len(*keyPair.Labels) > 0 {
 			var labels []string
 			for key, value := range *keyPair.Labels {
 				labels = append(labels, fmt.Sprintf("%s: %s", key, value))
@@ -157,17 +163,21 @@ func outputResult(p *print.Printer, outputFormat string, showOnlyPublicKey bool,
 			table.AddSeparator()
 		}
 
-		table.AddRow("FINGERPRINT", *keyPair.Fingerprint)
+		table.AddRow("FINGERPRINT", utils.PtrString(keyPair.Fingerprint))
 		table.AddSeparator()
 
-		truncatedPublicKey := (*keyPair.PublicKey)[:maxLengthPublicKey] + "..."
+		truncatedPublicKey := ""
+		if keyPair.PublicKey != nil {
+			truncatedPublicKey = (*keyPair.PublicKey)[:maxLengthPublicKey] + "..."
+		}
+
 		table.AddRow("PUBLIC KEY", truncatedPublicKey)
 		table.AddSeparator()
 
-		table.AddRow("CREATED AT", *keyPair.CreatedAt)
+		table.AddRow("CREATED AT", utils.PtrString(keyPair.CreatedAt))
 		table.AddSeparator()
 
-		table.AddRow("UPDATED AT", *keyPair.UpdatedAt)
+		table.AddRow("UPDATED AT", utils.PtrString(keyPair.UpdatedAt))
 		table.AddSeparator()
 
 		p.Outputln(table.Render())

internal/cmd/beta/key-pair/describe/describe_test.go

@@ -186,3 +186,44 @@ func TestBuildRequest(t *testing.T) {
 		})
 	}
 }
+
+func Test_outputResult(t *testing.T) {
+	type args struct {
+		outputFormat      string
+		showOnlyPublicKey bool
+		keyPair           *iaas.Keypair
+	}
+	tests := []struct {
+		name    string
+		args    args
+		wantErr bool
+	}{
+		{
+			name: "empty",
+			args: args{
+				outputFormat:      "",
+				showOnlyPublicKey: false,
+				keyPair:           nil,
+			},
+			wantErr: false,
+		},
+		{
+			name: "base",
+			args: args{
+				outputFormat:      "",
+				showOnlyPublicKey: false,
+				keyPair:           &iaas.Keypair{},
+			},
+			wantErr: false,
+		},
+	}
+	p := print.NewPrinter()
+	p.Cmd = NewCmd(p)
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			if err := outputResult(p, tt.args.outputFormat, tt.args.showOnlyPublicKey, tt.args.keyPair); (err != nil) != tt.wantErr {
+				t.Errorf("outputResult() error = %v, wantErr %v", err, tt.wantErr)
+			}
+		})
+	}
+}

internal/cmd/beta/key-pair/list/list.go

@@ -6,6 +6,8 @@ import (
 	"fmt"
 	"strings"
 
+	"github.com/stackitcloud/stackit-cli/internal/pkg/utils"
+
 	"github.com/stackitcloud/stackit-cli/internal/pkg/args"
 	"github.com/stackitcloud/stackit-cli/internal/pkg/errors"
 	"github.com/stackitcloud/stackit-cli/internal/pkg/examples"
@@ -158,11 +160,19 @@ func outputResult(p *print.Printer, outputFormat string, keyPairs []iaas.Keypair
 			keyPair := keyPairs[idx]
 
 			var labels []string
-			for key, value := range *keyPair.Labels {
-				labels = append(labels, fmt.Sprintf("%s: %s", key, value))
+			if keyPair.Labels != nil {
+				for key, value := range *keyPair.Labels {
+					labels = append(labels, fmt.Sprintf("%s: %s", key, value))
+				}
 			}
 
-			table.AddRow(*keyPair.Name, strings.Join(labels, ", "), *keyPair.Fingerprint, *keyPair.CreatedAt, *keyPair.UpdatedAt)
+			table.AddRow(
+				utils.PtrString(keyPair.Name),
+				strings.Join(labels, ", "),
+				utils.PtrString(keyPair.Fingerprint),
+				utils.PtrString(keyPair.CreatedAt),
+				utils.PtrString(keyPair.UpdatedAt),
+			)
 		}
 
 		p.Outputln(table.Render())

internal/cmd/beta/key-pair/list/list_test.go

@@ -186,3 +186,37 @@ func TestBuildRequest(t *testing.T) {
 		})
 	}
 }
+
+func Test_outputResult(t *testing.T) {
+	type args struct {
+		outputFormat string
+		keyPairs     []iaas.Keypair
+	}
+	tests := []struct {
+		name    string
+		args    args
+		wantErr bool
+	}{
+		{
+			name: "empty",
+			args: args{
+				outputFormat: "",
+				keyPairs: []iaas.Keypair{
+					{},
+				},
+			},
+			wantErr: false,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			p := print.NewPrinter()
+			p.Cmd = NewCmd(p)
+
+			if err := outputResult(p, tt.args.outputFormat, tt.args.keyPairs); (err != nil) != tt.wantErr {
+				t.Errorf("outputResult() error = %v, wantErr %v", err, tt.wantErr)
+			}
+		})
+	}
+}

internal/cmd/beta/key-pair/update/update.go

@@ -123,7 +123,18 @@ func parseInput(p *print.Printer, cmd *cobra.Command, inputArgs []string) (*inpu
 }
 
 func outputResult(p *print.Printer, model *inputModel, keyPair *iaas.Keypair) error {
-	switch model.OutputFormat {
+	if model == nil {
+		return fmt.Errorf("model is nil")
+	}
+	if keyPair == nil {
+		return fmt.Errorf("keyPair is nil")
+	}
+
+	var outputFormat string
+	if model.GlobalFlagModel != nil {
+		outputFormat = model.GlobalFlagModel.OutputFormat
+	}
+	switch outputFormat {
 	case print.JSONOutputFormat:
 		details, err := json.MarshalIndent(keyPair, "", "  ")
 		if err != nil {
@@ -137,7 +148,7 @@ func outputResult(p *print.Printer, model *inputModel, keyPair *iaas.Keypair) er
 		}
 		p.Outputln(string(details))
 	default:
-		p.Outputf("Updated labels of key pair %q\n", *model.KeyPairName)
+		p.Outputf("Updated labels of key pair %q\n", utils.PtrString(model.KeyPairName))
 	}
 	return nil
 }

internal/cmd/beta/key-pair/update/update_test.go

@@ -188,3 +188,38 @@ func TestBuildRequest(t *testing.T) {
 		})
 	}
 }
+
+func Test_outputResult(t *testing.T) {
+	type args struct {
+		model   *inputModel
+		keyPair *iaas.Keypair
+	}
+	tests := []struct {
+		name    string
+		args    args
+		wantErr bool
+	}{
+		{
+			name:    "empty",
+			args:    args{},
+			wantErr: true,
+		},
+		{
+			name: "base",
+			args: args{
+				model:   &inputModel{},
+				keyPair: &iaas.Keypair{},
+			},
+			wantErr: false,
+		},
+	}
+	p := print.NewPrinter()
+	p.Cmd = NewCmd(p)
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			if err := outputResult(p, tt.args.model, tt.args.keyPair); (err != nil) != tt.wantErr {
+				t.Errorf("outputResult() error = %v, wantErr %v", err, tt.wantErr)
+			}
+		})
+	}
+}

internal/cmd/beta/network-area/create/create.go

@@ -197,7 +197,7 @@ func outputResult(p *print.Printer, model *inputModel, orgLabel string, networkA
 
 		return nil
 	default:
-		p.Outputf("Created STACKIT Network Area for organization %q.\nNetwork area ID: %s\n", orgLabel, *networkArea.AreaId)
+		p.Outputf("Created STACKIT Network Area for organization %q.\nNetwork area ID: %s\n", orgLabel, utils.PtrString(networkArea.AreaId))
 		return nil
 	}
 }