Project

General

Profile

Bug(バグ) #3391

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

Added by Shinichi Urabe almost 6 years ago. Updated almost 4 years ago.

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

100%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)

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 to OpenPNE 3 - Backport(バックポート) #3813: デフォルトの OpenPNE.yml によって sf_factory_storage が設定されるため、 test 環境用のセッションストレージである sfSessionTestStorage が利用されず、セッションが発行されなくなる Fixed(完了) 2013-09-04
Copied to OpenPNE 3 - Backport(バックポート) #3814: デフォルトの OpenPNE.yml によって sf_factory_storage が設定されるため、 test 環境用のセッションストレージである sfSessionTestStorage が利用されず、セッションが発行されなくなる Fixed(完了) 2013-09-04

Associated revisions

Revision d48f33ce (diff)
Added by Kousuke Ebihara almost 4 years ago

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

Revision efcddf96
Added by Shinichi Urabe almost 4 years ago

Merge pull request #257 from C-nakajima/t-3391

(fixes #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 Shinichi Urabe almost 6 years ago

  • Description updated (diff)

#2 Updated by Shinichi Urabe over 4 years ago

  • Priority changed from Normal(通常) to High(高め)

#3 Updated by Shinichi Urabe about 4 years ago

修正案を添付しました。以下のコマンドで取り込めます

$ git am -3 0001-test-factories.yml-refs-1074.patch

#4 Updated by Chiharu Nakajima about 4 years ago

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

#5 Updated by Chiharu Nakajima about 4 years ago

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

#6 Updated by Chiharu Nakajima almost 4 years ago

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

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

#7 Updated by Shinichi Urabe almost 4 years ago

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

#8 Updated by Shinichi Urabe almost 4 years ago

  • Assignee set to Chiharu Nakajima

#9 Updated by Kousuke Ebihara almost 4 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

更新履歴 d48f33ce9076ec0a5804cf7be6ddcdf7d7b41f15 で適用されました。

Also available in: Atom PDF