Project

General

Profile

Backport(バックポート) #3967

壊れたテストコードを削除する

Added by Youichi Kimura about 3 years ago. Updated almost 2 years ago.

Status:
Rejected(差し戻し)
Priority:
Normal(通常)
Target version:
Start date:
2016-06-27
Due date:
% Done:

50%


Description

概要

現在 OpenPNE3 のテストコードは十分にメンテナンスされておらず、壊れた状態のテストケースが多数存在する。
このような状態は CI などにおけるテストコードの自動実行を妨げるだけでなく、退行バグによって既存のテストケースでエラーが発生しても他の大量のエラーに埋もれて発見できないなど、テストコードを十分に活用することができない。
過去にこのような壊れたテストコードを全て修正する試みがあった (#3403) が、エラーとなるテストケースがあまりに多いため「成功か否かに関わらず全てのテストケースが実行される」(Fatal Error による異常終了がない) 状態まで修正するに留まっている。
当チケットでは、壊れたテストコードをすべて削除し ./symfony test:all によって実行されるすべてのテストコードが正常に実行される状態にする。


Related issues

Related to OpenPNE 3 - Enhancement(機能追加・改善) #3962: 壊れたテストコードを削除する Pending Testing(テスト待ち) 2016-06-27
Blocks OpenPNE 3 - Backport(バックポート) #3970: #3967で削除したテストコードを動作する状態にして再び追加する New(新規) 2016-07-19

History

#1 Updated by Youichi Kimura about 3 years ago

#2 Updated by Youichi Kimura about 3 years ago

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

下記 Pull Request にて修正しました
https://github.com/openpne/OpenPNE3/pull/341

#3 Updated by Youichi Kimura about 3 years ago

#4 Updated by kaoru n about 3 years ago

  • Target version changed from OpenPNE 3.6.24 to OpenPNE 3.6.x

#5 Updated by Shinichi Urabe almost 3 years ago

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

バグチケットに沿い差し戻します

#7 Updated by Youichi Kimura over 2 years ago

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

#3962-10 と同様の修正を行いました。再度レビューをお願いします
https://github.com/openpne/OpenPNE3/pull/341

#8 Updated by Shinichi Urabe over 2 years ago

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

#3962-11 と同様の理由で差し戻し

#9 Updated by Youichi Kimura over 2 years ago

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

修正のコミットが漏れていたたためpushしました。再度レビューをお願いします

#10 Updated by Shinichi Urabe over 2 years ago

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

#11 Updated by kaoru n about 2 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.28

#12 Updated by kaoru n about 2 years ago

  • Target version changed from OpenPNE 3.6.28 to OpenPNE 3.6.x

#13 Updated by kaoru n almost 2 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.29

#14 Updated by Chiharu Nakajima almost 2 years ago

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

以下のエラーが発生します

unit/helper/opUtilHelperTest.........................................errors
Errors:
- sfException: Call to undefined method myUser::isSNSMember.
unit/model/doctrine/GadgetTest.......................................errors
Looks like you planned 12 tests but only ran 5.
Errors:
- sfException: Call to undefined method myUser::getMember.

#15 Updated by kaoru n almost 2 years ago

  • Target version changed from OpenPNE 3.6.29 to OpenPNE 3.6.x

Also available in: Atom PDF