fix(kms): do not reuse buffer during output tests

cgoetz-inovex · cgoetz-inovex · commit 31c58def9828 · 2025-11-21T18:12:20.000+01:00
diff --git a/internal/cmd/beta/kms/key/describe/describe_test.go b/internal/cmd/beta/kms/key/describe/describe_test.go
@@ -206,10 +206,10 @@ func TestOutputResult(t *testing.T) {
 	}
 	p := print.NewPrinter()
 	p.Cmd = NewCmd(&params.CmdParams{Printer: p})
-	var buf bytes.Buffer
-	p.Cmd.SetOut(&buf)
 	for _, tt := range tests {
 		t.Run(tt.description, func(t *testing.T) {
+			var buf bytes.Buffer
+			p.Cmd.SetOut(&buf)
 			if err := outputResult(p, tt.outputFmt, tt.keyRing); (err != nil) != tt.wantErr {
 				t.Errorf("outputResult() error = %v, wantErr %v", err, tt.wantErr)
 			}
diff --git a/internal/cmd/beta/kms/keyring/describe/describe_test.go b/internal/cmd/beta/kms/keyring/describe/describe_test.go
@@ -167,10 +167,10 @@ func TestOutputResult(t *testing.T) {
 	}
 	p := print.NewPrinter()
 	p.Cmd = NewCmd(&params.CmdParams{Printer: p})
-	var buf bytes.Buffer
-	p.Cmd.SetOut(&buf)
 	for _, tt := range tests {
 		t.Run(tt.description, func(t *testing.T) {
+			var buf bytes.Buffer
+			p.Cmd.SetOut(&buf)
 			if err := outputResult(p, tt.outputFmt, tt.keyRing); (err != nil) != tt.wantErr {
 				t.Errorf("outputResult() error = %v, wantErr %v", err, tt.wantErr)
 			}
diff --git a/internal/cmd/beta/kms/wrappingkey/describe/describe_test.go b/internal/cmd/beta/kms/wrappingkey/describe/describe_test.go
@@ -199,10 +199,10 @@ func TestOutputResult(t *testing.T) {
 	}
 	p := print.NewPrinter()
 	p.Cmd = NewCmd(&params.CmdParams{Printer: p})
-	var buf bytes.Buffer
-	p.Cmd.SetOut(&buf)
 	for _, tt := range tests {
 		t.Run(tt.description, func(t *testing.T) {
+			var buf bytes.Buffer
+			p.Cmd.SetOut(&buf)
 			if err := outputResult(p, tt.outputFmt, tt.keyRing); (err != nil) != tt.wantErr {
 				t.Errorf("outputResult() error = %v, wantErr %v", err, tt.wantErr)
 			}

Original file line number	Diff line number	Diff line change
`@@ -206,10 +206,10 @@ func TestOutputResult(t *testing.T) {`
`206`	`206`	`}`
`207`	`207`	`p := print.NewPrinter()`
`208`	`208`	`p.Cmd = NewCmd(&params.CmdParams{Printer: p})`
`209`		`- var buf bytes.Buffer`
`210`		`- p.Cmd.SetOut(&buf)`
`211`	`209`	`for _, tt := range tests {`
`212`	`210`	`t.Run(tt.description, func(t *testing.T) {`
	`211`	`+ var buf bytes.Buffer`
	`212`	`+ p.Cmd.SetOut(&buf)`
`213`	`213`	`if err := outputResult(p, tt.outputFmt, tt.keyRing); (err != nil) != tt.wantErr {`
`214`	`214`	`t.Errorf("outputResult() error = %v, wantErr %v", err, tt.wantErr)`
`215`	`215`	`}`
Original file line number	Diff line number	Diff line change
`@@ -167,10 +167,10 @@ func TestOutputResult(t *testing.T) {`
`167`	`167`	`}`
`168`	`168`	`p := print.NewPrinter()`
`169`	`169`	`p.Cmd = NewCmd(&params.CmdParams{Printer: p})`
`170`		`- var buf bytes.Buffer`
`171`		`- p.Cmd.SetOut(&buf)`
`172`	`170`	`for _, tt := range tests {`
`173`	`171`	`t.Run(tt.description, func(t *testing.T) {`
	`172`	`+ var buf bytes.Buffer`
	`173`	`+ p.Cmd.SetOut(&buf)`
`174`	`174`	`if err := outputResult(p, tt.outputFmt, tt.keyRing); (err != nil) != tt.wantErr {`
`175`	`175`	`t.Errorf("outputResult() error = %v, wantErr %v", err, tt.wantErr)`
`176`	`176`	`}`
Original file line number	Diff line number	Diff line change
`@@ -199,10 +199,10 @@ func TestOutputResult(t *testing.T) {`
`199`	`199`	`}`
`200`	`200`	`p := print.NewPrinter()`
`201`	`201`	`p.Cmd = NewCmd(&params.CmdParams{Printer: p})`
`202`		`- var buf bytes.Buffer`
`203`		`- p.Cmd.SetOut(&buf)`
`204`	`202`	`for _, tt := range tests {`
`205`	`203`	`t.Run(tt.description, func(t *testing.T) {`
	`204`	`+ var buf bytes.Buffer`
	`205`	`+ p.Cmd.SetOut(&buf)`
`206`	`206`	`if err := outputResult(p, tt.outputFmt, tt.keyRing); (err != nil) != tt.wantErr {`
`207`	`207`	`t.Errorf("outputResult() error = %v, wantErr %v", err, tt.wantErr)`
`208`	`208`	`}`