fix: tests regarding GET and the autoHeadInGet flag

Author: jean0t

echo.go

@@ -454,7 +454,7 @@ func (e *Echo) DELETE(path string, h HandlerFunc, m ...MiddlewareFunc) RouteInfo
 // to the same path.
 func (e *Echo) GET(path string, h HandlerFunc, m ...MiddlewareFunc) RouteInfo {
 	if e.autoHeadInGet {
-		_ = e.Add(http.MethodHead, path, h, m...)
+		_ = e.HEAD(path, h, m...)
 	}
 
 	return e.Add(http.MethodGet, path, h, m...)
@@ -658,7 +658,7 @@ func (e *Echo) Add(method, path string, handler HandlerFunc, middleware ...Middl
 
 // Group creates a new router group with prefix and optional group-level middleware.
 func (e *Echo) Group(prefix string, m ...MiddlewareFunc) (g *Group) {
-	g = &Group{prefix: prefix, echo: e}
+	g = &Group{prefix: prefix, echo: e, autoHeadInGet: true}
 	g.Use(m...)
 	return
 }

echo_test.go

@@ -598,9 +598,9 @@ func TestEchoAutoHead(t *testing.T) {
 		return c.String(http.StatusTeapot, "OK")
 	})
 
-	assert.Equal(t, http.MethodHead, ri.Method)
+	assert.Equal(t, http.MethodGet, ri.Method)
 	assert.Equal(t, "/", ri.Path)
-	assert.Equal(t, http.MethodHead+":/", ri.Name)
+	assert.Equal(t, http.MethodGet+":/", ri.Name)
 	assert.Nil(t, ri.Parameters)
 
 	status, body := request(http.MethodHead, "/", e)
@@ -937,7 +937,7 @@ func TestEchoMethodNotAllowed(t *testing.T) {
 	e.ServeHTTP(rec, req)
 
 	assert.Equal(t, http.StatusMethodNotAllowed, rec.Code)
-	assert.Equal(t, "OPTIONS, GET", rec.Header().Get(HeaderAllow))
+	assert.Equal(t, "OPTIONS, GET, HEAD", rec.Header().Get(HeaderAllow))
 }
 
 func TestEcho_OnAddRoute(t *testing.T) {
@@ -978,6 +978,7 @@ func TestEcho_OnAddRoute(t *testing.T) {
 		t.Run(tc.name, func(t *testing.T) {
 
 			e := New()
+			e.AutoHeadCancel()
 
 			added := make([]string, 0)
 			cnt := 0

group.go

@@ -42,7 +42,7 @@ func (g *Group) DELETE(path string, h HandlerFunc, m ...MiddlewareFunc) RouteInf
 // GET implements `Echo#GET()` for sub-routes within the Group. Panics on error.
 func (g *Group) GET(path string, h HandlerFunc, m ...MiddlewareFunc) RouteInfo {
 	if g.autoHeadInGet {
-		_ = g.Add(http.MethodHead, path, h, m...)
+		_ = g.HEAD(path, h, m...)
 	}
 
 	return g.Add(http.MethodGet, path, h, m...)
@@ -115,6 +115,7 @@ func (g *Group) Group(prefix string, middleware ...MiddlewareFunc) (sg *Group) {
 	m = append(m, g.middleware...)
 	m = append(m, middleware...)
 	sg = g.echo.Group(g.prefix+prefix, m...)
+	sg.autoHeadInGet = true
 	return
 }
 

group_test.go

@@ -218,14 +218,14 @@ func TestGroup_AutoHEAD_in_GET(t *testing.T) {
 	})
 
 	assert.Equal(t, true, users.autoHeadInGet)
-	assert.Equal(t, http.MethodHead, ri.Method)
+	assert.Equal(t, http.MethodGet, ri.Method)
 	assert.Equal(t, "/users/activate", ri.Path)
-	assert.Equal(t, http.MethodHead+":/users/activate", ri.Name)
+	assert.Equal(t, http.MethodGet+":/users/activate", ri.Name)
 	assert.Nil(t, ri.Parameters)
 
 	status, body := request(http.MethodHead, "/users/activate", e)
 	assert.Equal(t, http.StatusTeapot, status)
-	assert.Equal(t, `OK`, body)
+	assert.Equal(t, "OK", body)
 }
 
 func TestGroup_HEAD(t *testing.T) {