diff --git a/context.go b/context.go index d69df70b..99097691 100644 --- a/context.go +++ b/context.go @@ -894,7 +894,8 @@ func (c *Context) Render(code int, r render.Render) { } if err := r.Render(c.Writer); err != nil { - panic(err) + c.Error(err) + c.Abort() } } diff --git a/context_test.go b/context_test.go index 9e02aede..690360e3 100644 --- a/context_test.go +++ b/context_test.go @@ -642,25 +642,21 @@ func TestContextBodyAllowedForStatus(t *testing.T) { assert.True(t, true, bodyAllowedForStatus(http.StatusInternalServerError)) } -type TestPanicRender struct{} +type TestErrorRender struct{} -func (*TestPanicRender) Render(http.ResponseWriter) error { - return errors.New("TestPanicRender") +var errTestPanicRender = errors.New("TestPanicRender") + +func (*TestErrorRender) Render(http.ResponseWriter) error { + return errTestPanicRender } -func (*TestPanicRender) WriteContentType(http.ResponseWriter) {} +func (*TestErrorRender) WriteContentType(http.ResponseWriter) {} func TestContextRenderPanicIfErr(t *testing.T) { - defer func() { - r := recover() - assert.Equal(t, "TestPanicRender", fmt.Sprint(r)) - }() w := httptest.NewRecorder() c, _ := CreateTestContext(w) - - c.Render(http.StatusOK, &TestPanicRender{}) - - assert.Fail(t, "Panic not detected") + c.Render(http.StatusOK, &TestErrorRender{}) + assert.Equal(t, errorMsgs{&Error{Err: errTestPanicRender, Type: 1}}, c.Errors) } // Tests that the response is serialized as JSON diff --git a/render/json.go b/render/json.go index 3ebcee97..ac0452e6 100644 --- a/render/json.go +++ b/render/json.go @@ -54,10 +54,7 @@ var ( // Render (JSON) writes data with custom ContentType. func (r JSON) Render(w http.ResponseWriter) (err error) { - if err = WriteJSON(w, r.Data); err != nil { - panic(err) - } - return + return WriteJSON(w, r.Data) } // WriteContentType (JSON) writes JSON ContentType. diff --git a/render/render_test.go b/render/render_test.go index e417731a..20a9c875 100644 --- a/render/render_test.go +++ b/render/render_test.go @@ -39,12 +39,12 @@ func TestRenderJSON(t *testing.T) { assert.Equal(t, "application/json; charset=utf-8", w.Header().Get("Content-Type")) } -func TestRenderJSONPanics(t *testing.T) { +func TestRenderJSONError(t *testing.T) { w := httptest.NewRecorder() data := make(chan int) // json: unsupported type: chan int - assert.Panics(t, func() { assert.NoError(t, (JSON{data}).Render(w)) }) + assert.Error(t, (JSON{data}).Render(w)) } func TestRenderIndentedJSON(t *testing.T) {