Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Source Not Available" Error on Coveralls #1448

Closed
drsanguin opened this issue Jan 6, 2024 · 0 comments · Fixed by #1453
Closed

"Source Not Available" Error on Coveralls #1448

drsanguin opened this issue Jan 6, 2024 · 0 comments · Fixed by #1453
Assignees

Comments

@drsanguin
Copy link
Contributor

Describe the bug
Looks like the coverage report sent to Coveralls fails to match with the GitHub files. Here is an example on my project.

To Reproduce

  1. Fetch my project
  2. Install Tarpaulin:
    cargo install cargo-tarpaulin
    
  3. Run tarpaulin on my project
    cargo tarpaulin --coveralls Z53FpI2gSTvnQw64wGeSx1aX0u5M1PEV5
    
    Don't be shy of pushing coverage to the coveralls project. I will regenerate the token later.
  4. Go to the repository page on coveralls to see the error

Please note that this will allow you to reproduce the issue on local. On my project, I send coverage report using this GitHub Action. Error is the same in both ways.

Expected behavior
The source on Coveralls should match the GitHub's ones.

Environment

  • OS: windows
  • Cargo: 1.72.0 (103a7ff2e 2023-08-15)
  • tarpaulin: 0.27.2

Additional intel

My two cents on the rootcause
Since I work on windows, I believe the report is send to Coveralls with the Windows path separator \ and Coveralls search files in the repo with the linux one /.

Thanks in advance for the help !

drsanguin added a commit to drsanguin/convertible_couch that referenced this issue Jan 6, 2024
drsanguin added a commit to drsanguin/tarpaulin that referenced this issue Jan 12, 2024
Coveralls expect source path to have the Linux path separator. Otherwise, it cannot link them with the repository's source.

Fixes xd009642#1448
drsanguin added a commit to drsanguin/tarpaulin that referenced this issue Jan 12, 2024
Coveralls expect source path to have the Linux path separator. Otherwise, it cannot link them with the repository's source.

Fixes xd009642#1448
drsanguin added a commit to drsanguin/tarpaulin that referenced this issue Jan 12, 2024
Coveralls expect source path to have the Linux path separator. Otherwise, it cannot link them with the repository's source.

Fixes xd009642#1448
xd009642 pushed a commit that referenced this issue Jan 13, 2024
Coveralls expect source path to have the Linux path separator. Otherwise, it cannot link them with the repository's source.

Fixes #1448
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants