otelzap: Implement Uint methods on encoder

bridges/otelzap/encoder.go

@@ -74,8 +74,12 @@
 	m.kv = append(m.kv, log.String(k, v))
 }
 
-// TODO.
 func (m *objectEncoder) AddUint64(k string, v uint64) {
+	m.kv = append(m.kv,
+		log.KeyValue{
+			Key:   k,
+			Value: assignUintValue(v),
+		})
 }
 
 // TODO.
@@ -94,11 +98,34 @@
 func (m *objectEncoder) AddInt16(k string, v int16)     { m.AddInt64(k, int64(v)) }
 func (m *objectEncoder) AddInt8(k string, v int8)       { m.AddInt64(k, int64(v)) }
 
+func (m *objectEncoder) AddUint(k string, v uint) {
+	m.AddUint64(k, uint64(v))
+}
+
+func (m *objectEncoder) AddUint32(k string, v uint32) {
+	m.AddInt64(k, int64(v))
+}
+
+func (m *objectEncoder) AddUint16(k string, v uint16) {
+	m.AddInt64(k, int64(v))
+}
+
+func (m *objectEncoder) AddUint8(k string, v uint8) {
+	m.AddInt64(k, int64(v))
+}
+
+func (m *objectEncoder) AddUintptr(k string, v uintptr) {
+	m.AddUint64(k, uint64(v))
+}
+
 // TODO.
 func (m *objectEncoder) AddComplex64(k string, v complex64) {}
 func (m *objectEncoder) AddTime(k string, v time.Time)      {}
-func (m *objectEncoder) AddUint(k string, v uint)           {}
-func (m *objectEncoder) AddUint32(k string, v uint32)       {}
-func (m *objectEncoder) AddUint16(k string, v uint16)       {}
-func (m *objectEncoder) AddUint8(k string, v uint8)         {}
-func (m *objectEncoder) AddUintptr(k string, v uintptr)     {}
+
+func assignUintValue(v uint64) log.Value {
+	const maxInt64 = ^uint64(0) >> 1
+	if v > maxInt64 {
+		return log.Float64Value(float64(v))
+	}
+	return log.Int64Value(int64(v))
+}

bridges/otelzap/encoder_test.go

@@ -77,6 +77,36 @@ func TestObjectEncoder(t *testing.T) {
 			f:        func(e zapcore.ObjectEncoder) { e.AddString("k", "v") },
 			expected: "v",
 		},
+		{
+			desc:     "AddUint64",
+			f:        func(e zapcore.ObjectEncoder) { e.AddUint64("k", 42) },
+			expected: int64(42),
+		},
+		{
+			desc:     "AddUint64-Overflow",
+			f:        func(e zapcore.ObjectEncoder) { e.AddUint64("k", ^uint64(0)) },
+			expected: float64(^uint64(0)),
+		},
+		{
+			desc:     "AddUint32",
+			f:        func(e zapcore.ObjectEncoder) { e.AddUint32("k", 42) },
+			expected: int64(42),
+		},
+		{
+			desc:     "AddUint16",
+			f:        func(e zapcore.ObjectEncoder) { e.AddUint16("k", 42) },
+			expected: int64(42),
+		},
+		{
+			desc:     "AddUint8",
+			f:        func(e zapcore.ObjectEncoder) { e.AddUint8("k", 42) },
+			expected: int64(42),
+		},
+		{
+			desc:     "AddUintptr",
+			f:        func(e zapcore.ObjectEncoder) { e.AddUintptr("k", 42) },
+			expected: int64(42),
+		},
 	}
 
 	for _, tt := range tests {