Vajrasky Kok added the comment: Attached the second version patch to address R. David Murray's concerns.
1. test_dotted_attribute name has been taken, so I use test_instance_with_allow_dotted_names name. Or should I use test_instance_with_dotted_attribute name? Or maybe test_allow_dotted_names name? 2. There is no test testing allow_dotted_name feature with fail case. I did consider to chuck the test in this ticket. But one xmlrpc server can only register one instance, meaning I have to create a separate server for this test. Currently we have two: http_server and http_multi_server. I don't think it's a good idea to chuck the fail case to http_multi_server. 3. Okay, I removed the checking datetime test. Thanks. ---------- Added file: http://bugs.python.org/file32035/move_current_time_test_from_xmlrpc_net_to_xmlrpc_v2.patch _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue19192> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
