プロジェクト

全般

プロフィール

Enhancement(機能追加・改善) #3962

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

Youichi Kimura2年以上前に追加. ほぼ2年前に更新.

ステータス:
Pending Testing(テスト待ち)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2016-06-27
期日:
進捗率:

70%


説明

概要

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

result.log 表示 (7.21 KB) Shinichi Urabe, 2016-10-27 18:55


関連するチケット

関連している OpenPNE 3 - Backport(バックポート) #3967: 壊れたテストコードを削除する Rejected(差し戻し) 2016-06-27
関連している OpenPNE 3 - Backport(バックポート) #3968: 壊れたテストコードを削除する Rejected(差し戻し) 2016-06-27
ブロック先 OpenPNE 3 - Enhancement(機能追加・改善) #3969: #3962で削除したテストコードを動作する状態にして再び追加する New(新規)

履歴

#1 Youichi Kimura2年以上前に更新

  • トラッカーTask(タスク) から Enhancement(機能追加・改善) に変更

#2 Youichi Kimura2年以上前に更新

#3 Youichi Kimura2年以上前に更新

#4 Youichi Kimura2年以上前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#5 Youichi Kimura2年以上前に更新

#6 Youichi Kimura2年以上前に更新

#7 Youichi Kimura2年以上前に更新

#8 Shinichi Urabe2年以上前に更新

  • ファイル result.log 表示 を追加
  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

手元の環境で動作確認を行った感じ、 symfony test:all で通過しないケースがあるようです (添付)

#10 Youichi Kimura約2年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

#3962-8 について修正しました。再度レビューをお願いします
https://github.com/openpne/OpenPNE3/pull/339

なお、以下の点については修正していません:

  • pc_frontend の communityActionsTest, memberActionsTest のエラーについては、終了コードが 255 (Fatal Error) であり、特にテスト数の多いファイルである事からおそらくメモリリミットによるエラーによるものです
  • opDoctrineQueryTest のエラーについては APC または APCu 拡張がインストールされていない事によるものです

#11 Shinichi Urabeほぼ2年前に更新

  • ステータス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 があることで正常動作しました、問題ないです

#12 Youichi Kimuraほぼ2年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

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

#13 Shinichi Urabeほぼ2年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#14 kaoru nishizoeほぼ2年前に更新

  • ステータスPending Testing(テスト待ち) から Rejected(差し戻し) に変更
  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更
  • 進捗率70 から 50 に変更

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

#15 Youichi Kimuraほぼ2年前に更新

  • ステータス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

#16 Shinichi Urabeほぼ2年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

他の形式にエクスポート: Atom PDF