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.

Sign up for GitHub

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

Jump to bottom

Implement a limited number of high priority unit tests #13

Open
jackbaker1001 opened this issue Aug 11, 2023 · 2 comments
Open

Implement a limited number of high priority unit tests #13

jackbaker1001 opened this issue Aug 11, 2023 · 2 comments
Assignees
@jackbaker1001
Labels
enhancement New feature or request

Comments

@jackbaker1001
Copy link
Contributor

There are presently no unit tests at all in the repo. It is very likely that we will not have time to achieve high test coverage, but we should identify some key areas to test.

As I progress through understanding the code base internals, it will become more obvious which parts these are and we can discuss below.
@jackbaker1001 jackbaker1001 added the enhancement New feature or request label Aug 11, 2023
@jackbaker1001 jackbaker1001 self-assigned this Aug 11, 2023
@PabloAMC
Copy link
Collaborator

We may want to close this, or are there additional tests to be implemented?

@jackbaker1001
Copy link
Contributor Author

I think I will add some basic unit testing for Molecule and Functional classes. I think Integration testing is as good as it needs to be though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jackbaker1001 jackbaker1001

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@PabloAMC @jackbaker1001