1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
|
package logging
import (
"fmt"
"io"
"net/http"
"net/http/httptest"
"testing"
"time"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
)
type errRountripper struct{}
func (errRountripper) RoundTrip(*http.Request) (*http.Response, error) {
return nil, io.ErrClosedPipe
}
func Test_EnableHTTPClient(t *testing.T) {
tests := []struct {
name string
transport http.RoundTripper
fromCtx bool
wantErr error
wantLog string
}{
{
name: "nil transport / default",
transport: nil,
wantLog: `{
"level":"INFO",
"msg":"request roundtrip",
"time":"not",
"request":{"method":"GET","url":"%s"},
"duration":1000000000,
"response":{
"status":"200 OK",
"content_length":14
}
}`,
},
{
name: "transport set",
transport: http.DefaultTransport,
wantLog: `{
"level":"INFO",
"msg":"request roundtrip",
"time":"not",
"request":{"method":"GET","url":"%s"},
"duration":1000000000,
"response":{
"status":"200 OK",
"content_length":14
}
}`,
},
{
name: "roundtrip error",
transport: errRountripper{},
wantErr: io.ErrClosedPipe,
wantLog: `{
"level":"ERROR",
"msg":"request roundtrip",
"time":"not",
"request":{"method":"GET","url":"%s"},
"error":"io: read/write on closed pipe",
"duration":1000000000
}`,
},
{
name: "logger from ctx",
transport: http.DefaultTransport,
fromCtx: true,
wantLog: `{
"level":"INFO",
"msg":"request roundtrip",
"time":"not",
"ctx":{
"request":{"method":"GET","url":"%s"},
"duration":1000000000,
"response":{
"status":"200 OK",
"content_length":14
}
}
}`,
},
}
for _, tt := range tests {
t.Run(tt.name, func(t *testing.T) {
out, logger := newTestLogger()
c := &http.Client{
Transport: tt.transport,
}
EnableHTTPClient(c,
WithFallbackLogger(logger),
WithClientDurationFunc(func(t time.Time) time.Duration {
return time.Second
}),
WithClientRequestAttr(requestToAttr),
WithClientResponseAttr(responseToAttr),
)
ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
fmt.Fprintln(w, "Hello, client")
}))
defer ts.Close()
req, err := http.NewRequest(http.MethodGet, ts.URL, nil)
require.NoError(t, err)
if tt.fromCtx {
req = req.WithContext(ToContext(req.Context(), logger.WithGroup("ctx")))
}
_, err = c.Do(req)
require.ErrorIs(t, err, tt.wantErr)
wantLog := fmt.Sprintf(tt.wantLog, ts.URL)
assert.JSONEq(t, wantLog, out.String())
})
}
}
|
