Progaku-copy / progaku-archive

Progakuのアーカイブ用のアプリ
0 stars 1 forks source link

ブランチカバレッジの標準出力がsimplecov-html側で対応されていたので、gemをupdateする #91

Closed kakeru-one closed 2 months ago

kakeru-one commented 2 months ago

対応内容

- formatter SimpleCov::Formatter::BranchLineCovFormatter
+ SimpleCov.formatters = SimpleCov::Formatter::MultiFormatter.new([
+  SimpleCov::Formatter::HTMLFormatter,
+  SimpleCov::Formatter::BranchLineCovFormatter
+ ])

ブランチカバレッジの標準出力が対応されたPR

https://github.com/simplecov-ruby/simplecov-html/pull/103

kuri0616 commented 2 months ago

ブランチカバレッジの標準出力がsimplecov-html側で対応されていたので、gemをupdateする それとは別でformatterが競合してしまっていたのでその修正も含める 以下のように書く必要があり、今回の変更を加えるまではcoverage/index.htmlが出力されていない状態だった。(ただし、今回このフォーマッターは必要ないので、消した。)

修正ありがとうございます! 記載内容の意味としては

と言うことですかね?

kakeru-one commented 2 months ago

前回の対応でブランチカバレッジを出力するために下記のformatterを記載していたが、それによりformatterが競合し、デフォルトで出力されるHTMLが出力されなくなった。

合っています


リンクのPRでデフォルトでブランチカバレッジが出力されるようになっていたため、gemをUpdateし、その機能を反映。上記により、branchカバレッジのformatterも明示的に記載する必要もなくなったため削除した。

合っています!


今回は必要なかったがもし、複数のformatterに使用したい場合、MultiFormatterクラスのインスタンスを生成し、formatterを指定し、SimpleCovクラスのformattersプロパティに指定する必要があった。

合っています 💯