-
Notifications
You must be signed in to change notification settings - Fork 437
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
Test Directory Cleanup #2655
Test Directory Cleanup #2655
Conversation
For every duplicate test, I want to know where the duplicate is. |
|
Okay. Please rename |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looking good, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks very much, including improving the tester script. Nice negative LOC!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm. Merge conflicts need to be addressed before PR can continue.
Merge conflicts have been fixed. |
@loriab Is there a hold on merging this? |
No hold; merge away. |
Description
Cleanup of tests directory following #2555. Heals various typos preventing some tests from getting pytest registration as well as some discrepancies between ctest and pytest labels. Deletes many unused/broken tests.
Todos
cc5
,scf11-freq-from-energies
,dft-pbe0-2
,dft-dsd
,dft-dldf
are all unused duplicates of other tests. Deleted.min-input
Broken + unused. Deleted.fsapt-diff1
Unused + not a test. Deleted.dfmp2-freq1
,dfmp2-freq2
,props4
were unused but are passing, quick, and useful. These are saved and added toCMakeLists.txt
.tester.py
no longer erroneously complains about ecppint tests having mismatched labels.Checklist
Status