Project

General

Profile

Bug(バグ) #3443

Task(タスク) #3403: OpenPNE の既存のテストコードが正常に動作するように修正を行う

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

Added by Youichi Kimura about 6 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Target version:
Start date:
2013-10-08
Due date:
% Done:

0%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

Description

Overview (現象)

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

Way to fix (修正内容)

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


Related issues

Related to OpenPNE 3 - Backport(バックポート) #3444: MemberConfigFormTest, MemberRelationshipTableTest がsendmailコマンドを実行しようとする Fixed(完了) 2013-10-08
Related to OpenPNE 3 - Backport(バックポート) #3466: MemberConfigFormTest, MemberRelationshipTableTest がsendmailコマンドを実行しようとする Fixed(完了) 2013-10-08

Associated revisions

Revision 2d73654a (diff)
Added by Youichi Kimura about 6 years ago

mock out the Zend_Mail_Transport object in unit tests (fixes #3443)

Revision b9a08942 (diff)
Added by Youichi Kimura about 6 years ago

use opZendMailTransportMock for CommunityMemberTable::processJoinConfirm() tests (fixes #3443)

History

#1 Updated by Youichi Kimura about 6 years ago

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

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

#2 Updated by Shinichi Urabe about 6 years ago

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

レビューOKです

#3 Updated by Youichi Kimura about 6 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 50

CommunityMemberTableTest にもメール送信を伴うテストが存在していたため一旦差し戻しにします

#4 Updated by Youichi Kimura about 6 years ago

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

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

#5 Updated by Shinichi Urabe about 6 years ago

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

追加修正を確認しました。
レビューOKです

#7 Updated by kaoru n over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0
  • % Done changed from 70 to 0

OpenPNE 3.8.9 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF