search

codeclimate / test-reporter

Code Climate Test Reporter
MIT License
154 stars 76 forks source link

Bug/add prefix #506

Open the-kwisatz-haderach opened 1 year ago

the-kwisatz-haderach commented 1 year ago

My use case is running the cc-test-reporter from the project root whereas my coverage files for the frontend are located in ./frontend/test-coverage/lcov.info. I managed to get format-coverage to work only by first cd frontend. Which led me to believe the ADD_PREFIX flag isn't really working. At least for lcov files, because loading the source file is attempted before any prefix is added. And if adding a prefix is required then this will naturally fail. 🤔 Or maybe I'm missing something...

I'm reverting some changes made in this PR: https://github.com/codeclimate/test-reporter/pull/232

This bug is also mentioned here: https://github.com/codeclimate/test-reporter/issues/422#issuecomment-1242757968 Possibly but not necessarily related: https://github.com/codeclimate/test-reporter/issues/487

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

jdalrymple commented 1 year ago

Still running into this issue, whats the status of the PR?