lsmo-epfl / aiida-lsmo

AiiDA workflows for the LSMO laboratory at EPFL
Other
9 stars 13 forks source link

improve test coverage #48

Closed ltalirz closed 3 years ago

ltalirz commented 3 years ago

Only about 30% of lines are currently covered by CI tests https://codecov.io/gh/lsmo-epfl/aiida-lsmo/list/3ad5d629a216857cce643c32401bf9210229a56e/aiida_lsmo

image

If it stays like this for long, code is guaranteed to break

ltalirz commented 3 years ago

The situation has improved a bit now - there's still workchains missing but moving an examples/run_... file to an examples/test_... file (and thus making the workchain tested) is now quite straightforward (see existing examples).

Next on the to-do list are

image

ltalirz commented 3 years ago

Code coverage is now at ~78%.

image

@danieleongari There's still a couple of workchains that should be tested, but from here on it is easy to just copy-paste what has been done for the other workchains (and other people can contribute tests as well).

Closing.