Project

General

Profile

Backport（バックポート） #3967

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

Added by Youichi Kimura about 8 years ago. Updated over 4 years ago.

Status:
Won't fix（対応せず）
Priority:
Normal（通常）
Assignee:
Youichi Kimura
Target version:
OpenPNE 3.6.x
Start date:
2016-06-27
Due date:
% Done:

0%

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 8 years ago

#2 Updated by Youichi Kimura about 8 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 8 years ago

#4 Updated by kaoru n about 8 years ago

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

#5 Updated by Shinichi Urabe almost 8 years ago

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

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

#7 Updated by Youichi Kimura over 7 years ago

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

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

#8 Updated by Shinichi Urabe over 7 years ago

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

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

#9 Updated by Youichi Kimura over 7 years ago

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

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

#10 Updated by Shinichi Urabe over 7 years ago

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

#11 Updated by kaoru n about 7 years ago

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

#12 Updated by kaoru n about 7 years ago

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

#13 Updated by kaoru n almost 7 years ago

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

#14 Updated by Chiharu Nakajima almost 7 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 7 years ago

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

#16 Updated by isao sano over 4 years ago

  • Status changed from Rejected（差し戻し） to Won't fix（対応せず）
  • % Done changed from 50 to 0

3.6メンテナンス終了のため、対応せずとします

Also available in: Atom PDF