CloudRaider icon indicating copy to clipboard operation
CloudRaider copied to clipboard

Published 20 hours ago • verified publisher icon intuit

Include test coverage measurement & reporting tools

Open rattandeep1998 opened this issue 2 years ago • 4 comments

Add required tools to report and measure test coverage and put it back as label/hook on README.

rattandeep1998 avatar Oct 14 '22 16:10 rattandeep1998

I will take a look at this and explore on this. Please assign it to me.

vaisrinivasan avatar Oct 17 '22 10:10 vaisrinivasan

@rattandeep1998 : I see that Jacoco is already added in pom.xml. Can you please explain what other tool should be added as part of this issue? If nothing else is there to be added, there's currently an error coming while running the build and the jacoco report. I can fix the issues and then add the path to report in the README file. I can do the above changes as part of this PR. Let me know.

vaisrinivasan avatar Oct 19 '22 14:10 vaisrinivasan

@vaisrinivasan Thanks for the investigation. We are looking for a way to run the tests as a part of pipeline (more details under https://github.com/intuit/CloudRaider/issues/39) and add a hook for that CI test reports on the Readme. We might not have to push the entire jacoco report to remote.

rattandeep1998 avatar Oct 24 '22 19:10 rattandeep1998

@rattandeep1998 : Sure. I've removed the entire report. So as part of this PR, I've fixed a couple of issues while running the jacoco report. Since there's a separate issue for CI/CD + hook for the report, I think this PR is good to be merged and the issue closed? Let me know if there's anything else to be done as part of this PR. Thanks.

vaisrinivasan avatar Oct 26 '22 05:10 vaisrinivasan