Project

General

Profile

Actions

Backport(バックポート) #3968

open

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

Added by Youichi Kimura over 8 years ago. Updated almost 7 years ago.

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

50%

Estimated time:

Description

概要

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


Related issues 3 (3 open0 closed)

Related to OpenPNE 3 - Enhancement(機能追加・改善) #3962: 壊れたテストコードを削除するPending Testing(テスト待ち)Youichi Kimura2016-06-27

Actions
Blocks OpenPNE 3 - Backport(バックポート) #3971: #3968で削除したテストコードを動作する状態にして再び追加するNew(新規)2016-07-19

Actions
Copied to OpenPNE 3 - Backport(バックポート) #4382: 壊れたテストコードを削除するPending Review(レビュー待ち)isao sano2016-06-27

Actions
Actions #1

Updated by Youichi Kimura over 8 years ago

Actions #2

Updated by Youichi Kimura over 8 years ago

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

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

Actions #3

Updated by Youichi Kimura over 8 years ago

Actions #4

Updated by kaoru n about 8 years ago

  • Target version changed from OpenPNE 3.8.21 to OpenPNE 3.8.x
Actions #5

Updated by Shinichi Urabe about 8 years ago

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

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

Actions #7

Updated by Youichi Kimura almost 8 years ago

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

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

Actions #8

Updated by Shinichi Urabe almost 8 years ago

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

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

Actions #9

Updated by Youichi Kimura almost 8 years ago

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

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

Actions #10

Updated by Shinichi Urabe over 7 years ago

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

Updated by kaoru n about 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.25
Actions #12

Updated by kaoru n about 7 years ago

  • Target version changed from OpenPNE 3.8.25 to OpenPNE 3.8.x
Actions #13

Updated by kaoru n about 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.26
Actions #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.

Actions #15

Updated by kaoru n almost 7 years ago

  • Target version changed from OpenPNE 3.8.26 to OpenPNE 3.8.x
Actions #16

Updated by isao sano over 4 years ago

Actions

Also available in: Atom PDF