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 test expectations #6316
fix test expectations #6316
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #6316 +/- ##
==========================================
- Coverage 83.92% 83.88% -0.05%
==========================================
Files 161 161
Lines 11972 11972
Branches 1865 1927 +62
==========================================
- Hits 10048 10043 -5
- Misses 1588 1594 +6
+ Partials 336 335 -1 |
tests/test_utils_python.py
Outdated
@@ -239,8 +239,11 @@ def __call__(self, a, b, c): | |||
self.assertEqual(get_func_args(" ".join, stripself=True), ["iterable"]) | |||
|
|||
if platform.python_implementation() == "CPython": | |||
# doesn't work on CPython: https://bugs.python.org/issue42785 | |||
self.assertEqual(get_func_args(operator.itemgetter(2)), []) | |||
# Didn't used to work on CPython: https://bugs.python.org/issue42785 |
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.
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.
Updated
tests/test_utils_python.py
Outdated
@@ -239,8 +239,11 @@ def __call__(self, a, b, c): | |||
self.assertEqual(get_func_args(" ".join, stripself=True), ["iterable"]) | |||
|
|||
if platform.python_implementation() == "CPython": | |||
# doesn't work on CPython: https://bugs.python.org/issue42785 | |||
self.assertEqual(get_func_args(operator.itemgetter(2)), []) | |||
# This never worked on CPython: https://bugs.python.org/issue42785 |
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.
Well it didn't work but now works :) How about "This didn't work on older versions of CPython"?
Also I would update the issue URL to python/cpython#86951
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.
Updated the link and the comment
Thanks! |
Should I make a similar PR for scrapy/itemloaders#76? |
That would be awesome! |
Fixes #6312