Closed wehs7661 closed 7 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 98.38%. Comparing base (
27320da
) to head (5175648
). Report is 21 commits behind head on master.:exclamation: Current head 5175648 differs from pull request most recent head 7faf9cf. Consider uploading reports for the commit 7faf9cf to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The currently implemented unit tests could trigger 21 warnings when performed. 18 of them are either DeprecationWarning
or MatplotlibDeprecation Warning
, which are harmless for the short term. The other 3 are RuntimeWarning
:
_df_combine_adjacent
in analyze_free_energy.py
. They are harmless.traj2transmtx
in analyze_traj.py
. It is also harmless.For more details please check the changes made here.
Now the code coverage has been enhanced to 96%. However, we want to further refine the tests by accomplishing the following tasks, as listed in #41.
- [ ] Re-examine previously written tests to see if there are better ways to test the functions. Presumably a lot of tests could be made more rigorous/meaningful using unittest.mock. test_analyze_traj.py should be free from this since most of its tests have just been written recently.UserWarning
tomsm_analysis.py
to reflect the fact that the scriptmsm_analysis.py
is still a work in progress and we are not planning to expand or test the module in the short term.Update: We have deferred the following task and noted it Issue #50