-
Notifications
You must be signed in to change notification settings - Fork 177
Description
17/01/20 : Test coverage is about 70%.
Edit : 17/02/21 : 78%, getting there! 💪
Edit :: 12/11/21 : down to 75% :(
Edit : 21/03/22 : up to 77%
- Goal:: should be > 80%.
That's a good issue to learn about the package!
Identify modules that need the more coverage :
-
for instance a MergeSlabs with
intersect: https://codecov.io/gh/radis/radis/pull/91/src/radis/los/slabs.py#L464 -
in base.py there are the _add_EvibErot_CDSD_pc and _add_Evib123Erot_CDSD_pc function (advanced calculations for nonequilibrium CO2 in "p,c,j,n" nomenclature ) . See how the similar _add_EvibErot_CDSD_pcN function was tested in Add online tests for calculations per band, and CO2 (p,c,j,n) nonequilibrium modes #99
Clean deprecated parts of the code (ask on Gitter if needed)
Add new tests as needed
-
for instance move
_test()frommisc/signals.pyin the test folder. And run it. -
maybe exclude some parts from the coverage: see https://coverage.readthedocs.io/en/v4.5.x/excluding.html#advanced-exclusion