Skip to content

Commit d5b7aec

Browse files
committed
fix: code review fixes
1 parent 9916303 commit d5b7aec

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

internal/tools/semconvkit/templates/error_type_test.go.tmpl

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,17 +33,17 @@ func TestErrorType(t *testing.T) {
3333
wantStr string
3434
}{
3535
{
36-
name: "BuiltinError",
36+
name: "builtin error",
3737
err: builtinErr,
3838
want: attribute.String("error.type", "*errors.errorString"),
3939
},
4040
{
41-
name: "CustomError",
41+
name: "custom error",
4242
err: customErr,
4343
wantStr: wantCustomStr,
4444
},
4545
{
46-
name: "NilError",
46+
name: "nil error",
4747
err: nilErr,
4848
want: ErrorTypeOther,
4949
},
@@ -52,13 +52,13 @@ func TestErrorType(t *testing.T) {
5252
for _, tt := range tests {
5353
t.Run(tt.name, func(t *testing.T) {
5454
got := ErrorType(tt.err)
55-
if tt.name == "CustomError" {
55+
if tt.name == "custom error" {
5656
if got.Value.AsString() != tt.wantStr {
57-
t.Errorf("Want %s, got %s", tt.wantStr, got.Value.AsString())
57+
t.Errorf("ErrorType(%v) = %v, want %v", tt.err, got.Value.AsString(), tt.wantStr)
5858
}
5959
} else {
6060
if got != tt.want {
61-
t.Errorf("Want %v, got %v", tt.want, got)
61+
t.Errorf("ErrorType(%v) = %v, want %v", tt.err, got, tt.want)
6262
}
6363
}
6464
})

semconv/v1.34.0/error_type_test.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,17 +33,17 @@ func TestErrorType(t *testing.T) {
3333
wantStr string
3434
}{
3535
{
36-
name: "BuiltinError",
36+
name: "builtin error",
3737
err: builtinErr,
3838
want: attribute.String("error.type", "*errors.errorString"),
3939
},
4040
{
41-
name: "CustomError",
41+
name: "custom error",
4242
err: customErr,
4343
want: attribute.String("error.type", wantCustomStr),
4444
},
4545
{
46-
name: "NilError",
46+
name: "nil error",
4747
err: nilErr,
4848
want: ErrorTypeOther,
4949
},
@@ -52,13 +52,13 @@ func TestErrorType(t *testing.T) {
5252
for _, tt := range tests {
5353
t.Run(tt.name, func(t *testing.T) {
5454
got := ErrorType(tt.err)
55-
if tt.name == "CustomError" {
55+
if tt.name == "custom error" {
5656
if got.Value.AsString() != tt.wantStr {
57-
t.Errorf("Want %s, got %s", tt.wantStr, got.Value.AsString())
57+
t.Errorf("ErrorType(%v) = %v, want %v", tt.err, got.Value.AsString(), tt.wantStr)
5858
}
5959
} else {
6060
if got != tt.want {
61-
t.Errorf("Want %v, got %v", tt.want, got)
61+
t.Errorf("ErrorType(%v) = %v, want %v", tt.err, got, tt.want)
6262
}
6363
}
6464
})

0 commit comments

Comments
 (0)