Qiskit / qiskit / 14912826564
88%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 806
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 88.37% (+0.5%) from 87.824%
14912826564

push

github

web-flow 
Add ctest execution to coverage reporting (#14046)

* Add ctest execution to coverage reporting

This commit updates the coverage CI job to also include execution of the
C API tests. Right now the cext crate is showing 0 coverage because we
only test it without instrumention and don't collect any coverage
numbers for the C API testing. This commit adds a ctest execution to the
coverage job to provide coverage numbers for our C API testing.

* Install cbindgen in coverage job

* Run ctests manually

* Copy coverage data from ctests to grcov working dir

* Fix source path for profile data copy

* Don't build tests in release mode

* Add split path for debug vs release mode in Makefile

This commit updates the makefile organization to add a debug mode path
and a release mode path. The debug mode path is needed for running the
ctests, while release mode is what we want to use for building the clib
for direct use. With this the makefile is able to build in either
release mode or debug mode and the targets are setup so we default to
release mode unless debug is explicitly needed.

* Update CMakeLists.txt

* Apply suggestions from code review

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Copy header to expected dir

* Update Makefile

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Deduplicate header target in makefile

* Fix header generation for docs

* Update Makefile

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Build in debug mode explicitly for ctest makefile target

This commit reworks the makefile to minimize the changes. It changes the
ctest target to explicitly call cargo to build in debug mode and copy
the header explicitly rather than trying to split the internal logic
based on debug mode versus release mode.

* Revert tox.ini change

---------

Co-authored-by: Julien Gacon <gaconju@gmail.com>

75053 of 84930 relevant lines covered (88.37%)

434383.06 hits per line

Uncovered Existing Lines

Lines Coverage ∆ File
1
94.85
 0.0% crates/accelerate/src/unitary_synthesis.rs
3
92.23
 -0.5% crates/qasm2/src/lex.rs
Jobs
ID Job ID Ran Files Coverage
1 14912826564.1 806
88.37
 GitHub Action Run
Source Files on build 14912826564