Ticket #11832 (closed: fixed)

Opened 5 years ago

Last modified 5 years ago

measure code coverage

Reported by: Steven Hahn Owned by: Steven Hahn
Priority: major Milestone: Release 3.5
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Pete Peterson

Description

Attempt to measure C++ code coverage in Mantid using gcov. To filter out the generated CxxTest files, I plan to use ​https://github.com/JoakimSoderberg/coveralls-cmake and submit the resulting json file to coveralls.io

Change History

comment:1 Changed 5 years ago by Steven Hahn

I have this working on Kernel. Next, expanding this to the rest of the framework.

comment:2 Changed 5 years ago by Steven Hahn

  • Status changed from new to inprogress

Refs #11832. Kernel coverage.

Changeset: f7c53754d3eed0af6620abc86ca7d1da544c3cfc

comment:3 Changed 5 years ago by Steven Hahn

Refs #11832. All of framework except SINQ covered.

Changeset: cf49aefe016849756cc4523f75529a15390070db

comment:4 Changed 5 years ago by Steven Hahn

Refs #11832. Trying to fix coveralls files.

Changeset: c9d027bc35adaf482d71b265fc81ae053708f159

comment:5 Changed 5 years ago by Steven Hahn

Refs #11832. Use python to generate json file.

Changeset: e67976833d27c4b88f92e63b997283fb00dd5492

comment:6 Changed 5 years ago by Steven Hahn

Refs #11832. Cleanup whitespace.

Changeset: 714b1720fad45144c8d6b2bdcdffa26eb7964a66

comment:7 Changed 5 years ago by Steven Hahn

Refs #11832. Enable slow parts of build.

Changeset: c626145cf005640f0686ec1c9226d51427a264fa

comment:8 Changed 5 years ago by Steven Hahn

  • Status changed from inprogress to verify
  • Resolution set to fixed

This is being verified as ​pull request #799.

comment:9 Changed 5 years ago by Steven Hahn

I'm trying this on a fork of the mantid repository. We can switch it over to the main repository once coveralls is setup for the MantidProject organization.

​http://builds.mantidproject.org/view/Static Analysis/job/coveralls_build_and_submit

comment:10 Changed 5 years ago by Pete Peterson

  • Status changed from verify to verifying
  • Tester set to Pete Peterson

comment:11 Changed 5 years ago by Pete Peterson

The failed doc tests are a red herring. This looks good and the job can be switched to point at the real repository.

comment:12 Changed 5 years ago by Pete Peterson

  • Status changed from verifying to closed

Merge pull request #799 from mantidproject/11832_code_coverage

quantify code coverage

Full changeset: ​d10a7b3f2941d06b4a6d69eac9d16fb5ae1edd80

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​12670

Note: See TracTickets for help on using tickets.