プロジェクト

全般

プロフィール

Backport(バックポート) #3980

機能テスト実行時にopBrowserによる各リクエスト間でMemberConfigのキャッシュが意図せず引き継がれてしまう

Chiharu Nakajima7年以上前に追加. 約7年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2015-11-05
期日:
進捗率:

100%


説明

Overview (現象)

MemberConfigTable クラスには MemberConfig をキャッシュするための MemberConfigTable::$results が用意されおり、多くのコードはこのキャッシュがリクエストごとに初期化される前提で実装されているが、機能テスト時には MemberConfigTable::$results のキャッシュがリクエストを跨いで引き継がれてしまうため、テスト時のみ発生する意図しない動作の原因となっている。

Causes (原因)

MemberConfigTable クラスには MemberConfig をキャッシュするための MemberConfigTable::$results が用意されているが、通常のブラウザからのアクセスによって OpenPNE が動作する場合はリクエスト毎にこのキャッシュは破棄される。
しかし機能テストで使用する opBrowser では、リクエストごとに sfContext の初期化は行うものの静的変数の初期化は行われないため MemberConfigTable のインスタンス(Doctrine_Core::getTable() によって生成されるシングルトン)は残ったままとなり、MemberConfigTable::$results も初期化されない。

Way to fix (修正内容)

opBrowser がリクエストを実行する度(sfContext を生成するタイミング)に MemberConfigTable::$results の中身を空にする


関連するチケット

コピー元 OpenPNE 3 - Bug(バグ) #3852: 機能テスト実行時にopBrowserによる各リクエスト間でMemberConfigのキャッシュが意図せず引き継がれてしまう Won't fix(対応せず) 2015-11-05

関係しているリビジョン

リビジョン dff2095b (差分)
Chiharu Nakajima7年以上前に追加

(refs #3980, BP from #3852) clear MemberConfig caches on every request by opBrowser

リビジョン 909108d1
kaoru n約7年前に追加

Merge pull request #354 from C-nakajima/t-3980

(refs #3980, BP from #3852) clear MemberConfig caches on every request by opBrowser

履歴

#1 Chiharu Nakajima7年以上前に更新

  • コピー元 Bug(バグ) #3852: 機能テスト実行時にopBrowserによる各リクエスト間でMemberConfigのキャッシュが意図せず引き継がれてしまう を追加

#2 Chiharu Nakajima7年以上前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

プルリクエストしました。
https://github.com/openpne/OpenPNE3/pull/354

#3 kaoru n7年以上前に更新

  • 対象バージョンOpenPNE 3.6.24 から OpenPNE 3.6.x に変更

#4 Shinichi Urabe7年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#6 kaoru n7年以上前に更新

  • 対象バージョンOpenPNE 3.6.x から OpenPNE 3.6.25 に変更

#7 isao sano7年以上前に更新

  • 対象バージョンOpenPNE 3.6.25 から OpenPNE 3.6.x に変更

#8 kaoru n約7年前に更新

  • 対象バージョンOpenPNE 3.6.x から OpenPNE 3.6.26 に変更

#9 isao sano約7年前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

試験完了しました。
問題ありません。

#10 kaoru n約7年前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

他の形式にエクスポート: Atom PDF