Project

General

Profile

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

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

Added by Youichi Kimura almost 3 years ago. Updated about 2 years ago.

Status:
Pending Testing(テスト待ち)
Priority:
Normal(通常)
Target version:
Start date:
2016-06-27
Due date:
% Done:

70%


Description

概要

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

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


Related issues

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

History

#1 Updated by Youichi Kimura almost 3 years ago

  • Tracker changed from Task(タスク) to Enhancement(機能追加・改善)

#2 Updated by Youichi Kimura almost 3 years ago

#3 Updated by Youichi Kimura almost 3 years ago

#4 Updated by Youichi Kimura almost 3 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 almost 3 years ago

#6 Updated by Youichi Kimura almost 3 years ago

#7 Updated by Youichi Kimura almost 3 years ago

#8 Updated by Shinichi Urabe over 2 years ago

  • File result.log View added
  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

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

#10 Updated by Youichi Kimura over 2 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 2 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 2 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

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

#13 Updated by Shinichi Urabe over 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#14 Updated by kaoru n about 2 years ago

  • 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 about 2 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 about 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

Also available in: Atom PDF