Closed IkumaTadokoro closed 2 years ago
Closes #123
変更前のテストでは、Factoryからランダムに選ばれた地方公共団体コードを用いて、「国民健康保険料登録」画面の「市区町村」セレクトボックスの市区町村名を選択していた。
日本には「同じ名前の市区町村」が複数存在する。それに該当する市区町村のコードが選択された場合、Capybara側で選択対象を絞りきれずエラーになっていた。
そもそも画面から登録する場合にも「同じ名前の市区町村」を判別する手段がなかったため、「都道府県名」をセレクトボックスに表示するように変更し、テストもそれにあわせる形で修正しました。
テスト修正に際し、複数入力をfill_in単体で行なっていたので、withinでまとめました。
fill_in
within
PR提出前のチェックリスト:
main
bin/lint
bin/rspec
Closes #123
バグの原因
やっていたこと
変更前のテストでは、Factoryからランダムに選ばれた地方公共団体コードを用いて、「国民健康保険料登録」画面の「市区町村」セレクトボックスの市区町村名を選択していた。
バグの原因
日本には「同じ名前の市区町村」が複数存在する。それに該当する市区町村のコードが選択された場合、Capybara側で選択対象を絞りきれずエラーになっていた。
対応
方針
そもそも画面から登録する場合にも「同じ名前の市区町村」を判別する手段がなかったため、「都道府県名」をセレクトボックスに表示するように変更し、テストもそれにあわせる形で修正しました。
UI
申し送り事項
テスト修正に際し、複数入力を
fill_in
単体で行なっていたので、within
でまとめました。PR提出前のチェックリスト:
main
ブランチに追随している (そうでなければrebaseすること)bin/lint
とbin/rspec
を実行した