Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix unhandled errors and remove unused parameter #2061

Merged
merged 12 commits into from Aug 30, 2022
Merged

fix unhandled errors and remove unused parameter #2061

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
fe7f105
fix unhandled errors
Kamandlou Aug 26, 2022
7f1028d
fix unhandled error in cache package test
Kamandlou Aug 26, 2022
bf387fa
omit variable type
Kamandlou Aug 26, 2022
d09213b
omit variable type
Kamandlou Aug 26, 2022
14b6646
rename variable because collide with the imported package name
Kamandlou Aug 26, 2022
1c45f7e
handle file error on closing
Kamandlou Aug 26, 2022
a021d0c
Merge branch 'gofiber:master' into master
Kamandlou Aug 26, 2022
89bf878
fix unhandled in common_linux.go
Kamandlou Aug 28, 2022
8dded84
Merge branch 'gofiber:master' into master
Kamandlou Aug 29, 2022
08a9256
fix unhandled errors in helpers_test.go
Kamandlou Aug 29, 2022
038315d
fix unhandled errors in listen_test.go
Kamandlou Aug 30, 2022
e5b8dad
remove unused parameter in emptyHandler method
Kamandlou Aug 30, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
15 changes: 10 additions & 5 deletions helpers_test.go
View file
Open in desktop
Expand Up @@ -22,7 +22,8 @@ func Test_Utils_ETag(t *testing.T) {
t.Run("Not Status OK", func(t *testing.T) {
c := app.AcquireCtx(&fasthttp.RequestCtx{})
defer app.ReleaseCtx(c)
c.SendString("Hello, World!")
err := c.SendString("Hello, World!")
utils.AssertEqual(t, nil, err)
c.Status(201)
setETag(c, false)
utils.AssertEqual(t, "", string(c.Response().Header.Peek(HeaderETag)))
Expand All @@ -38,7 +39,8 @@ func Test_Utils_ETag(t *testing.T) {
t.Run("Has HeaderIfNoneMatch", func(t *testing.T) {
c := app.AcquireCtx(&fasthttp.RequestCtx{})
defer app.ReleaseCtx(c)
c.SendString("Hello, World!")
err := c.SendString("Hello, World!")
utils.AssertEqual(t, nil, err)
c.Request().Header.Set(HeaderIfNoneMatch, `"13-1831710635"`)
setETag(c, false)
utils.AssertEqual(t, 304, c.Response().StatusCode())
Expand All @@ -49,7 +51,8 @@ func Test_Utils_ETag(t *testing.T) {
t.Run("No HeaderIfNoneMatch", func(t *testing.T) {
c := app.AcquireCtx(&fasthttp.RequestCtx{})
defer app.ReleaseCtx(c)
c.SendString("Hello, World!")
err := c.SendString("Hello, World!")
utils.AssertEqual(t, nil, err)
setETag(c, false)
utils.AssertEqual(t, `"13-1831710635"`, string(c.Response().Header.Peek(HeaderETag)))
})
Expand All @@ -60,7 +63,8 @@ func Benchmark_Utils_ETag(b *testing.B) {
app := New()
c := app.AcquireCtx(&fasthttp.RequestCtx{})
defer app.ReleaseCtx(c)
c.SendString("Hello, World!")
err := c.SendString("Hello, World!")
utils.AssertEqual(b, nil, err)
for n := 0; n < b.N; n++ {
setETag(c, false)
}
Expand All @@ -73,7 +77,8 @@ func Test_Utils_ETag_Weak(t *testing.T) {
t.Run("Set Weak", func(t *testing.T) {
c := app.AcquireCtx(&fasthttp.RequestCtx{})
defer app.ReleaseCtx(c)
c.SendString("Hello, World!")
err := c.SendString("Hello, World!")
utils.AssertEqual(t, nil, err)
setETag(c, true)
utils.AssertEqual(t, `W/"13-1831710635"`, string(c.Response().Header.Peek(HeaderETag)))
})
Expand Down
12 changes: 9 additions & 3 deletions listen_test.go
View file
Open in desktop
Expand Up @@ -166,12 +166,18 @@ func captureOutput(f func()) string {
go func() {
var buf bytes.Buffer
wg.Done()
io.Copy(&buf, reader)
_, err := io.Copy(&buf, reader)
if err != nil {
panic(err)
}
out <- buf.String()
}()
wg.Wait()
f()
writer.Close()
err = writer.Close()
if err != nil {
panic(err)
}
return <-out
}

Expand Down Expand Up @@ -244,6 +250,6 @@ func Test_App_print_Route_with_group(t *testing.T) {
utils.AssertEqual(t, true, strings.Contains(printRoutesMessage, "/v1/test/fiber/*"))
}

func emptyHandler(c *Ctx) error {
func emptyHandler(_ *Ctx) error {
return nil
}