[OWASP ZAP] change_password のテストを追加

[nanasess]

概要(Overview・Refs Issue)

• change_password のテストを追加

方針(Policy)

パスワードを変更してしまうと、 OWASP ZAP の自動ログインが無効になり、アクティブスキャンが継続できなくなってしまうため、パスワードの変更を抑制するパッチを当てる必要がある

実装に関する補足(Appendix)

• workflow で、以下のようにパッチを適用する処理を追加

      - name: Apply patch to change_password
        if: matrix.group == 'test/admin/change_password.test.ts'
        working-directory: zap/selenium/ci/TypeScript
        run: git am patches/0001-Member.patch

テスト（Test)

OWASP ZAP のテストが通ることを確認

相談（Discussion）

• POST のテストの際に、 2つ前の履歴を取得する必要がある。 結構こういったケースが多そうなので、 zapClient.getLastMessage(url) に offset のパラメータを追加したらどうか

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5308 (6c01ec3) into 4.1 (710b64c) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                4.1    #5308   +/-   ##
=========================================
  Coverage     68.42%   68.42%           
  Complexity     6159     6159           
=========================================
  Files           463      463           
  Lines         25280    25280           
=========================================
+ Hits          17298    17299    +1     
+ Misses         7982     7981    -1     

Flag	Coverage Δ
E2E	57.37% <ø> (ø)
Unit	76.16% <ø> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
src/Eccube/Entity/TaxRule.php	85.18% <0.00%> (-1.24%)	⬇️
...be/Service/PurchaseFlow/Processor/TaxProcessor.php	73.77% <0.00%> (+3.27%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 710b64c...6c01ec3. Read the comment docs.

[chihiro-adachi]

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