Enhancement（機能追加・改善） #3962
未完了
壊れたテストコードを削除する
70%
説明
概要¶
現在 OpenPNE3 のテストコードは十分にメンテナンスされておらず、壊れた状態のテストケースが多数存在する。
このような状態は CI などにおけるテストコードの自動実行を妨げるだけでなく、退行バグによって既存のテストケースでエラーが発生しても他の大量のエラーに埋もれて発見できないなど、テストコードを十分に活用することができない。
過去にこのような壊れたテストコードを全て修正する試みがあった (#3403) が、エラーとなるテストケースがあまりに多いため「成功か否かに関わらず全てのテストケースが実行される」(Fatal Error による異常終了がない) 状態まで修正するに留まっている。
当チケットでは、壊れたテストコードをすべて削除し ./symfony test:all によって実行されるすべてのテストコードが正常に実行される状態にする。
ファイル
Youichi Kimura さんが8年以上前に更新
- ステータス を New（新規） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
下記 Pull Request にて修正しました。
https://github.com/openpne/OpenPNE3/pull/339
Youichi Kimura さんが8年以上前に更新
- 次のチケットに先行 Enhancement（機能追加・改善） #3969: #3962で削除したテストコードを動作する状態にして再び追加する を追加
Youichi Kimura さんが8年以上前に更新
- 次のチケットに先行 を削除 (Enhancement（機能追加・改善） #3969: #3962で削除したテストコードを動作する状態にして再び追加する)
Youichi Kimura さんが8年以上前に更新
- ブロック先 Enhancement（機能追加・改善） #3969: #3962で削除したテストコードを動作する状態にして再び追加する を追加
Shinichi Urabe さんが約8年前に更新
- ファイル result.log result.log を追加
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
手元の環境で動作確認を行った感じ、 symfony test:all で通過しないケースがあるようです (添付)
Youichi Kimura さんがほぼ8年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
#3962-8 について修正しました。再度レビューをお願いします
https://github.com/openpne/OpenPNE3/pull/339
なお、以下の点については修正していません:
- pc_frontend の communityActionsTest, memberActionsTest のエラーについては、終了コードが 255 (Fatal Error) であり、特にテスト数の多いファイルである事からおそらくメモリリミットによるエラーによるものです
- opDoctrineQueryTest のエラーについては APC または APCu 拡張がインストールされていない事によるものです
Shinichi Urabe さんがほぼ8年前に更新
- ステータス を Pending Review（レビュー待ち） から 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 があることで正常動作しました、問題ないです
Youichi Kimura さんが7年以上前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
修正のコミットが漏れていたたためpushしました。再度レビューをお願いします
Shinichi Urabe さんが7年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
Youichi Kimura さんが7年以上前に更新
- ステータス を Rejected（差し戻し） から 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
Shinichi Urabe さんが7年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更