Enhancement(機能追加・改善) #3962
open
Added by Youichi Kimura over 8 years ago.
Updated over 4 years ago.
Status:
Pending Testing(テスト待ち)
Estimated time:
(Total: 0:00 h)
Description
現在 OpenPNE3 のテストコードは十分にメンテナンスされておらず、壊れた状態のテストケースが多数存在する。
このような状態は CI などにおけるテストコードの自動実行を妨げるだけでなく、退行バグによって既存のテストケースでエラーが発生しても他の大量のエラーに埋もれて発見できないなど、テストコードを十分に活用することができない。
過去にこのような壊れたテストコードを全て修正する試みがあった (#3403) が、エラーとなるテストケースがあまりに多いため「成功か否かに関わらず全てのテストケースが実行される」(Fatal Error による異常終了がない) 状態まで修正するに留まっている。
当チケットでは、壊れたテストコードをすべて削除し ./symfony test:all
によって実行されるすべてのテストコードが正常に実行される状態にする。
Files
- Tracker changed from Task(タスク) to Enhancement(機能追加・改善)
- Status changed from New(新規) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
手元の環境で動作確認を行った感じ、 symfony test:all
で通過しないケースがあるようです (添付)
- Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
#3962-8 について修正しました。再度レビューをお願いします
https://github.com/openpne/OpenPNE3/pull/339
なお、以下の点については修正していません:
- pc_frontend の communityActionsTest, memberActionsTest のエラーについては、終了コードが 255 (Fatal Error) であり、特にテスト数の多いファイルである事からおそらくメモリリミットによるエラーによるものです
- opDoctrineQueryTest のエラーについては APC または APCu 拡張がインストールされていない事によるものです
- Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
Youichi Kimura さんは書きました:
#3962-8 について修正しました。再度レビューをお願いします
https://github.com/openpne/OpenPNE3/pull/339
こちら追加修正分が見当たりませんでした (最後のコミットが Tue Jul 5 17:15:40 2016 +0900 なので、 #3962-4 から追加は push できていないのかもしれません)
- pc_frontend の communityActionsTest, memberActionsTest のエラーについては、終了コードが 255 (Fatal Error) であり、特にテスト数の多いファイルである事からおそらくメモリリミットによるエラーによるものです
単独でメモリリミットをあげて実施したところ、正常動作しましたので、問題ないです
- opDoctrineQueryTest のエラーについては APC または APCu 拡張がインストールされていない事によるものです
こちらも APC があることで正常動作しました、問題ないです
- Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
修正のコミットが漏れていたたためpushしました。再度レビューをお願いします
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
- Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
- Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0
- % Done changed from 70 to 50
対象バージョン変更により修正内容の確認が必要であるため差し戻します。
- Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
- Target version changed from OpenPNE 3.9.0 to OpenPNE 3.10.x
- Target version changed from OpenPNE 3.10.x to OpenPNE 3.11.x
Also available in: Atom
PDF