Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve existing unit tests and overall code coverage #7

Closed
4 of 10 tasks
wehs7661 opened this issue Mar 3, 2023 · 1 comment
Closed
4 of 10 tasks

Improve existing unit tests and overall code coverage #7

wehs7661 opened this issue Mar 3, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@wehs7661
Copy link
Owner

wehs7661 commented Mar 3, 2023

Before the next release (presumably 0.5.0), one of the action items is to improve the quality of existing unit tests and covered the untested functions. Here is a checklist for the modules to test. An item is checked only if the existing tests have been re-examined and new tests have been added (when necessary) to increase the code coverage.

  • ensemble_EXE.py (finished at commit 09adb37)
  • utils/utils.py (commits d83e5a)
  • utils/gmx_parser.py
  • analysis/analyze_matrix.py
  • analysis/analyze_traj.py
  • analysis/analyze_free_energy.py
  • analysis/msm_analysis.py

Notably, there should not be a need to cover the CLIs and utils/exceptions.py. We do need to re-examine the CLIs to make sure they are all sufficiently compartmentalized, so I list them below as well.

  • Compartmentalize run_EEXE.py
  • Compartmentalize analyze_EEXe.py
  • Compartmentalize explore_EEXE.py
@wehs7661 wehs7661 self-assigned this May 3, 2023
@wehs7661 wehs7661 added the enhancement New feature or request label May 3, 2023
@wehs7661 wehs7661 added the help wanted Extra attention is needed label Feb 25, 2024
@wehs7661
Copy link
Owner Author

wehs7661 commented Apr 9, 2024

Closing this issue now since some action items have become outdated, and the valid ones have been incorporated/restated in issue #41.

@wehs7661 wehs7661 closed this as completed Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant