プロジェクト

全般

プロフィール

Backport(バックポート) #3814

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

Chiharu Nakajima8年以上前に追加. 8年以上前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
対象バージョン:
開始日:
2013-09-04
期日:
進捗率:

100%


説明

現象

セッションを 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 表示 (1.59 KB) Shinichi Urabe, 2015-06-19 16:50


関連するチケット

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

関係しているリビジョン

リビジョン 45ed0013 (差分)
Kousuke Ebihara8年以上前に追加

(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

リビジョン bfb44e12
Shinichi Urabe8年以上前に追加

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

履歴

#1 Chiharu Nakajima8年以上前に更新

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

#2 Chiharu Nakajima8年以上前に更新

  • ステータスNew(新規) から Pending Testing(テスト待ち) に変更
  • 進捗率0 から 70 に変更

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

#3 Shinichi Urabe8年以上前に更新

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

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

#4 Shinichi Urabe8年以上前に更新

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

レビューOK

#5 Shinichi Urabe8年以上前に更新

  • 担当者Chiharu Nakajima にセット

#6 isao sano8年以上前に更新

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

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

#7 Shinichi Urabe8年以上前に更新

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

merged

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