You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Until now the testing is done manually via notebooks.
There should be some tests, that call the notebook from the commandline. This should be possible without modifying the module
Maybe there should also be some unit tests, that use the package as regular python-module, and not as ipython. This might require some changes. But at first glance, beside the registration with ipython, only 2 parts of the package require ipython:
shell for execution. This could be replaced by eval for non-ipython execution
display(HTML(...)) for tables. This could be replaced by "print" for non-ipython execution
The implementation would be like this: Those 2 things get moved function-parameters in the constructor, so the caller (e.g. the magics-class) has to supply them.
The real question is: Should it even be possible to use this module outside ipython?
The text was updated successfully, but these errors were encountered:
this guide suggests to use doctest.
Until now the testing is done manually via notebooks.
There should be some tests, that call the notebook from the commandline. This should be possible without modifying the module
Maybe there should also be some unit tests, that use the package as regular python-module, and not as ipython. This might require some changes. But at first glance, beside the registration with ipython, only 2 parts of the package require ipython:
shell
for execution. This could be replaced by eval for non-ipython executiondisplay(HTML(...))
for tables. This could be replaced by "print" for non-ipython executionThe implementation would be like this: Those 2 things get moved function-parameters in the constructor, so the caller (e.g. the magics-class) has to supply them.
The real question is: Should it even be possible to use this module outside ipython?
The text was updated successfully, but these errors were encountered: