-
Notifications
You must be signed in to change notification settings - Fork 49
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
swap unittest with pytest #58
Comments
Open
eltonlaw
added a commit
that referenced
this issue
Jul 28, 2019
eltonlaw
added a commit
that referenced
this issue
Jul 28, 2019
For `test/imputation/ts/test_moving_window.py` a pytest.fixture needed to be used so that tests didn't interfere with each other. For the future we will probably want to convert everything to use fixtures so no weird errors occur.
eltonlaw
added a commit
that referenced
this issue
Jul 28, 2019
eltonlaw
added a commit
that referenced
this issue
Jul 28, 2019
For `test/imputation/ts/test_moving_window.py` a pytest.fixture needed to be used so that tests didn't interfere with each other. For the future we will probably want to convert everything to use fixtures so no weird errors occur.
eltonlaw
added a commit
that referenced
this issue
Jul 28, 2019
For `test/imputation/ts/test_moving_window.py` a pytest.fixture needed to be used so that tests didn't interfere with each other. For the future we will probably want to convert everything to use fixtures so no weird errors occur.
eltonlaw
added a commit
that referenced
this issue
Jul 28, 2019
For `test/imputation/ts/test_moving_window.py` a pytest.fixture needed to be used so that tests didn't interfere with each other. For the future we will probably want to convert everything to use fixtures so no weird errors occur.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
mainly replace
self.*
methods withassert
+ some additional clean upThe text was updated successfully, but these errors were encountered: