search

the-virtual-brain / tvb-gdist

Geodesic Library (adaptation by TVB Team)
Other
48 stars 30 forks source link

Improve Code Coverage #57

Closed ayan-b closed 4 years ago

ayan-b commented 4 years ago

We have a couple of lines out of coverage: https://codecov.io/gh/the-virtual-brain/tvb-gdist/src/f19b70e431867bebee8693b6ae807807b8e0c335/gdist.pyx.

liadomide commented 4 years ago

The latest coverage report seems to measure coverage only on gdist.pyx. Is this the case? If so, could we include also files from under geodesic_library ? geodesic_utils.h for example in #49 contains critical code and it will be interesting to know what lines are checked by the current unit-tests there.

ayan-b commented 4 years ago

The latest coverage report seems to measure coverage only on gdist.pyx. Is this the case? If so, could we include also files from under geodesic_library ? geodesic_utils.h for example in #49 contains critical code and it will be interesting to know what lines are checked by the current unit-tests there.

That will require a cpptest framework. Google test seems to be the most popular one.