diff options
| author | tjpcc <tjp@ctrl-c.club> | 2023-02-15 16:44:29 -0700 |
|---|---|---|
| committer | tjpcc <tjp@ctrl-c.club> | 2023-02-15 16:44:29 -0700 |
| commit | 46ad450327111b9d28b592658d75ef57da498298 (patch) | |
| tree | 2b837bac9ae36d5a34dda06ba745850da216257d /contrib/fs/dir_test.go | |
| parent | bc96af40db6104580c22086c8db7c8119a404257 (diff) | |
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.
Diffstat (limited to 'contrib/fs/dir_test.go')
| -rw-r--r-- | contrib/fs/dir_test.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/contrib/fs/dir_test.go b/contrib/fs/dir_test.go index 6109a3c..9c6770d 100644 --- a/contrib/fs/dir_test.go +++ b/contrib/fs/dir_test.go @@ -47,7 +47,7 @@ func TestDirectoryDefault(t *testing.T) { require.Nil(t, err) request := &gus.Request{URL: u} - response := handler(context.Background(), request) + response := handler.Handle(context.Background(), request) if response == nil { assert.Equal(t, test.status, gemini.StatusNotFound) @@ -108,7 +108,7 @@ func TestDirectoryListing(t *testing.T) { require.Nil(t, err) request := &gus.Request{URL: u} - response := handler(context.Background(), request) + response := handler.Handle(context.Background(), request) if response == nil { assert.Equal(t, test.status, gemini.StatusNotFound) |