[4.2]クーポン利用時に「税率が変更されました」のメッセージが表示されるのを修正

[nanasess]

概要(Overview・Refs Issue)

• クーポン利用時に「税率が変更されました」のメッセージが表示されるのを修正
• [4.2] クーポン利用時に「税率が変更されました」のメッセージが表示される #5804

方針(Policy)

税額ではなく、税率を比較するよう修正

実装に関する補足(Appendix)

テスト（Test)

ユニットテストが通るのを確認

相談（Discussion）

税率のみの変更をチェックするようにしたが、税額かつ税率をチェックした方がよいか？

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更はありません
• フックポイントの呼び出しタイミングの変更はありません
• フックポイントのパラメータの削除・データ型の変更はありません
• twigファイルに渡しているパラメータの削除・データ型の変更はありません
• Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
• 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか
  • 権限を超えた操作が可能にならないか
  • 不要なファイルアップロードがないか
  • 外部へ公開されるファイルや機能の追加ではないか
  • テンプレートでのエスケープ漏れがないか

[codecov-commenter]

Codecov Report

Merging #5807 (967bc6f) into 4.2 (3315ae0) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                4.2    #5807      +/-   ##
============================================
- Coverage     79.00%   79.00%   -0.01%     
  Complexity     6287     6287              
============================================
  Files           470      470              
  Lines         21105    21105              
============================================
- Hits          16675    16674       -1     
- Misses         4430     4431       +1     

Flag	Coverage Δ
E2E	65.01% <100.00%> (-0.02%)	⬇️
Unit	77.83% <100.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
.../PurchaseFlow/Processor/TaxRateChangeValidator.php	100.00% <100.00%> (ø)
...be/Service/PurchaseFlow/Processor/TaxProcessor.php	82.45% <0.00%> (-1.76%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[chihiro-adachi]

@nanasess
テストが落ちているようで、見ていただけると幸いです。
https://github.com/EC-CUBE/ec-cube/actions/runs/3065395865/jobs/4950623617

[nanasess]

@chihiro-adachi
ローカルで再現しないのと、個人アカウントの方では通っているので、ガチャかなと。
https://github.com/nanasess/ec-cube/actions/runs/3065387073/jobs/4949431551
2回連続で落ちているので修正しておきたいですね

[chihiro-adachi]

@nanasess
了解です。ありがとうございます。
動作は問題なさそうです。

[chihiro-adachi]

ガチャはissueたてておきました。#5808

[nanasess]

@chihiro-adachi
再実行したら通りましたね
https://github.com/EC-CUBE/ec-cube/actions/runs/3065395865/jobs/4951021228

[chihiro-adachi]

@nanasess
ありがとうございます。マージしました。