操作
Backport（バックポート） #3967
完了
壊れたテストコードを削除する
開始日:
2016-06-27
期日:
進捗率:
0%
予定工数:
説明
概要¶
現在 OpenPNE3 のテストコードは十分にメンテナンスされておらず、壊れた状態のテストケースが多数存在する。
このような状態は CI などにおけるテストコードの自動実行を妨げるだけでなく、退行バグによって既存のテストケースでエラーが発生しても他の大量のエラーに埋もれて発見できないなど、テストコードを十分に活用することができない。
過去にこのような壊れたテストコードを全て修正する試みがあった (#3403) が、エラーとなるテストケースがあまりに多いため「成功か否かに関わらず全てのテストケースが実行される」(Fatal Error による異常終了がない) 状態まで修正するに留まっている。
当チケットでは、壊れたテストコードをすべて削除し ./symfony test:all によって実行されるすべてのテストコードが正常に実行される状態にする。
Youichi Kimura さんが8年以上前に更新
- ステータス を New（新規） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
下記 Pull Request にて修正しました
https://github.com/openpne/OpenPNE3/pull/341
Youichi Kimura さんが8年以上前に更新
- ブロック先 Backport（バックポート） #3970: #3967で削除したテストコードを動作する状態にして再び追加する を追加
Youichi Kimura さんがほぼ8年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
#3962-10 と同様の修正を行いました。再度レビューをお願いします
https://github.com/openpne/OpenPNE3/pull/341
Shinichi Urabe さんがほぼ8年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
#3962-11 と同様の理由で差し戻し
Youichi Kimura さんが7年以上前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
修正のコミットが漏れていたたためpushしました。再度レビューをお願いします
Shinichi Urabe さんが7年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
Chiharu Nakajima さんがほぼ7年前に更新
- ステータス を Pending Testing（テスト待ち） から Rejected（差し戻し） に変更
- 進捗率 を 70 から 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.
操作