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/contrib/sharedhost/replacement_test.go b/contrib/sharedhost/replacement_test.go
index cab80bb..67c3754 100644
--- a/contrib/sharedhost/replacement_test.go
+++ b/contrib/sharedhost/replacement_test.go
@@ -43,12 +43,12 @@ func TestReplaceTilde(t *testing.T) {
 
 			replacer := sharedhost.ReplaceTilde(test.replacement)
 			request := &gus.Request{URL: u}
-			handler := replacer(func(_ context.Context, request *gus.Request) *gus.Response {
+			handler := replacer(gus.HandlerFunc(func(_ context.Context, request *gus.Request) *gus.Response {
 				assert.Equal(t, test.replacedPath, request.Path)
 				return &gus.Response{}
-			})
+			}))
 
-			handler(context.Background(), request)
+			handler.Handle(context.Background(), request)
 
 			// original request was unmodified
 			assert.Equal(t, originalPath, request.Path)