Function TestGroupRoute_catchAllMasksMethodHandling

group_method_handling_test.go:91–115

Characterization of the regression the 405/OPTIONS tests above guard against: registering a group-wide catch-all (the manual equivalent of #2996's auto- registration) makes it match every method, so method mismatches and the automatic OPTIONS response are masked as 404 even though the concrete route

(t *testing.T)

// OPTIONS response are masked as 404 even though the concrete route still resolves.
// If a future change teaches the catch-all to preserve method semantics, update this.
func TestGroupRoute_catchAllMasksMethodHandling(t *testing.T) {
	e := New()
	g := e.Group("/api")
	g.GET("/users", func(c *Context) error { return c.String(http.StatusOK, "users") })
	g.RouteNotFound("/*", func(c *Context) error { return c.NoContent(http.StatusNotFound) })

	// The concrete route still resolves.
	status, body := request(http.MethodGet, "/api/users", e)
	assert.Equal(t, http.StatusOK, status)
	assert.Equal(t, "users", body)

	// But the catch-all masks the method mismatch (would be 405) ...
	post := httptest.NewRequest(http.MethodPost, "/api/users", nil)
	postRec := httptest.NewRecorder()
	e.ServeHTTP(postRec, post)
	assert.Equal(t, http.StatusNotFound, postRec.Code,
		"a group-wide catch-all masks the 405 method-mismatch as 404")

	// ... and the automatic OPTIONS response (would be 204).
	opts := httptest.NewRequest(http.MethodOptions, "/api/users", nil)
	optsRec := httptest.NewRecorder()
	e.ServeHTTP(optsRec, opts)
	assert.Equal(t, http.StatusNotFound, optsRec.Code,
		"a group-wide catch-all masks the automatic OPTIONS (204) response as 404")
}