mirror of https://github.com/go-redis/redis.git
chore: fix test
This commit is contained in:
parent
0798cccba5
commit
515043aaab
|
@ -4,7 +4,6 @@ import (
|
||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
. "github.com/bsm/ginkgo/v2"
|
. "github.com/bsm/ginkgo/v2"
|
||||||
. "github.com/bsm/ginkgo/v2/extensions/table"
|
|
||||||
. "github.com/bsm/gomega"
|
. "github.com/bsm/gomega"
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
|
@ -34,32 +34,6 @@ func TestNewWithTracerProvider(t *testing.T) {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestNewWithAttributes(t *testing.T) {
|
|
||||||
provider := sdktrace.NewTracerProvider()
|
|
||||||
hook := newTracingHook("", WithTracerProvider(provider), WithAttributes(semconv.NetPeerNameKey.String("localhost")))
|
|
||||||
ctx, span := provider.Tracer("redis-test").Start(context.TODO(), "redis-test")
|
|
||||||
cmd := redis.NewCmd(ctx, "ping")
|
|
||||||
defer span.End()
|
|
||||||
|
|
||||||
processHook := hook.ProcessHook(func(ctx context.Context, cmd redis.Cmder) error {
|
|
||||||
attrs := trace.SpanFromContext(ctx).(sdktrace.ReadOnlySpan).Attributes()
|
|
||||||
if !(attrs[0] == semconv.DBSystemRedis) {
|
|
||||||
t.Fatalf("expected attrs[0] to be semconv.DBSystemRedis, got: %v", attrs[0])
|
|
||||||
}
|
|
||||||
if !(attrs[1] == semconv.NetPeerNameKey.String("localhost")) {
|
|
||||||
t.Fatalf("expected attrs[1] to be semconv.NetPeerNameKey.String(\"localhost\"), got: %v", attrs[1])
|
|
||||||
}
|
|
||||||
if !(attrs[2] == semconv.DBStatementKey.String("ping")) {
|
|
||||||
t.Fatalf("expected attrs[2] to be semconv.DBStatementKey.String(\"ping\"), got: %v", attrs[2])
|
|
||||||
}
|
|
||||||
return nil
|
|
||||||
})
|
|
||||||
err := processHook(ctx, cmd)
|
|
||||||
if err != nil {
|
|
||||||
t.Fatal(err)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
func TestWithDBStatement(t *testing.T) {
|
func TestWithDBStatement(t *testing.T) {
|
||||||
provider := sdktrace.NewTracerProvider()
|
provider := sdktrace.NewTracerProvider()
|
||||||
hook := newTracingHook(
|
hook := newTracingHook(
|
||||||
|
|
Loading…
Reference in New Issue