work: Handle too many redirects

This was implicitly handled before, but now we define a specific error
type; as well as verify the concrete error type.

Author: stroiman

html/window.go

@@ -16,6 +16,8 @@ import (
 	"github.com/gost-dom/browser/url"
 )
 
+var ErrTooManyRedirects = errors.New("Too many redirects")
+
 type ScriptHost interface {
 	NewContext(window Window) ScriptContext
 	Close()
@@ -110,7 +112,7 @@ func newWindow(windowOptions ...WindowOption) *window {
 
 func (w *window) checkRedirect(req *http.Request, via []*http.Request) error {
 	if len(via) > 9 {
-		return errors.New("Too many redirects")
+		return ErrTooManyRedirects
 	}
 	w.history.ReplaceState(EMPTY_STATE, req.URL.String())
 	return nil

html/window_test.go

@@ -10,6 +10,7 @@ import (
 	"github.com/stretchr/testify/suite"
 
 	"github.com/gost-dom/browser/dom"
+	"github.com/gost-dom/browser/html"
 	. "github.com/gost-dom/browser/html"
 	. "github.com/gost-dom/browser/internal/testing/gomega-matchers"
 	"github.com/gost-dom/browser/internal/testing/gosttest"
@@ -72,6 +73,10 @@ func (s *WindowNavigationTestSuite) SetupTest() {
 		)
 		w.Write([]byte(respBody))
 	})
+	m.HandleFunc("/infinite-redirects", func(w http.ResponseWriter, r *http.Request) {
+		http.Redirect(w, r, "/infinite-redirects", 301)
+	})
+
 	s.win = htmltest.NewWindowHelper(s.T(), NewWindowFromHandler(m))
 	s.win.Navigate("https://example.com/page-1")
 }
@@ -85,3 +90,8 @@ func (s *WindowNavigationTestSuite) TestRedirectGetRequests() {
 	s.win.HTMLDocument().GetHTMLElementById("link").Click()
 	s.Assert().Equal("https://example.com/new-page-2", s.win.Location().Href())
 }
+
+func (s *WindowNavigationTestSuite) TestInfinteRedirects() {
+	err := s.win.Navigate("/infinite-redirects")
+	s.Assert().ErrorIs(err, html.ErrTooManyRedirects, "Error is too many redirects")
+}