Project

General

Profile

Backport(バックポート) #3980

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

Added by Chiharu Nakajima almost 3 years ago. Updated over 2 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2015-11-05
Due date:
% Done:

100%


Description

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 の中身を空にする


Related issues

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

Associated revisions

Revision dff2095b (diff)
Added by Chiharu Nakajima almost 3 years ago

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

Revision 909108d1
Added by kaoru n over 2 years ago

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

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

History

#1 Updated by Chiharu Nakajima almost 3 years ago

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

#2 Updated by Chiharu Nakajima almost 3 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#3 Updated by kaoru n almost 3 years ago

  • Target version changed from OpenPNE 3.6.24 to OpenPNE 3.6.x

#4 Updated by Shinichi Urabe over 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#6 Updated by kaoru n over 2 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.25

#7 Updated by isao sano over 2 years ago

  • Target version changed from OpenPNE 3.6.25 to OpenPNE 3.6.x

#8 Updated by kaoru n over 2 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.26

#9 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

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

#10 Updated by kaoru n over 2 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF