Skip to content

Commit d677796

Browse files
Update common_client_test.go
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
1 parent 526d130 commit d677796

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

common_client_test.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,11 @@ func TestWithLogger(t *testing.T) {
190190
client, err := opts.newRetryableHTTPClient()
191191
require.NoError(t, err)
192192
assert.NotNil(t, client.Logger, "retryable client should have logger set from WithLogger(nil)")
193-
assert.Equal(t, slog.DiscardHandler, opts.logger.Handler(), "retryable client logger should be a discard logger from WithLogger(nil)")
193+
194+
logger, ok := client.Logger.(*slog.Logger)
195+
require.True(t, ok, "retryable client logger should be a *slog.Logger")
196+
assert.Same(t, opts.logger, logger, "retryable client logger should be the same logger set by WithLogger(nil)")
197+
assert.Equal(t, slog.DiscardHandler, logger.Handler(), "retryable client logger should be a discard logger from WithLogger(nil)")
194198
})
195199

196200
t.Run("WithLogger(customLogger) propagates custom logger to retryable HTTP client", func(t *testing.T) {

0 commit comments

Comments
 (0)