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

move aiida-core tests to pytest #2771

Closed
2 tasks
ltalirz opened this issue Apr 17, 2019 · 0 comments · Fixed by #3653
Closed
2 tasks

move aiida-core tests to pytest #2771

ltalirz opened this issue Apr 17, 2019 · 0 comments · Fixed by #3653
Milestone

Comments

@ltalirz
Copy link
Member

ltalirz commented Apr 17, 2019

We are already recommending pytest for plugins because of generally increased usability & the potential for code reuse via fixtures.
At some point we should move aiida-core tests to pytest as well.

  • move all tests outside the aiida folder and inside a top-level tests folder with the same folder hierarchy. This will significantly reduce the size of the aiida package on PyPI. We can think whether it is then still necessary to rebuild the test hierarchy of verdi devel tests or whether it might be just simpler to do pytest tests/the/tests/I/want/to/run
  • get rid of the need for creating a "test profile" by defaulting to the dynamically created postgres cluster provided by pgtest. Potentially give users who don't want this overhead the option to stay with their system postgres cluster by fixing faster plugin tests #2648

as discussed with @sphuber

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

Successfully merging a pull request may close this issue.

1 participant