pyMBE-dev / pyMBE

pyMBE provides tools to facilitate building up molecules with complex architectures in the Molecular Dynamics software ESPResSo. For an up-to-date API documention please check our website:
https://pymbe-dev.github.io/pyMBE/pyMBE.html
GNU General Public License v3.0
5 stars 7 forks source link

Improve the code coverage of pyMBE #58

Open pm-blanco opened 1 month ago

pm-blanco commented 1 month ago

Here is the list of the features that require our attention:

jngrad commented 1 month ago

Outcome of the pyMBE online meeting: address the assigned tasks in a best effort manner by June 1, 2024. If you don't get all tasks done by then, open the PR anyway with the unit tests you already have. The goal is to have about 80% code coverage, so that we can be confident we are not breaking any feature while doing the big refactoring (#13) in June.

Later this year, we might organize a pyMBE coding day to take care of the remaining unit tests.

jngrad commented 1 month ago

To everyone: code coverage is now available on the main branch. Be sure to start your work on top of the most recent commit of the main branch to reduce the risk of introducing merge conflicts. You can use np.testing.assert_allclose() or pd.testing.assert_frame_equal() checks, or use unittest assertions to make the unit tests more structured and expressive (testsuite/serialization_test.py can be used as template).