test-reporter icon indicating copy to clipboard operation
test-reporter copied to clipboard

Published 20 hours ago • verified publisher icon codeclimate

Bug/add prefix

Open the-kwisatz-haderach opened this issue 2 years ago • 2 comments

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

  • Add prefix before loading source file.
  • Update tests.
  • Add some logging.

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

the-kwisatz-haderach avatar Dec 27 '22 10:12 the-kwisatz-haderach

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Dec 27 '22 10:12 CLAassistant

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

jdalrymple avatar May 12 '23 22:05 jdalrymple