Skip to content

Commit cf513d0

Browse files
committed
Clarity: pod:snap1 style instead of pod "snap1"
1 parent a708654 commit cf513d0

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

internal/output/plain_format.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ func formatResourceSummary(e ResourceSummaryEvent) string {
200200

201201
func formatPodSnapshotSaved(e PodSnapshotSavedEvent) string {
202202
var sb strings.Builder
203-
sb.WriteString(SuccessMarker() + fmt.Sprintf(" Snapshot saved to pod %q", e.PodName))
203+
sb.WriteString(SuccessMarker() + fmt.Sprintf(" Snapshot saved to pod:%s", e.PodName))
204204
if e.Version > 0 {
205205
sb.WriteString(fmt.Sprintf("\n• Version: %d", e.Version))
206206
}

internal/output/plain_format_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -188,7 +188,7 @@ func TestFormatEventLine(t *testing.T) {
188188
Services: []string{"dynamodb", "s3", "sqs"},
189189
Size: 2621440,
190190
},
191-
want: SuccessMarker() + " Snapshot saved to pod \"my-baseline\"\n• Version: 3\n• Services: dynamodb, s3, sqs\n• Size: 2.5 MB",
191+
want: SuccessMarker() + " Snapshot saved to pod:my-baseline\n• Version: 3\n• Services: dynamodb, s3, sqs\n• Size: 2.5 MB",
192192
wantOK: true,
193193
},
194194
{
@@ -199,15 +199,15 @@ func TestFormatEventLine(t *testing.T) {
199199
Services: []string{"s3", "sqs", "sns", "dynamodb", "lambda", "apigateway", "iam", "sts", "ec2", "rds", "kinesis", "firehose", "cloudwatch", "cloudformation", "route53"},
200200
Size: 10485760,
201201
},
202-
want: SuccessMarker() + " Snapshot saved to pod \"big-pod\"\n• Version: 1\n• Services: s3, sqs, sns, dynamodb, lambda, apigateway, iam, sts, ec2, rds, kinesis, firehose, cloudwatch, cloudformation, route53\n• Size: 10.0 MB",
202+
want: SuccessMarker() + " Snapshot saved to pod:big-pod\n• Version: 1\n• Services: s3, sqs, sns, dynamodb, lambda, apigateway, iam, sts, ec2, rds, kinesis, firehose, cloudwatch, cloudformation, route53\n• Size: 10.0 MB",
203203
wantOK: true,
204204
},
205205
{
206206
name: "pod snapshot saved omits zero fields",
207207
event: PodSnapshotSavedEvent{
208208
PodName: "minimal-pod",
209209
},
210-
want: SuccessMarker() + " Snapshot saved to pod \"minimal-pod\"",
210+
want: SuccessMarker() + " Snapshot saved to pod:minimal-pod",
211211
wantOK: true,
212212
},
213213
}

0 commit comments

Comments
 (0)