eugeniy / pytest-tornado

A py.test plugin providing fixtures and markers to simplify testing of asynchronous tornado applications.
Apache License 2.0
121 stars 34 forks source link

Update from fork #43

Closed vidartf closed 5 years ago

vidartf commented 5 years ago

This PR brings back in some changes from my fork:

okin commented 5 years ago

Wouldn't it make sense to remove the combinations with old pytest versions from the build matrix if this new version requires at least pytest 3.6 (stated in setup.py)?

Also I'd suggest adding tests for the latest pytest which as of this moment seems to be 4.3.1.

In this regard I think it would make sense to make use of the compatible release version statement at least for the pytest part. We don't want to need to update the travis file every time pytest releases a patch version.

vidartf commented 5 years ago

Wouldn't it make sense to remove the combinations with old pytest versions from the build matrix if this new version requires at least pytest 3.6 (stated in setup.py)?

@okin Hah, yes, I missed that. Will fix.

Also I'd suggest adding tests for the latest pytest which as of this moment seems to be 4.3.1.

Thought that was in, but it was in another branch were I'm trying for tornado 6 compat.

In this regard I think it would make sense to make use of the compatible release version statement at least for the pytest part. We don't want to need to update the travis file every time pytest releases a patch version.

Are you suggesting we use 4 in the travis config then? Your link seem to be related to version specifiers in setup.py so I'm a little confused.

vidartf commented 5 years ago

Ah, I figure out what you meant: Use ~= in the pip install line, right?

okin commented 5 years ago

@vidartf Yes, I was referring to the usage of ~= in combination with pip install. Your changes look good to me!