Skip to content

Commit e4c3547

Browse files
committed
Add test cases for RedirectHandler methods
1 parent b66cd61 commit e4c3547

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

redirecthandler/redirecthandler_test.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@ import (
1010
"github.com/stretchr/testify/assert"
1111
)
1212

13+
// TestRedirectHandler_CheckRedirect tests the checkRedirect method of the RedirectHandler.
14+
// It covers various scenarios including redirect loop detection, maximum redirects limit,
15+
// resolving relative redirects, cross-domain security measures, and handling of 303 See Other response.
1316
func TestRedirectHandler_CheckRedirect(t *testing.T) {
1417
mockLogger := mocklogger.NewMockLogger()
1518

@@ -77,6 +80,8 @@ func TestRedirectHandler_CheckRedirect(t *testing.T) {
7780
})
7881
}
7982

83+
// TestRedirectHandler_ResolveRedirectURL tests the resolveRedirectURL method of the RedirectHandler.
84+
// It checks the correct resolution of absolute and relative URLs including those with query parameters and fragments.
8085
func TestRedirectHandler_ResolveRedirectURL(t *testing.T) {
8186
redirectHandler := RedirectHandler{}
8287

@@ -105,6 +110,8 @@ func TestRedirectHandler_ResolveRedirectURL(t *testing.T) {
105110
})
106111
}
107112

113+
// TestRedirectHandler_SecureRequest tests the secureRequest method of the RedirectHandler.
114+
// It verifies that sensitive headers are correctly removed when a request is redirected to a different domain.
108115
func TestRedirectHandler_SecureRequest(t *testing.T) {
109116
mockLogger := mocklogger.NewMockLogger()
110117
mockLogger.SetLevel(logger.LogLevelDebug)

0 commit comments

Comments
 (0)