Project

General

Profile

Backport(バックポート) #3466

MemberConfigFormTest, MemberRelationshipTableTest がsendmailコマンドを実行しようとする

Added by Mutsumi Imamura almost 6 years ago. Updated almost 6 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2013-10-08
Due date:
% Done:

100%


Description

Overview (現象)

MemberConfigFormTest, MemberRelationshipTableTest は、それぞれメールアドレスの変更やフレンド申請のテスト時に sendmail コマンドを実行しようとする。
これではテストの度に example@docomo.ne.jpsns4@example.com へのメール送信が実際に行われてしまうほか、メール送信が可能な環境でないとテストが実行できない状態となってしまう。

Way to fix (修正内容)

テスト実行前に Zend_Mail が使用する Transport クラスをモックに差し替えるコードを追加する


Related issues

Related to OpenPNE 3 - Bug(バグ) #3443: MemberConfigFormTest, MemberRelationshipTableTest がsendmailコマンドを実行しようとする Won't fix(対応せず) 2013-10-08

Associated revisions

Revision 9b2761e7 (diff)
Added by Youichi Kimura almost 6 years ago

mock out the Zend_Mail_Transport object in unit tests (fixes #3466, BP
from #3443)
(cherry picked from commit 2d73654a5ed2755c2cfe52a8be9e10e49e95eb1a)

Revision 0fa7d5ad (diff)
Added by Youichi Kimura almost 6 years ago

use opZendMailTransportMock for CommunityMemberTable::processJoinConfirm()
tests (fixes #3466, BP from #3443)
(cherry picked from commit b9a0894299d19644281bf3eca2a37ec572371b41)

Revision 8519b1d2
Added by Youichi Kimura almost 6 years ago

Merge branch 't-3466' into stable-3.8.x

History

#1 Updated by Akihiro KOBAYASHI almost 6 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Akihiro KOBAYASHI

#2 Updated by Akihiro KOBAYASHI almost 6 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

#3 Updated by Youichi Kimura almost 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

バックポート元の #3443 には 2 つコミットがありますが、Pull Request には b9a0894299d19644281bf3eca2a37ec572371b41 が含まれていませんでした。
t-3466 ブランチに cherry-pick を行った上でステータスを Pending Review に戻して下さい。

#4 Updated by Akihiro KOBAYASHI almost 6 years ago

  • Status changed from Rejected(差し戻し) to Pending Fixing(修正待ち)
  • % Done changed from 50 to 0

t-3466 ブランチに cherry-pick を行った上でステータスを Pending Review に戻しました。

#5 Updated by Akihiro KOBAYASHI almost 6 years ago

  • Status changed from Pending Fixing(修正待ち) to Accepted(着手)

#6 Updated by Akihiro KOBAYASHI almost 6 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

#7 Updated by Youichi Kimura almost 6 years ago

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

#8 Updated by Youichi Kimura almost 6 years ago

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

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

#9 Updated by Youichi Kimura almost 6 years ago

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

#10 Updated by Youichi Kimura almost 6 years ago

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

#11 Updated by Mutsumi Imamura almost 6 years ago

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

./symfony test:all を実行し異常終了しないことを確認しました。
ブラウザからの動作テストも問題ありません。

Also available in: Atom PDF