From 5fa62aeefb8cdaa442641dd13876a154f74088d6 Mon Sep 17 00:00:00 2001 From: Arjan Bal Date: Tue, 27 Aug 2024 00:11:13 +0530 Subject: [PATCH] improve error messages --- picker_wrapper_test.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/picker_wrapper_test.go b/picker_wrapper_test.go index 8a7f8f5cd156..7cdd63f3caae 100644 --- a/picker_wrapper_test.go +++ b/picker_wrapper_test.go @@ -86,7 +86,7 @@ func (s) TestBlockingPick(t *testing.T) { for i := goroutineCount; i > 0; i-- { go func() { if tr, _, err := bp.pick(ctx, true, balancer.PickInfo{}); err != nil || tr != testT { - t.Errorf("bp.pick returned non-nil error: %v", err) + t.Errorf("bp.pick returned transport: %v, error: %v, want transport: %v, error: nil", tr, err, testT) } atomic.AddUint64(&finishedCount, 1) wg.Done() @@ -113,7 +113,7 @@ func (s) TestBlockingPickNoSubAvailable(t *testing.T) { for i := goroutineCount; i > 0; i-- { go func() { if tr, _, err := bp.pick(ctx, true, balancer.PickInfo{}); err != nil || tr != testT { - t.Errorf("bp.pick returned non-nil error: %v", err) + t.Errorf("bp.pick returned transport: %v, error: %v, want transport: %v, error: nil", tr, err, testT) } atomic.AddUint64(&finishedCount, 1) wg.Done() @@ -141,7 +141,7 @@ func (s) TestBlockingPickTransientWaitforready(t *testing.T) { for i := goroutineCount; i > 0; i-- { go func() { if tr, _, err := bp.pick(ctx, false, balancer.PickInfo{}); err != nil || tr != testT { - t.Errorf("bp.pick returned non-nil error: %v", err) + t.Errorf("bp.pick returned transport: %v, error: %v, want transport: %v, error: nil", tr, err, testT) } atomic.AddUint64(&finishedCount, 1) wg.Done() @@ -168,7 +168,7 @@ func (s) TestBlockingPickSCNotReady(t *testing.T) { for i := goroutineCount; i > 0; i-- { go func() { if tr, _, err := bp.pick(ctx, true, balancer.PickInfo{}); err != nil || tr != testT { - t.Errorf("bp.pick returned non-nil error: %v", err) + t.Errorf("bp.pick returned transport: %v, error: %v, want transport: %v, error: nil", tr, err, testT) } atomic.AddUint64(&finishedCount, 1) wg.Done()