Heatmap link broken

Support
1

Description

In the heatmap, the link generated that I want to view coverage decrease is Search/Filter.php

image
image1350×900 53.1 KB

When I click the link, it takes me to the Files tab with an empty page (since i’m a new user I can’t post more than one screenshot :roll_eyes:).

If i manually click the Files tab it shows me File path not found. Did you mean one of the paths below and one option is lib/Search/Filter.php

There is no file Search/Filter.php in my source code, but there is a file lib/Search/Filter.php so why does the heatmap generate incorrect links?

Repository

A link to the repository in question (or private)

Private repo

CI/CD

Please provide which CI/CD you are using

Github actions

Uploader

Please provide which uploader you are using and command used to run it. If you are not using the bash uploader, we recommend switching to it before opening an issue here.

codecov/codecov-action@v1

Commit SHAs

Please include the commit SHA(s)

Codecov YAML

Please include the codecov.yml as code

Not using one

Codecov Output

Please provide the full output of running the uploader on your CI/CD

Additional Information

Any additional information, configuration or data that might be necessary to reproduce the issue.

2

Hi @toymachiner62 this is most likely a path fixing issue. My guess is you might need something like this in your codecov.yml

fixes:
  - "::lib/"

Let me know if that fixes your issue.

3

I already merged that PR and I forgot this repo is public. Here’s the PR https://github.com/catalyticlabs/catalytic-sdk-php/pull/43

I tried to open another PR to check if your suggestion fixed it, but I had to make some code changes.

I have another PR in a different repo with the same behavior and your suggestion doesn’t seem to be fixing it, unless i’m doing something wrong.

Here’s a PR and if you click the Codecov report and click on the leftmost box clients/Instances.java you can see the empty Files tab. What should the fix be for this particular PR? (I won’t touch it until I hear back from you).

4

@toymachiner62 can you link me to the PR with the codecov.yml changes?

5

Sorry I forgot the link https://github.com/catalyticlabs/catalytic-sdk-java/pull/36

6

Hi @toymachiner62, I made a quick suggestion in your PR. I THINK that will fix it, but let me know either way.

7

I made the suggested change, but it didn’t seem to do anything.

8

@toymachiner62, looks like this is an issue on our side, I have made a request to our product team.

9

Thanks for letting me know. I’m going to merge this PR as is then. If you could let me know when it’s fixed on your end I can create a new PR and try again.

1 Like
11

@tom any update on this?

12

Hi @toymachiner62, we have not forgotten about this issue and still have it in our system. Thanks for checking back.

1 Like
13

Hi @toymachiner62, just an update that we are still taking a look at this. Thanks for your patience here.

14

Hi @toymachiner62, thanks again for your patience here. I haven’t forgotten about this issue, and it is still on our roadmap.

15

Hey @tom - any update on this? It’s been about 6 months.

16

Hi @toymachiner62, apologies on this. We are actually in the middle of re-vamping our UI and that entire page. This particular issue is expected to be resolved in that update. We expect this page to be pushed middle to end of Q1 next year. Thanks for your incredible patience on this issue.

1 Like