diff options
| author | Lin Jen-Shin <godfat@godfat.org> | 2017-08-11 15:50:35 +0300 |
|---|---|---|
| committer | Lin Jen-Shin <godfat@godfat.org> | 2017-08-11 15:50:35 +0300 |
| commit | 4f0fa13eb85994b747c1eb253e346b76b98b5c5b (patch) | |
| tree | f690c9013ce1653272d0a345cb4840a7e6336ec0 /doc/development/testing.md | |
| parent | c62ae6cfd70d518386fdbcb9714d18ac4f5e8c31 (diff) | |
Show error message for API 500 error in tests, and
document have_gitlab_http_status
Diffstat (limited to 'doc/development/testing.md')
| -rw-r--r-- | doc/development/testing.md | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/doc/development/testing.md b/doc/development/testing.md index ea94c87d8c6..ac1865f6682 100644 --- a/doc/development/testing.md +++ b/doc/development/testing.md @@ -270,6 +270,15 @@ complexity of RSpec expectations.They should be placed under a certain type of specs only (e.g. features, requests etc.) but shouldn't be if they apply to multiple type of specs. +#### have_gitlab_http_status + +Prefer `have_gitlab_http_status` over `have_http_status` because the former +could also show the response body whenever the status mismatched. This would +be very useful whenever some tests start breaking and we would love to know +why without editing the source and rerun the tests. + +This is especially useful whenever it's showing 500 internal server error. + ### Shared contexts All shared contexts should be be placed under `spec/support/shared_contexts/`. |