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
Extract OrderedDict tests into separate file #69802
Comments
OrderedDict tests now are about 1/3 of test_collections, and I think their volume will grow. I suggest to extract these tests into separate file test_ordereddict. There are precedences: test_deque (deque also is implemented in C), and test_userdict, test_userlist and test_userstring (for historical reasons). The main reason to me is the possibility to run quick OrderedDict tests (and only OrderedDict tests). |
FWIW, I totally support this. I originally had a patch up that did so but Raymond indicated that he'd rather not split the tests out though I made exactly the same arguments that you have. Perhaps he's changed his mind in the intervening years (or I misunderstood his objections)? :) |
If it helps with all the great work you're doing, go ahead and move the OD tests out to a separate file. |
Thank you Raymond. Here is a patch. It is in Git format, hence Rietveld will not work with it. During creating a patch I found a bug in test_dict_update, it tested only default implementation and didn't test subclasses. |
New changeset 8d9a0540adf9 by Serhiy Storchaka in branch '3.4': New changeset 46e95e0eaae5 by Serhiy Storchaka in branch '3.5': New changeset 81fa901162f9 by Serhiy Storchaka in branch 'default': New changeset afaad8dc8edf by Serhiy Storchaka in branch '2.7': |
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: