Switch Handler to an interface.

HandlerFunc is much better as a function returning a Handler, rather
than a newtype for the function type itself. This way there is no
confusion creating a type-inferenced variable with HandlerFunc(func(...
and then using a HandlerFunc where a Handler is expected. Much better to
only have one public type.

1 files changed, 3 insertions, 3 deletions

diff --git a/logging/middleware_test.go b/logging/middleware_test.go
index 288c960..76406ef 100644
--- a/logging/middleware_test.go
+++ b/logging/middleware_test.go
@@ -14,11 +14,11 @@ import (
 
 func TestLogRequests(t *testing.T) {
 	logger := logRecorder{}
-	handler := logging.LogRequests(&logger)(func(_ context.Context, _ *gus.Request) *gus.Response {
+	handler := logging.LogRequests(&logger)(gus.HandlerFunc(func(_ context.Context, _ *gus.Request) *gus.Response {
 		return &gus.Response{}
-	})
+	}))
 
-	response := handler(context.Background(), &gus.Request{})
+	response := handler.Handle(context.Background(), &gus.Request{})
 	_, err := io.ReadAll(response.Body)
 	assert.Nil(t, err)