[issue19192] Move test_current_time from test_xmlrpc_net to test_xmlrpc

Vajrasky Kok report at bugs.python.org
Thu Oct 10 17:52:26 CEST 2013


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 <report at bugs.python.org>
<http://bugs.python.org/issue19192>
_______________________________________


More information about the Python-bugs-list mailing list