Project

General

Profile

Backport(バックポート) #3814

デフォルトの OpenPNE.yml によって sf_factory_storage が設定されるため、 test 環境用のセッションストレージである sfSessionTestStorage が利用されず、セッションが発行されなくなる

Added by Chiharu Nakajima almost 6 years ago. Updated over 5 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2013-09-04
Due date:
% Done:

100%


Description

現象

セッションを database に保持する設定とした場合、デフォルトの OpenPNE.yml 設定によって sf_factory_storage として opPDODatabaseSessionStorage が設定される。これによって、 factories.yml の設定を変更することなく、このセッションストレージクラスが使用されることになる。

だが、これには、 test 環境でもこのストレージクラスが使用されてしまうという問題がある。 symfony デフォルトの factories.yml においては、 test 環境用の sfSessionTestStorage を使用するような配慮がなされているが、 OpenPNE.yml のデフォルト設定によって無力化してしまう。これによって、

  • 機能テストの出力の存在をストレージクラスが考慮せず、セッションの開始が拒否される
  • セッションの継続ができない

などといった問題が生じる。

原因

バグが発生した原因を記入

修正案

解決策としては「sf_factory_storage の設定内容に応じた test 環境用のラッパークラスを作成する」「test 環境では OpenPNE.yml のセッションストレージの上書きを無効にする」のどちらかを選択することになるが、実装の容易さ等を考慮し、後者を選択することが適当と思われる。

0001-test-factories.yml-refs-1074.patch View (1.59 KB) Shinichi Urabe, 2015-06-19 16:50


Related issues

Copied from OpenPNE 3 - Bug(バグ) #3391: デフォルトの OpenPNE.yml によって sf_factory_storage が設定されるため、 test 環境用のセッションストレージである sfSessionTestStorage が利用されず、セッションが発行されなくなる Fixed(完了) 2013-09-04

Associated revisions

Revision 45ed0013 (diff)
Added by Kousuke Ebihara over 5 years ago

(refs #3814, BP from #3391) In the test environment it was not allowed to override the session storage settings that are defined in the factories.yml

Revision bfb44e12
Added by Shinichi Urabe over 5 years ago

Merge pull request #259 from C-nakajima/t-3814

(refs #3814, BP from #3391) In the test environment it was not allowed to override the session storage settings that are defined in the factories.yml

History

#1 Updated by Chiharu Nakajima almost 6 years ago

  • Copied from Bug(バグ) #3391: デフォルトの OpenPNE.yml によって sf_factory_storage が設定されるため、 test 環境用のセッションストレージである sfSessionTestStorage が利用されず、セッションが発行されなくなる added

#2 Updated by Chiharu Nakajima over 5 years ago

  • Status changed from New(新規) to Pending Testing(テスト待ち)
  • % Done changed from 0 to 70

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

#3 Updated by Shinichi Urabe over 5 years ago

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

ステータスを多分間違えているようです

#4 Updated by Shinichi Urabe over 5 years ago

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

レビューOK

#5 Updated by Shinichi Urabe over 5 years ago

  • Assignee set to Chiharu Nakajima

#6 Updated by isao sano over 5 years ago

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

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

#7 Updated by Shinichi Urabe over 5 years ago

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

merged

Also available in: Atom PDF