4月・5月であっても住民税が正しく計算できるように修正 - Githubissues

IkumaTadokoro / quitcost

「無職になったらいくらかかる？」を簡単に計算できるサービス

https://quitcost.herokuapp.com/

25 stars 2 forks source link

4月・5月であっても住民税が正しく計算できるように修正 #175

Closed IkumaTadokoro closed 2 years ago

IkumaTadokoro commented 2 years ago

目的

Closes: #164 Closes: #166

4月5月に計算を行うと、昨昨年度の所得情報（所得・社会保険料）がないためにAPIが落ちてしまうバグを修正。

やったこと

SImulationAPI側で、昨昨年度の所得情報（所得・社会保険料）を受け取れるように修正
- 項目名：所得：previous_salary、社会保険料：previous_social_insurance
- 修正に合わせて、インスタンス変数が増えたことParamParserクラスの見通しが悪くなったため、アクセサとインスタンス変数を整理
- 修正の前提として、バグが発生している状態のテストを通せるようにするため、不必要にモックを使用して通ってしまっていたテストを、直接処理を呼び出すように修正
フォーム側で、昨昨年度の所得情報（所得・社会保険料）を受け取れるように修正
- フォーム追加に際して、全体のステップ数等を変更する必要があったが、今後の項目増減に備え、動的にルーティングおよびステップ情報を取得できるように修正

PR提出前のチェックリスト:

[x] PRの関心はただ一つだけになっている & 文法的に正しく、明確かつ完全なタイトルと本文になっている
[x] 良いコミットメッセージを書いている
[x] 関連issueがある場合、コミットメッセージにClosing Keywordsを使っている
[x] Featureブランチは最新版のmainブランチに追随している (そうでなければrebaseすること)
[x] 関連するコミットはsquashした
[x] テストを追加した
[x] bin/lintとbin/rspecを実行した