mirror of
https://github.com/gofiber/fiber.git
synced 2025-05-11 18:26:34 +00:00
* 🔥 Feature: Add thread-safe reading from a closed testConn * 🔥 Feature: Add TestConfig to app.Test() This commit is summarized as: - Add the struct `TestConfig` as a parameter for `app.Test()` instead of `timeout` - Add documentation of `TestConfig` to docs/api/app.md and in-line - Modify middleware to use `TestConfig` instead of the previous implementation Fixes #3149 * 📚 Doc: Add more details about TestConfig in docs * 🩹 Fix: Correct testConn tests - Fixes Test_Utils_TestConn_Closed_Write - Fixes missing regular write test * 🎨 Style: Respect linter in Add App Test Config * 🎨 Styles: Update app.go to respect linter * ♻️ Refactor: Rename TestConfig's ErrOnTimeout to FailOnTimeout - Rename TestConfig.ErrOnTimeout to TestConfig.FailOnTimeout - Update documentation to use changed name - Also fix stale documentation about passing Timeout as a single argument * 🩹 Fix: Fix typo in TestConfig struct comment in app.go * ♻️ Refactor: Change app.Test() fail on timeouterror to os.ErrDeadlineExceeded * ♻️ Refactor:Update middleware that use the same TestConfig to use a global variable * 🩹 Fix: Update error from FailOnTimeout to os.ErrDeadlineExceeded in tests * 🩹 Fix: Remove errors import from middlware/proxy/proxy_test.go * 📚 Doc: Add `app.Test()` config changes to docs/whats_new.md * ♻ Refactor: Change app.Test() and all uses to accept 0 as no timeout instead of -1 * 📚 Doc: Add TestConfig option details to docs/whats_new.md * 🎨 Styles: Update docs/whats_new.md to respect markdown-lint * 🎨 Styles: Update docs/whats_new.md to use consistent style for TestConfig options description --------- Co-authored-by: Juan Calderon-Perez <835733+gaby@users.noreply.github.com>
205 lines
4.7 KiB
Go
205 lines
4.7 KiB
Go
package pprof
|
|
|
|
import (
|
|
"bytes"
|
|
"io"
|
|
"net/http/httptest"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/gofiber/fiber/v3"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
var testConfig = fiber.TestConfig{
|
|
Timeout: 5 * time.Second,
|
|
FailOnTimeout: true,
|
|
}
|
|
|
|
func Test_Non_Pprof_Path(t *testing.T) {
|
|
app := fiber.New()
|
|
|
|
app.Use(New())
|
|
|
|
app.Get("/", func(c fiber.Ctx) error {
|
|
return c.SendString("escaped")
|
|
})
|
|
|
|
resp, err := app.Test(httptest.NewRequest(fiber.MethodGet, "/", nil))
|
|
require.NoError(t, err)
|
|
require.Equal(t, 200, resp.StatusCode)
|
|
|
|
b, err := io.ReadAll(resp.Body)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "escaped", string(b))
|
|
}
|
|
|
|
func Test_Non_Pprof_Path_WithPrefix(t *testing.T) {
|
|
app := fiber.New()
|
|
|
|
app.Use(New(Config{Prefix: "/federated-fiber"}))
|
|
|
|
app.Get("/", func(c fiber.Ctx) error {
|
|
return c.SendString("escaped")
|
|
})
|
|
|
|
resp, err := app.Test(httptest.NewRequest(fiber.MethodGet, "/", nil))
|
|
require.NoError(t, err)
|
|
require.Equal(t, 200, resp.StatusCode)
|
|
|
|
b, err := io.ReadAll(resp.Body)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "escaped", string(b))
|
|
}
|
|
|
|
func Test_Pprof_Index(t *testing.T) {
|
|
app := fiber.New()
|
|
|
|
app.Use(New())
|
|
|
|
app.Get("/", func(c fiber.Ctx) error {
|
|
return c.SendString("escaped")
|
|
})
|
|
|
|
resp, err := app.Test(httptest.NewRequest(fiber.MethodGet, "/debug/pprof/", nil))
|
|
require.NoError(t, err)
|
|
require.Equal(t, 200, resp.StatusCode)
|
|
require.Equal(t, fiber.MIMETextHTMLCharsetUTF8, resp.Header.Get(fiber.HeaderContentType))
|
|
|
|
b, err := io.ReadAll(resp.Body)
|
|
require.NoError(t, err)
|
|
require.True(t, bytes.Contains(b, []byte("<title>/debug/pprof/</title>")))
|
|
}
|
|
|
|
func Test_Pprof_Index_WithPrefix(t *testing.T) {
|
|
app := fiber.New()
|
|
|
|
app.Use(New(Config{Prefix: "/federated-fiber"}))
|
|
|
|
app.Get("/", func(c fiber.Ctx) error {
|
|
return c.SendString("escaped")
|
|
})
|
|
|
|
resp, err := app.Test(httptest.NewRequest(fiber.MethodGet, "/federated-fiber/debug/pprof/", nil))
|
|
require.NoError(t, err)
|
|
require.Equal(t, 200, resp.StatusCode)
|
|
require.Equal(t, fiber.MIMETextHTMLCharsetUTF8, resp.Header.Get(fiber.HeaderContentType))
|
|
|
|
b, err := io.ReadAll(resp.Body)
|
|
require.NoError(t, err)
|
|
require.Contains(t, string(b), "<title>/debug/pprof/</title>")
|
|
}
|
|
|
|
func Test_Pprof_Subs(t *testing.T) {
|
|
app := fiber.New()
|
|
|
|
app.Use(New())
|
|
|
|
app.Get("/", func(c fiber.Ctx) error {
|
|
return c.SendString("escaped")
|
|
})
|
|
|
|
subs := []string{
|
|
"cmdline", "profile", "symbol", "trace", "allocs", "block",
|
|
"goroutine", "heap", "mutex", "threadcreate",
|
|
}
|
|
|
|
for _, sub := range subs {
|
|
t.Run(sub, func(t *testing.T) {
|
|
target := "/debug/pprof/" + sub
|
|
if sub == "profile" {
|
|
target += "?seconds=1"
|
|
}
|
|
resp, err := app.Test(httptest.NewRequest(fiber.MethodGet, target, nil), testConfig)
|
|
require.NoError(t, err)
|
|
require.Equal(t, 200, resp.StatusCode)
|
|
})
|
|
}
|
|
}
|
|
|
|
func Test_Pprof_Subs_WithPrefix(t *testing.T) {
|
|
app := fiber.New()
|
|
|
|
app.Use(New(Config{Prefix: "/federated-fiber"}))
|
|
|
|
app.Get("/", func(c fiber.Ctx) error {
|
|
return c.SendString("escaped")
|
|
})
|
|
|
|
subs := []string{
|
|
"cmdline", "profile", "symbol", "trace", "allocs", "block",
|
|
"goroutine", "heap", "mutex", "threadcreate",
|
|
}
|
|
|
|
for _, sub := range subs {
|
|
t.Run(sub, func(t *testing.T) {
|
|
target := "/federated-fiber/debug/pprof/" + sub
|
|
if sub == "profile" {
|
|
target += "?seconds=1"
|
|
}
|
|
resp, err := app.Test(httptest.NewRequest(fiber.MethodGet, target, nil), testConfig)
|
|
require.NoError(t, err)
|
|
require.Equal(t, 200, resp.StatusCode)
|
|
})
|
|
}
|
|
}
|
|
|
|
func Test_Pprof_Other(t *testing.T) {
|
|
app := fiber.New()
|
|
|
|
app.Use(New())
|
|
|
|
app.Get("/", func(c fiber.Ctx) error {
|
|
return c.SendString("escaped")
|
|
})
|
|
|
|
resp, err := app.Test(httptest.NewRequest(fiber.MethodGet, "/debug/pprof/302", nil))
|
|
require.NoError(t, err)
|
|
require.Equal(t, 302, resp.StatusCode)
|
|
}
|
|
|
|
func Test_Pprof_Other_WithPrefix(t *testing.T) {
|
|
app := fiber.New()
|
|
|
|
app.Use(New(Config{Prefix: "/federated-fiber"}))
|
|
|
|
app.Get("/", func(c fiber.Ctx) error {
|
|
return c.SendString("escaped")
|
|
})
|
|
|
|
resp, err := app.Test(httptest.NewRequest(fiber.MethodGet, "/federated-fiber/debug/pprof/302", nil))
|
|
require.NoError(t, err)
|
|
require.Equal(t, 302, resp.StatusCode)
|
|
}
|
|
|
|
// go test -run Test_Pprof_Next
|
|
func Test_Pprof_Next(t *testing.T) {
|
|
app := fiber.New()
|
|
|
|
app.Use(New(Config{
|
|
Next: func(_ fiber.Ctx) bool {
|
|
return true
|
|
},
|
|
}))
|
|
|
|
resp, err := app.Test(httptest.NewRequest(fiber.MethodGet, "/debug/pprof/", nil))
|
|
require.NoError(t, err)
|
|
require.Equal(t, 404, resp.StatusCode)
|
|
}
|
|
|
|
// go test -run Test_Pprof_Next_WithPrefix
|
|
func Test_Pprof_Next_WithPrefix(t *testing.T) {
|
|
app := fiber.New()
|
|
|
|
app.Use(New(Config{
|
|
Next: func(_ fiber.Ctx) bool {
|
|
return true
|
|
},
|
|
Prefix: "/federated-fiber",
|
|
}))
|
|
|
|
resp, err := app.Test(httptest.NewRequest(fiber.MethodGet, "/federated-fiber/debug/pprof/", nil))
|
|
require.NoError(t, err)
|
|
require.Equal(t, 404, resp.StatusCode)
|
|
}
|