Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

プラグインに Codeception を同梱できるよう修正 #4505

Merged
merged 1 commit into from Apr 7, 2020

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Apr 6, 2020

概要(Overview・Refs Issue)

プラグインに Codeception を同梱しようとした場合、本体に同梱している Codeception の Actor(AcceptanceTester など) と、プラグインの Actor が競合し、 Symfony DI キャッシュの生成に失敗してしまう。
プラグインの Codeception を Tests 以下に配置すれば回避できるが、別ディレクトリに配置したい

方針(Policy)

services.yaml の exclude に Codeception を追加する

実装に関する補足(Appendix)

本来であれば、プラグイン側の Actor に namespace を付与するのが PSR-4 的には正しいが、namespace を設定し、ディレクトリ名を namespace と一致させると Codeception が Actor を見つけられなくなってしまう。

テスト(Test)

プラグインの Codeception が動作するのを確認

相談(Discussion)

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

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

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか

@okazy okazy added the improvement 機能改善 label Apr 7, 2020
@okazy okazy modified the milestones: 4.0.4, 4.0.x Apr 7, 2020
Copy link
Contributor

@okazy okazy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

動作に問題ないかと思います。

@chihiro-adachi chihiro-adachi modified the milestones: 4.0.x, 4.0.4 Apr 7, 2020
@chihiro-adachi chihiro-adachi merged commit 08243ac into EC-CUBE:4.0 Apr 7, 2020
@chihiro-adachi
Copy link
Contributor

@nanasess 問題ないかと思います。マージしました。

@okazy okazy added the affected:plugin プラグインに影響する可能性のある実装変更 label Jun 16, 2020
@nanasess nanasess deleted the codeception-plugin branch October 3, 2022 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected:plugin プラグインに影響する可能性のある実装変更 improvement 機能改善
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants