テストで裏を取る

OrderServiceImplTest は findById・createOrder・changeStatus・cancelOrder・findAll の代表ケースが正常系・異常系とも実装済みで、changeStatus の @Nested に4件の TODO コメントが残っています。CONFIRMED から SHIPPED、SHIPPED から DELIVERED、CANCELLED からの遷移は全部不可、同一ステータスへの遷移は不可、の4件です。テスト基盤は組んであるので @ExtendWith や @Mock を書き足す必要はありません。AssertJ の assertThat と assertThatThrownBy、Mockito の when と verify、既存テストと同じ書き方に揃えるだけです。追記先は既存ファイルで、新規ファイルは作りません。

.\mvnw.cmd test -Dtest=OrderServiceImplTest は9件中1件が赤です。Day2 で初めてテストを走らせると赤が1件出ますが、これは配布時点で仕込んである失敗で、環境の不備ではありません。DELIVERED の受注をキャンセルできてしまう穴が cancelOrder に残っていて、例外が出ずにキャンセルが通り、スタブしていない save が null を返すので、続く行で NullPointerException になります。画面に出るのは expecting ... InvalidOrderStateException but was ... NullPointerException の行です。Day3 の題材なので、今日は直しません。走らせる前に配布時点の状態を一度見ておくと、自分が足したテストの失敗と切り分けやすくなります。

@ を打つと候補が出るので、ファイル名でもパスでも選べます。実行結果の読み方には癖がひとつ。Maven のコンソールに出るのはクラス単位の集計行と落ちたテストの名前だけで、通ったテストのメソッド名は表示されません。名前まで確かめたいときは target/surefire-reports/TEST-com.example.order.service.OrderServiceImplTest$ChangeStatusTest.xml を開き、testcase name に自分が足したメソッド名が並ぶのを見てください。

同一ステータスへの遷移には専用のチェック文がありませんが、どの case も自分自身を遷移先に挙げていないので例外が出ます。このテストは配布時点でも緑です。もうひとつ、CONFIRMED から CANCELLED が成功する前提のテストは、Day1 の D1-5 を済ませていなければ落ちます。落ちたときは validateStatusTransition の CONFIRMED の case を開いて、SHIPPED しか許していないことを自分で確かめてください。生成物の誤りなのか実装の穴なのかは、ここが分かれ目です。採否の基準は docs/生成物チェックリスト.md の B-5 に書いてあります。

BuggyOrderCalculator は受注5件を内部に持つ単体クラスで、公開メソッドは3つです。仕様は Javadoc に書いてあります。数量10個以上で5%、20個以上で10%、50個以上で15%の割引。指定顧客の受注が1件もなければ合計0円。納品予定日の当日にまだ出荷されていない受注も遅延に数える。この3つを頭に入れてからテストを書きます。

置き場所には一手いります。このクラスは exercises/day2 にあり、Maven は src/main/java の下しかコンパイルしないため、テストクラスから参照できません。移さずに走らせると「シンボルを見つけられません」でコンパイルが止まります。移したあとは修正もそのファイルに対して行い、exercises 側には戻しません。

期待値は電卓で先に出しておきます。突き合わせる基準がないと、生成物の粗は見つけにくくなります。数量10・単価50000なら5%引きで475000円、いまの実装が返すのは500000円。getCustomerOrderTotal は合計の初期値が null のままなので、該当受注ゼロの顧客では null がそのまま返ります。メソッド自体は例外を投げません。NullPointerException になるのは戻り値を int で受けたときで、自動アンボクシングで null を数値に展開しようとして落ちます。期待値0で突き合わせるテストは値が合わずに Failures、int で受けるテストは Errors 側に出ます。

/gen-test はこのプロジェクトに同梱したコマンドで、実体は .claude/commands/gen-test.md です。handson フォルダを開いた状態で claude を起動すると、/ge まで打てば候補に出ます。テストが1件でも赤いと Maven は BUILD FAILURE で終わりますが、今日はそれが狙いどおりの状態なので、Results: の下の Tests run と Failures、Errors の行だけ読んでください。

修正後の確認は java src\main\java\com\example\order\buggy\BuggyOrderCalculator.java です。main に自己診断が入っていて、直る前は「バグ発生!」が3か所、直ったあとは割引と遅延が「正常」、存在しない会社の合計が0円と出ます。

LegacyReportService は309行、出力は月次レポートと顧客別サマリーと高額アラートの3本です。動いてはいます。ただ、変数が d, q, p, val, tmp, cnt のまま、税率の 1.1 と高額判定の 1000000 と緊急判定の 2000000 が直接埋まっていて、明細は line = line + ... の連結で組み立てられ、月次と顧客別で集計処理がほぼ丸ごと重複しています。まず出力を固定してください。

java exercises\day2\LegacyReportService.java > before.txt を打つと、月次は合計件数8件・合計金額10,571,000円・平均金額1,321,375円、顧客別（東京電機）は2件で1,584,000円、高額アラートは1件と出ます。この数字が最後まで変わらなければ、内部をどう書き換えても振る舞いは守れています。比較に使うのは fc.exe。PowerShell で fc とだけ打つと別のコマンドの別名として解釈されるので、.exe まで付けてください。

定数名には注意がいります。1.1 は税率ではなく税込にする倍率なので、OrderServiceImpl にある TAX_RATE = 0.10 と同じ名前を付けると、値の意味が違う定数が2つできます。TAX_INCLUDED_RATE のように別の名前にするか、TAX_RATE = 0.10 を宣言して本文を subtotal * (1 + TAX_RATE) に書き換えるか、どちらかを選んでください。後者に振っても出力は変わらないので、fc.exe でそのまま確かめられます。

置換の範囲も先に決めておきます。1.1 は generateMonthlyReport と generateCustomerSummary と generateHighValueAlert の3か所、1000000 と 2000000 は generateHighValueAlert だけです。変数名のほうは generateMonthlyReport に絞り、ループの i は1文字のままで構いません。

順番は、差分が小さく後ろに響かないものから進めます。変数名、次にマジックナンバーの定数化、余裕があれば文字列連結と共通処理の抽出です。この時間内に型安全なクラスへの置き換えまで届かなくて構いません。1段階ごとに比較を挟む進め方が残れば、この演習で狙ったところは足ります。なお、このファイルは src/main/java の下にないので .\mvnw.cmd compile の対象外です。コンパイルの確認は java の単一ファイル実行が兼ねます。