From ee8d24126b108f168b166ca1be69baa4c2f52d5a Mon Sep 17 00:00:00 2001
From: Semyon Krestyaninov <ensiouel@gmail.com>
Date: Thu, 15 Feb 2024 10:41:52 +0300
Subject: [PATCH] fix typo
---
log/zap/field.go | 18 +++++++++---------
log/zap/field_test.go | 20 ++++++++++----------
2 files changed, 19 insertions(+), 19 deletions(-)
diff --git a/log/zap/field.go b/log/zap/field.go
index ba27acba..acc6932d 100644
--- a/log/zap/field.go
+++ b/log/zap/field.go
@@ -30,32 +30,32 @@ func Event(event string) zap.Field {
return zap.String("event", event)
}
-// ObjectID возвращает поле и устанавливает передаваемый аргумент в качестве идентификатора объекта в формате ObjectId.
+// Object возвращает поле и устанавливает передаваемый аргумент в качестве идентификатора объекта в формате ObjectId.
// Поддерживает типы в формате ObjectId: id.Descriptor, string, map[string]any, системные объекты.
-func ObjectID(v any) zap.Field {
+func Object(v any) zap.Field {
oid, err := id.NewObjectId(v)
if err != nil {
return zap.Skip()
}
- return zap.Reflect("object_id", oid)
+ return zap.Reflect("object", oid)
}
-// CallerID возвращает поле и устанавливает передаваемый аргумент в качестве "вызывающего" в формате ObjectId.
+// Caller возвращает поле и устанавливает передаваемый аргумент в качестве "вызывающего" в формате ObjectId.
// Поддерживает типы в формате ObjectId: id.Descriptor, string, map[string]any, системные объекты.
-func CallerID(v any) zap.Field {
+func Caller(v any) zap.Field {
oid, err := id.NewObjectId(v)
if err != nil {
return zap.Skip()
}
- return zap.Reflect("caller_id", oid)
+ return zap.Reflect("caller", oid)
}
-// CallerIDFromContext извлекает auth.Principal из контекста и устанавливает его в качестве "вызывающего" в формате ObjectID.
-func CallerIDFromContext(ctx context.Context) zap.Field {
+// CallerFromContext извлекает auth.Principal из контекста и устанавливает его в качестве "вызывающего" в формате Object.
+func CallerFromContext(ctx context.Context) zap.Field {
if ctx == nil {
return zap.Skip()
}
- return CallerID(auth.GetPrincipal(ctx))
+ return Caller(auth.GetPrincipal(ctx))
}
func Attr(attr any) zap.Field {
diff --git a/log/zap/field_test.go b/log/zap/field_test.go
index 40d6822b..643506ba 100644
--- a/log/zap/field_test.go
+++ b/log/zap/field_test.go
@@ -79,10 +79,10 @@ func TestObjectID(t *testing.T) {
field zap.Field
want zap.Field
}{
- {name: "system object", field: ObjectID(item), want: zap.Reflect("object_id", oid)},
- {name: "object id", field: ObjectID(itemId), want: zap.Reflect("object_id", oid)},
- {name: "string", field: ObjectID(oid.String()), want: zap.Reflect("object_id", oid)},
- {name: "invalid", field: ObjectID(nil), want: zap.Skip()},
+ {name: "system object", field: Object(item), want: zap.Reflect("object", oid)},
+ {name: "object id", field: Object(itemId), want: zap.Reflect("object", oid)},
+ {name: "string", field: Object(oid.String()), want: zap.Reflect("object", oid)},
+ {name: "invalid", field: Object(nil), want: zap.Skip()},
}
for _, tc := range tests {
@@ -109,10 +109,10 @@ func TestCallerID(t *testing.T) {
field zap.Field
want zap.Field
}{
- {name: "system object", field: CallerID(user), want: zap.Reflect("caller_id", oid)},
- {name: "object id", field: CallerID(userId), want: zap.Reflect("caller_id", oid)},
- {name: "string", field: CallerID(oid.String()), want: zap.Reflect("caller_id", oid)},
- {name: "invalid", field: CallerID(nil), want: zap.Skip()},
+ {name: "system object", field: Caller(user), want: zap.Reflect("caller", oid)},
+ {name: "object id", field: Caller(userId), want: zap.Reflect("caller", oid)},
+ {name: "string", field: Caller(oid.String()), want: zap.Reflect("caller", oid)},
+ {name: "invalid", field: Caller(nil), want: zap.Skip()},
}
for _, tc := range tests {
@@ -135,8 +135,8 @@ func TestCallerIDFromContext(t *testing.T) {
field zap.Field
want zap.Field
}{
- {name: "ok", field: CallerIDFromContext(ctx), want: zap.Reflect("caller_id", oid)},
- {name: "invalid", field: CallerIDFromContext(nil), want: zap.Skip()},
+ {name: "ok", field: CallerFromContext(ctx), want: zap.Reflect("caller", oid)},
+ {name: "invalid", field: CallerFromContext(nil), want: zap.Skip()},
}
for _, tc := range tests {
--
GitLab