-
Notifications
You must be signed in to change notification settings - Fork 98
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
Implement unit testing by means of Tox #172
Conversation
1cad80a
to
c77617e
Compare
I've never used tox, been away from python for quite a while.. will leave this one to @dpnova. |
@@ -149,7 +149,7 @@ def test_object_dict(self): | |||
def test_import_object(self): | |||
import os.path | |||
other_os = import_object("os.path") | |||
self.assertIs(os.path, other_os) | |||
self.assertIdentical(os.path, other_os) |
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.
Is assertIdentical in python 2.7 @evilaliv3 ?
Aside from my one question this is looking good @evilaliv3 @fiorix - the move to tox will be good for catching edge cases with versions of python/twisted we don't all use ourselves. |
yes @dpnova the one is the alias of the other: https://github.com/twisted/twisted/blob/trunk/twisted/trial/_synctest.py#L454 |
awesome. but need to resolved conflicts |
Should be easy to resolve @evilaliv3 - I'll accept it once it's sorted as this is a good step. |
Stuck for too long and conflicting. |
With this commit i've switched unit-tests to use Tox
This way we could start testing properly against the different versions of python (at least 2.7 and 3.5) as for ticket #171
With this pull request: