Codecov によるコードカバレッジを追加

[nanasess]

概要(Overview・Refs Issue)

従来、 Coveralls でカバレッジを計測していたが、 Travis 渋滞が発生するため活用されていない。
最近は Codecov.io が主流になってきているようなので、 Codecov によるカバレッジを追加する

方針(Policy)

• phpdbg で PHPUnit を実行し、カバレッジレポートを出力する
• カバレッジレポートを codecov へアップロードする
• 既存のテストとは並列実行するため、渋滞は発生しにくい

実装に関する補足(Appendix)

• 大量のメモリを消費するため、 -dmemory_limit=-1 を付与する
• PHPUnit を実行する場合は continue-on-error: true を付与して、テストがエラーになっても無視する
• キャッシュクリアを伴うテストはメモリリークが発生するため実行しない

テスト（Test)

個人リポジトリでカバレッジ計測できるのを確認 https://codecov.io/gh/nanasess/ec-cube/tree/816bc627cb4307de3105ac3a11f8634d49ceb090/src/Eccube

相談（Discussion）

• 分割したカバレッジレポートの XML ファイルも簡単に統合してくれるので、 E2Eテストのカバレッジレポートも統合したい

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[codecov-io]

Codecov Report

❗ No coverage uploaded for pull request base (4.0@76e63a7). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             4.0    #4943   +/-   ##
======================================
  Coverage       ?   76.14%           
  Complexity     ?     6105           
======================================
  Files          ?      437           
  Lines          ?    20697           
  Branches       ?        0           
======================================
  Hits           ?    15759           
  Misses         ?     4938           
  Partials       ?        0           

Flag	Coverage Δ	Complexity Δ
tests	76.14% <0.00%> (?)	0.00% <0.00%> (?%)

Flags with carried forward coverage won't be shown. Click here to find out more.

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 76e63a7...886aff7. Read the comment docs.

[okazy]

個人リポジトリでテスト中
okazy#70

バッジの表示は unknown ですがマージされてデータができれば問題ないかと思います。

[okazy]

個人リポジトリにて動作を確認しました。
カバレッジを確認できました。
コードレベルでも視覚的に確認が可能でわかりやすいです。

バッジの表示は unknown の謎は解明できず
GitHub Actions の場合は Upload Token いらないみたいなのでよくわかりません。
マージして unknown のままならまた改善します。

[okazy]

GitHub Actions の設定はこちらで確認しました。
https://github.com/codecov/codecov-action

[okazy]

出ました！
デフォルトブランチの設定が関係していたようです。

https://app.codecov.io/gh/okazy/ec-cube

[okazy]

ありがとうございます！マージしました。