`response.success?`であることを調べたくて`be_success`書いてるそのしたで、
しっかりステータスコードまで200かどうか確認してるの違和感ある。
1つのexampleに2つ書くのも良くない。
`response.success?`が`true`で`response.status`が`200`以外かえってきて落ちた場合に、
`response.success?`と`response.status`どっちが原因で落ちたのか分かりづらい。分けた方がよいのでは〜。

``` ruby
it "リクエストが成功する" do
  get 'index'
  expect(response).to be_success
end

it 'ステータス200が返る' do
  get :index
  expect(response.status).to eq(200)
end
```