Enhance/#101 implement memo search test #109
Conversation
backend/spec/requests/memos_spec.rb
Outdated
| @@ -58,6 +58,63 @@ | |||
| expect(response).to have_http_status(:unauthorized) | |||
| end | |||
| end | |||
|
|
|||
| context 'あいまい検索する場合' do | |||
There was a problem hiding this comment.
contextは前提条件や状況を記載するので「あいまい検索する場合」だと少し抽象的かなと思いました！
条件や状況をとしては
クエリパラメータでタイトルが指定された時、コンテンツが指定された時、並び順が指定された時とあるので
contextブロックもその条件ごとに区切る方が意味がわかりやすいかなと思いました(仕様書としての機能もあるため)
たとえばここでいうと
context 'ログイン中かつ、タイトルが指定された場合'
のような形です！
| let!(:first_memo) { create(:memo, title: 'メモタイトル１', content: 'メモコンテンツ１') } | ||
| let!(:second_memo) { create(:memo, title: 'メモタイトル２', content: 'メモコンテンツ２') } | ||
|
|
||
| before { sign_in(user) } |
There was a problem hiding this comment.
上記2件のデータだけだと、部分一致検索ができている、いないにも関わらず2件ヒットすると思うので
let!(:first_memo) { create(:memo, title: 'メモタイトル１', content: 'メモコンテンツ１') }
let!(:second_memo) { create(:memo, title: 'メモタイトル２', content: 'メモコンテンツ２') }
let!(:third_memo) { create(:memo, title: 'その他タイトル', content: 'その他コンテンツ') }
と合致しないデータも含めた方が良いかなと思いました！
backend/spec/requests/memos_spec.rb
Outdated
| it 'コンテンツで部分一致するメモが返る' do | ||
| aggregate_failures do | ||
| get '/memos', params: { content: 'メモ' } | ||
| assert_request_schema_confirm | ||
| assert_response_schema_confirm(200) | ||
| expect(response.parsed_body['memos'].length).to eq(2) | ||
| result_memo_ids = response.parsed_body['memos'].pluck('id') | ||
| expect(result_memo_ids).to contain_exactly(first_memo.id, second_memo.id) | ||
| end | ||
| end |
There was a problem hiding this comment.
ここも
context 'ログイン中かつ、コンテンツが指定された場合
のように条件ごとに切り分けましょう！
| result_memo_ids = response.parsed_body['memos'].pluck('id') | ||
| expect(result_memo_ids).to eq([memo_c.id, memo_b.id, memo_a.id]) | ||
| end | ||
| end |
There was a problem hiding this comment.
ここも同様に昇順の場合、降順の場合でcontextをわけましょう！
例context 'ログイン中かつ、並び順にdescが指定された場合'
describe GET /memosの記述が2つあることによりrubocopの警告を避けるためrubocop:disable RSpec/RepeatedExampleGroupDescription等の記述も追加。
backend/spec/requests/memos_spec.rb
Outdated
| @@ -60,6 +61,71 @@ | |||
| end | |||
| end | |||
|
|
|||
| describe 'GET /memos' do | |||
There was a problem hiding this comment.
こちら新たに describeを追加する必要はないかなと思いました！
同様のPATHなので、今ある describe 'GET /memos' ブロックの中にテストを追加していくだけで良いと思いました。
よってExampleの重複の警告を除外するrubocopの記述も不要となるかと思います！
対応するissue
対応内容