-
Notifications
You must be signed in to change notification settings - Fork 965
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
Fix all remaining unit tests under Python 3 #6450
Fix all remaining unit tests under Python 3 #6450
Conversation
daf7b7f
to
6c7773c
Compare
I think all unit tests should be passing now, and I even managed to run a couple of tools successfully! |
6c7773c
to
25d871a
Compare
@@ -1546,7 +1546,7 @@ def check_limits(self, runtime=None): | |||
return None | |||
|
|||
def has_limits(self): | |||
has_output_limit = self.app.job_config.limits.output_size > 0 | |||
has_output_limit = self.app.job_config.limits.output_size and self.app.job_config.limits.output_size > 0 |
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.
Need to do the same at line 1533
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.
Good catch, done that in aaf968c
Hmm, the failing tests in https://travis-ci.org/mvdbeek/galaxy/jobs/399571214 pass fine locally, but I'm running 3.6 instead of 3.4 ... do you think it's worth switching travis to 3.6 ? |
aaf968c
to
0cc6fe3
Compare
ca4fbeb
to
58a4518
Compare
@@ -26,7 +26,7 @@ | |||
from galaxy.util import stringify_dictionary_keys | |||
|
|||
# ensure supported version | |||
assert sys.version_info[:2] >= (2, 6) and sys.version_info[:2] <= (2, 7), 'Python version must be 2.6 or 2.7, this is: %s' % sys.version | |||
assert sys.version_info[:2] >= (2, 7) and sys.version_info[:2] <= (3, 7), 'Python version must be at least 2.7, this is: %s' % sys.version |
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.
Can we just drop the sys.version_info[:2] <= ()
?
2e9c123
to
73e98cd
Compare
@natefoo Can you help with the last remaining unit test error which involves |
a2a4519
to
4ae2d97
Compare
eef2d4d
to
256f569
Compare
Might fix the last failing unit test?
256f569
to
d15b6cc
Compare
All green! |
@@ -34,7 +34,6 @@ matrix: | |||
- env: TOX_ENV=py34-unit | |||
addons: *py3_addons | |||
allow_failures: | |||
- env: TOX_ENV=py34-unit |
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.
🎉
xref. #1715