Enhancement（機能追加・改善） #3962
壊れたテストコードを削除する
70%
Description
概要¶
現在 OpenPNE3 のテストコードは十分にメンテナンスされておらず、壊れた状態のテストケースが多数存在する。
このような状態は CI などにおけるテストコードの自動実行を妨げるだけでなく、退行バグによって既存のテストケースでエラーが発生しても他の大量のエラーに埋もれて発見できないなど、テストコードを十分に活用することができない。
過去にこのような壊れたテストコードを全て修正する試みがあった (#3403) が、エラーとなるテストケースがあまりに多いため「成功か否かに関わらず全てのテストケースが実行される」(Fatal Error による異常終了がない) 状態まで修正するに留まっている。
当チケットでは、壊れたテストコードをすべて削除し ./symfony test:all によって実行されるすべてのテストコードが正常に実行される状態にする。
Subtasks
Related issues
History
#1
Updated by Youichi Kimura about 7 years ago
Updated by - Tracker changed from Task（タスク） to Enhancement（機能追加・改善）
#2
Updated by Youichi Kimura about 7 years ago
- Related to Backport（バックポート） #3967: 壊れたテストコードを削除する added
#3
Updated by Youichi Kimura about 7 years ago
- Related to Backport（バックポート） #3968: 壊れたテストコードを削除する added
#4
Updated by Youichi Kimura about 7 years ago
- Status changed from New（新規） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
下記 Pull Request にて修正しました。
https://github.com/openpne/OpenPNE3/pull/339
#5
Updated by Youichi Kimura about 7 years ago
- Precedes Enhancement（機能追加・改善） #3969: #3962で削除したテストコードを動作する状態にして再び追加する added
#6
Updated by Youichi Kimura about 7 years ago
- Precedes deleted (Enhancement（機能追加・改善） #3969: #3962で削除したテストコードを動作する状態にして再び追加する)
#7
Updated by Youichi Kimura about 7 years ago
- Blocks Enhancement（機能追加・改善） #3969: #3962で削除したテストコードを動作する状態にして再び追加する added
#8
Updated by Shinichi Urabe almost 7 years ago
Updated by - File result.log View added
- Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
手元の環境で動作確認を行った感じ、 symfony test:all で通過しないケースがあるようです (添付)
#10
Updated by Youichi Kimura over 6 years ago
- 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 拡張がインストールされていない事によるものです
#11
Updated by Shinichi Urabe over 6 years ago
- 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 があることで正常動作しました、問題ないです
#12
Updated by Youichi Kimura over 6 years ago
- Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
修正のコミットが漏れていたたためpushしました。再度レビューをお願いします
#13
Updated by Shinichi Urabe over 6 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
#14
Updated by kaoru n over 6 years ago
Updated by - 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
対象バージョン変更により修正内容の確認が必要であるため差し戻します。
#15
Updated by Youichi Kimura over 6 years ago
- Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
新しい master ブランチ宛に Pull Request を作成しました。レビューをお願いします
https://github.com/openpne/OpenPNE3/pull/461
Pull Request の表示では Travis CI がエラーとなっていますが、これは master ブランチにマージした後の状態を表しており PHP7 対応で新たに“壊れた”箇所です。（今後バグチケットを起票して修正します）
https://travis-ci.org/openpne/OpenPNE3/builds/217938200
マージ前の状態では Travis CI によるテストは正常に終了しています。
https://travis-ci.org/upsilon/OpenPNE3/builds/217938188
#16
Updated by Shinichi Urabe over 6 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
#17
Updated by kaoru n over 3 years ago
- Target version changed from OpenPNE 3.9.0 to OpenPNE 3.10.x
#18
Updated by kaoru n about 3 years ago
- Target version changed from OpenPNE 3.10.x to OpenPNE 3.11.x