Fix unnecessary typecasting on bytes.Buffer

withshubh · withshubh · commit 27af642656ca · 2021-02-25T14:01:48.000+05:30
diff --git a/middleware/logger_test.go b/middleware/logger_test.go
@@ -164,7 +164,7 @@ func TestLoggerCustomTimestamp(t *testing.T) {
 	e.ServeHTTP(rec, req)
 
 	var objs map[string]*json.RawMessage
-	if err := json.Unmarshal([]byte(buf.String()), &objs); err != nil {
+	if err := json.Unmarshal(buf.Bytes(), &objs); err != nil {
 		panic(err)
 	}
 	loggedTime := *(*string)(unsafe.Pointer(objs["time"]))

Original file line number	Diff line number	Diff line change
`@@ -164,7 +164,7 @@ func TestLoggerCustomTimestamp(t *testing.T) {`
`164`	`164`	`e.ServeHTTP(rec, req)`
`165`	`165`
`166`	`166`	`var objs map[string]*json.RawMessage`
`167`		`- if err := json.Unmarshal([]byte(buf.String()), &objs); err != nil {`
	`167`	`+ if err := json.Unmarshal(buf.Bytes(), &objs); err != nil {`
`168`	`168`	`panic(err)`
`169`	`169`	`}`
`170`	`170`	`loggedTime := (string)(unsafe.Pointer(objs["time"]))`