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

tests to be added #9

Open
2 of 5 tasks
vtlim opened this issue Jan 6, 2018 · 2 comments
Open
2 of 5 tasks

tests to be added #9

vtlim opened this issue Jan 6, 2018 · 2 comments

Comments

@vtlim
Copy link
Owner

vtlim commented Jan 6, 2018

Ongoing list of tests to be added:

  • diff mols having the same name (what happens if all mols have the same name? #22)
  • executor.py
  • what happens if Psi4 job has multiple calculations in single output file
  • (later) test for getTurbResults fail on datetime checking but passes locally. updated pckgs but no cigar.

There are also some placeholder functions in existing test scripts that must be filled in.

  • (later) meta-testing detail, remove temp headers to address ModuleImportError with Travis-CI once starting to use with setup.py (getPsiResults, getTurbResults)
@vtlim vtlim closed this as completed Oct 12, 2018
@vtlim vtlim reopened this Oct 20, 2018
@vtlim vtlim changed the title add tests, code coverage tests to be added Oct 20, 2018
@vtlim
Copy link
Owner Author

vtlim commented Jan 9, 2019

maybe move helper script as done here referencing here

@vtlim
Copy link
Owner Author

vtlim commented Apr 11, 2019

test for omega failing with smiles: CCCF(F)F alkyl_70

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant