mi-kas / kover-report Goto Github PK

My automated build just started failing, and I'm at a loss as to what to do with this error:

Run mi-kas/[email protected]
with:
path: /home/runner/work/<repo>/<repo>/build/jacoco/jacoco_shared_result.xml
token: ***
title: Code Coverage
update-comment: true
env:
JAVA_HOME: /opt/hostedtoolcache/Java_Corretto_jdk/11.0.17.8.1/x64
JAVA_HOME_11_X64: /opt/hostedtoolcache/Java_Corretto_jdk/11.0.17.8.1/x64
Event is pull_request

Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

/home/runner/work/_actions/mi-kas/kover-report/v1.1/webpack:/kover-report/lib/reader.js:45
const lineCounter = (_a = counters.find(counter => counter['$'].type === 'LINE')) === null || _a === void 0 ? void 0 : _a['$'];
^
TypeError: Cannot read properties of undefined (reading 'find')
at getCoverageFromCounters (/home/runner/work/_actions/mi-kas/kover-report/v1.1/webpack:/kover-report/lib/reader.js:45:1)
at /home/runner/work/_actions/mi-kas/kover-report/v1.1/webpack:/kover-report/lib/reader.js:75:1
at Array.map (<anonymous>)
at /home/runner/work/_actions/mi-kas/kover-report/v1.1/webpack:/kover-report/lib/reader.js:68:1
at Array.reduce (<anonymous>)
at getFileCoverage (/home/runner/work/_actions/mi-kas/kover-report/v1.1/webpack:/kover-report/lib/reader.js:66:1)
at /home/runner/work/_actions/mi-kas/kover-report/v1.1/webpack:/kover-report/lib/action.js:47:1
at Generator.next (<anonymous>)
at fulfilled (/home/runner/work/_actions/mi-kas/kover-report/v1.1/webpack:/kover-report/lib/action.js:5:1)
at processTicksAndRejections (node:internal/process/task_queues:96:5)

Running into this issue and wondering if any recent changes has caused this.

My action is available here

HttpError: Resource not accessible by integration
    at /home/runner/work/_actions/mi-kas/kover-report/v1/webpack:/kover-report/node_modules/@octokit/request/dist-node/index.js:86:1
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Assuming a simple data class Foo(val a: Int), and this class is used in some tests, the coverage of this class will be 100% of lines, but it will report NaN% on branch coverage. While I'm ~ok to display "NaN%", I'm bothered by the red cross next to it. Given we have at least 1 line covered in a code which has no branch, I guess we should either discard those files from the reporting or consider them 100% covered anyway.

Maybe document required permissions, for the action to work?

I feel like we can avoid some friction like that, when setting up the action 🙂

permissions:          
  contents: read      
  pull-requests: write

Im new to code coverage, so at first i am hust using the example workflow and getting build error: "Input required and not supplied: distribution".

What can i do, to solve this issue?

Total coverage seems to be incorrect (higher than expected) compared to the coverage of individual files sometimes.
Would you please clarify what the reason might be?

I have a multi-module project and would like to publish each individual coverage report without having to manually specify the file. However, if I use a glob format, I get the error:
Error: ENOENT: no such file or directory, open '**/reports/kover/report.xml'

Ideally this would work similarly to other test reporting actions (I am currently using dorny/test-reporter@v1, for example).

I realize I could use a merged report, but I was hoping to avoid that in this case.

GitHub severely limits what access the default GITHUB_TOKEN for a PR originating from a fork has, which prevents the default configuration of this action from successfully posting the report as a PR comment. You can see an example failure here.

The README should be updated so that the example also works for this scenario.

Github Actions YAML usually isn't as useful as it could be.

krzema12/github-actions-typing

This project setup a way to define types for github actions inputs to build a type-safe Kotlin DSL.

I have tried to use this action to publish the coverage with kover 0.7.3, but even if the step doesn't fail I can't see the message after the commits.

my pull request
my workflow

In the Action terminal, the output is

Run mi-kas/kover-report@v1
with:
path: /home/runner/work/CheatMultiplayer/CheatMultiplayer/model/build/reports/kover/report.xml
token: ***
title: Code Coverage
update-comment: false
min-coverage-overall: 80
min-coverage-changed-files: 80
coverage-counter-type: LINE
env:
JAVA_HOME: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/17.0.9-9/x64
JAVA_HOME_17_X64: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/17.0.9-9/x64
Event is push

The XML report that is output shows a lot more data than is currently in the comment.

In my workflow, I have it print out the entire XML report, which you can see here: https://github.com/paypal/paypal-messages-android/actions/runs/7391501463/job/20108408639?pr=16#step:5:8

But the comment only shows 4 classes, which you can see here: paypal/paypal-messages-android#16 (comment)

How can I make the coverage table show all the classes?

Using kover-report raises this error:

Are you considering to update?

Some time ago this commit was merged: f795c03
It would be cool to have it released, so that it's available under v1 tag 😄 Could you please prepare a release?

It's quite a problem for me that I can't trigger a workflow "by hand" even on a PR. What is the rationale for disallowing this?

Seems like the "total project coverage" currently shows the class coverage.
It would be nice if it could be configured e.g. to show branch coverage.

See https://docs.github.com/de/actions/using-workflows/workflow-commands-for-github-actions#adding-a-job-summary

Hi!
Is it possible to have an output with changed file to coverage map? I would like to know if every single (changed) file hits min test coverage.

I have a project that has kover 0.7.3 setup using exclusions. I'm getting different values on the kover report that I run manually vs the results on CI, and I detected that the exclusions are not accounted on the CI run

Hello,
I would like to set up this Github action for my repository, but I get an error:

Run mi-kas/[email protected]
Event is push
HttpError: Not Found
    at /runner/_work/_actions/mi-kas/kover-report/v1.5/webpack:/kover-report/node_modules/@octokit/request/dist-node/index.js:86:1
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Obviously it's not possible to getChangedFiles and I am wondering what can be the cause or how to debug it? I think token should have required permissions. The PR contains only version change for kover-report from v1 to v1.5.