[issue5665] Add more pickling tests
report at bugs.python.org
Fri Apr 3 05:46:26 CEST 2009
Alexandre Vassalotti <alexandre at peadrop.com> added the comment:
Overall, the patch looks good. I haven't reviewed the patch thoroughly
yet, but there is a few things I am not sure about. First, why do you
bother supporting 2.4 (i.e. with a copy of run_with_locale) in a patch
aimed at 2.7?
In test_many_puts_and_gets(), I believe this:
for proto in [0, 1, 2]:
should be changed to:
for proto in protocols:
Also, I think the tests in AbstractPicklerUnpicklerObjectTests could be
stronger if they used the pickletools module to decompile to check that
only PUT opcodes have changed or not. However, the extra complexity may
not worth it.
And one last thing, why AbstractCompatTests is hard-coded to use
cPickle? Shouldn't the standard pickle module be tested too?
Python tracker <report at bugs.python.org>
More information about the Python-bugs-list