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

Testing: add testing data and integration tests #101

Closed
4 tasks
jsheunis opened this issue Jul 4, 2022 · 1 comment
Closed
4 tasks

Testing: add testing data and integration tests #101

jsheunis opened this issue Jul 4, 2022 · 1 comment
Labels
cleanup Cleaning up code, data, styling, etc tests related to (unit / integration / automated) testing

Comments

@jsheunis
Copy link
Member

jsheunis commented Jul 4, 2022

There are existing unit tests, although altogether these are probably not sufficient as a full test suite. A few integration tests are needed, especially to test complicated and/or recently developed cases e.g. the addition of top_display and additional_display metadata to a catalog. This extension uses pytest but tests were developed without considering the migration of datalad's tests from nose to pytest. Perhaps there are now reusable snippets of code from datalad.

While working on this issue, take note of the relevant information in these issues:

To do:

  • create test data that includes more complicated and possibly whole-catalog metadata, in order to allow testing more complicated catalog add operations as well as a full catalog generation integration test.
  • write tests for more recent/complicated metadata add operations
  • write a full catalog generation integration test
  • check test modules to see if redundancy or inefficiencies can be removed and if decorators/snippets/utilities/etc can be imported from datalad's updated testing framework.
@jsheunis jsheunis added the cleanup Cleaning up code, data, styling, etc label Jul 4, 2022
@jsheunis jsheunis added the tests related to (unit / integration / automated) testing label Jul 4, 2022
@jsheunis
Copy link
Member Author

Closed by #309

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Cleaning up code, data, styling, etc tests related to (unit / integration / automated) testing
Projects
None yet
Development

No branches or pull requests

1 participant