ydata-profiling
ydata-profiling copied to clipboard
Added support for NBTest, for testing notebooks with assertions in the `Multi-software test` CI pipeline
Closes #1761
Hi, We are developers of the NBTest framework. We are following up on the previous discussion #1761. We can now use NBTest's assertions directly in the notebooks which were part of your library's tests. Hence, we have replaced nbval with nbtest in the CI. Please let us know if these changes make sense. We are happy to answer any questions! Thanks!
Description of the changes:
- Modified all the notebooks in the
tests/notebooks/directory to utilise NBTest assertions - Replaced
--nbvalflag with--nbtestin the Makefile - Reordered the test steps to demonstrate NBTest's pytest plugin in the CI (which were not executed due to failing unit tests)
- Updated
pyproject.tomlto includenbtest-genas it's test dependency, instead ofnbval
Note: This notebook has some dataset API errors, but the rest of them do work as expected.
@fabclmnt Please review