Skip to content

refactor(test): SameSiteNoneCompatSessionHandler用テストフィクスチャを削除#6555

Open
nanasess wants to merge 3 commits intoEC-CUBE:4.3-symfony7from
nanasess:refactor/remove-session-fixtures
Open

refactor(test): SameSiteNoneCompatSessionHandler用テストフィクスチャを削除#6555
nanasess wants to merge 3 commits intoEC-CUBE:4.3-symfony7from
nanasess:refactor/remove-session-fixtures

Conversation

@nanasess
Copy link
Copy Markdown
Contributor

@nanasess nanasess commented Jan 5, 2026

概要(Overview・Refs Issue)

SameSiteNoneCompatSessionHandler クラスとそのテスト (SameSiteNoneCompatSessionHandlerTest.php) が削除されたため、関連するテストフィクスチャも不要になりました。

このPRでは tests/Fixtures/session/ ディレクトリ全体を削除します。

方針(Policy)

  • SameSiteNoneCompatSessionHandler は Symfony 7 対応で削除済み
  • PHP 7.3.0 以降では setcookie() で SameSite 属性をネイティブサポート
  • これらのフィクスチャは SameSiteNoneCompatSessionHandlerTest.php 専用であり、他のテストから参照されていない

実装に関する補足(Appendix)

削除されたファイル:

  • tests/Fixtures/session/common.php - TestSessionHandler定義(存在しない SameSiteNoneCompatSessionHandler を継承)
  • tests/Fixtures/session/storage.php, regenerate.php, empty_destroys.php など - テスト用PHPスクリプト
  • 各種 .expected ファイル - 期待値ファイル

テスト(Test)

  • PHPUnit テスト
  • PHPStan

相談(Discussion)

特になし

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

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

レビュワー確認項目

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

🤖 Generated with Claude Code

SameSiteNoneCompatSessionHandlerクラスとそのテストが削除されたため、
関連するテストフィクスチャも不要になりました。

削除されたファイル:
- tests/Fixtures/session/ ディレクトリ全体
  - common.php (TestSessionHandler定義)
  - storage.php, regenerate.php, empty_destroys.php など
  - 各種 .expected ファイル

理由:
- SameSiteNoneCompatSessionHandler は Symfony 7 対応で削除済み
- PHP 7.3.0 以降では SameSite 属性をネイティブサポート
- これらのフィクスチャは他のテストから参照されていない

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@dotani1111 dotani1111 added the refactor リファクタリング label Jan 7, 2026
@dotani1111 dotani1111 added this to the 4.4.0 milestone Jan 7, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8c89d95e-9c6d-44d7-a1ce-4bd8cda2d34a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor リファクタリング

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants