Project

General

Profile

Actions

Backport(バックポート) #3814

closed

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

Added by Chiharu Nakajima over 9 years ago. Updated over 9 years ago.

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

100%

Estimated time:

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 のセッションストレージの上書きを無効にする」のどちらかを選択することになるが、実装の容易さ等を考慮し、後者を選択することが適当と思われる。


Files


Related issues 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Chiharu Nakajima over 9 years ago

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

Updated by Chiharu Nakajima over 9 years ago

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

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

Actions #3

Updated by Shinichi Urabe over 9 years ago

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

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

Actions #4

Updated by Shinichi Urabe over 9 years ago

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

レビューOK

Actions #5

Updated by Shinichi Urabe over 9 years ago

  • Assignee set to Chiharu Nakajima
Actions #6

Updated by isao sano over 9 years ago

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

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

Actions #7

Updated by Shinichi Urabe over 9 years ago

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

merged

Actions

Also available in: Atom PDF