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
Empty tests in pickletester need to be implemented or removed #63847
Comments
There are a couple of tests in pickletester that should either be implemented or removed, namely test_reduce and test_getinitargs of AbstractPickleTests. I don't know enough about pickling to suggest which route should be taken or how, so I'll leave that up to anyone who wants to write a patch :) |
Hi, I created 2 simple tests for test_getinitargs and test_reduce. |
The patch looks good to me, but I can't claim to know enough about pickle to say whether the tests are correct. Alexandre or Antoine? |
The patch looks fine to me. Gennadiy, could you go and sign a contributor's agreement? http://www.python.org/psf/contrib/ |
Antoine, sure! I have just signed it. Thank you! |
New changeset fef075ddaec9 by Antoine Pitrou in branch 'default': |
I've pushed the patch. Thank you for your contribution! |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: