From 46ad450327111b9d28b592658d75ef57da498298 Mon Sep 17 00:00:00 2001 From: tjpcc Date: Wed, 15 Feb 2023 16:44:29 -0700 Subject: 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. --- contrib/sharedhost/replacement_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'contrib/sharedhost/replacement_test.go') 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) -- cgit v1.2.3