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

doctest to unittest #67

Open
dizak opened this issue Aug 11, 2018 · 1 comment
Open

doctest to unittest #67

dizak opened this issue Aug 11, 2018 · 1 comment
Labels
bug is something worng? update generic change, too small to call it a feature
Milestone

Comments

@dizak
Copy link
Owner

dizak commented Aug 11, 2018

ab2a2de on branch UPD#64

Description

doctests do not work after py extension is removed from the python scripts. Also, these are the basic level math functions that would be suitable for that kind of tests.

Steps to reproduce

  1. Run python -m doctest on any of the mothulity scripts

Expected behaviour

Passing doctests

Actual behaviour

Failing doctest mostly due to undefined variables since the tested functions self-reference fails without py extension.

@dizak dizak added bug is something worng? update generic change, too small to call it a feature labels Aug 11, 2018
@dizak dizak added this to the distribution milestone Aug 11, 2018
@dizak
Copy link
Owner Author

dizak commented Aug 11, 2018

mothulity scripts are not importable after removal of py extension. Probably of the functions besides main from the script should be moved to utilities or other part of the module and the tests should be converted.

@dizak dizak mentioned this issue Aug 11, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug is something worng? update generic change, too small to call it a feature
Projects
None yet
Development

No branches or pull requests

1 participant