-
Notifications
You must be signed in to change notification settings - Fork 101
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
Move interop tests outside tests
#308
Move interop tests outside tests
#308
Conversation
It is moved to the top level package `tests_interop`, to avoid circular dependency, with the dependency moved to `tox`.
tests
To make `tox -e py37-interop` run.
@carver |
Interop tests seemed to fail. I will fix it tomorrow. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, besides the test failure, looks good to me!
- Use `from_id`, the changed field name in `PSMessage`. - PR feedbacks - Add label `test` in `testenv` in tox.ini, to avoid wrong dispatching an environment's command in the future. - Use `pytest` over `py.test`.
To be more confident that the stream handler is registered in the daemon.
@carver Thanks for the review! |
It is moved to the top level package
tests_interop
, to avoid circular dependency, with the dependency onp2pclient
moved totox
.Fixes #296 .