)]}'
{"nova/tests/functional/api_samples_test_base.py":[{"author":{"_account_id":9550,"name":"Eugeniya Kudryashova","email":"ekudryashova@mirantis.com","username":"ekudryashova"},"change_message_id":"f9e1f4147038610633a3b8e137cf7b9c1b4360a8","unresolved":false,"context_lines":[{"line_number":330,"context_line":"                                  api_version\u003d(api_version or"},{"line_number":331,"context_line":"                                               self.request_api_version))"},{"line_number":332,"context_line":""},{"line_number":333,"context_line":"    def _do_put(self, url, name, subs, api_version\u003dNone):"},{"line_number":334,"context_line":"        return self._do_post(url, name, subs, method\u003d\u0027PUT\u0027,"},{"line_number":335,"context_line":"                             api_version\u003d(api_version or"},{"line_number":336,"context_line":"                                          self.request_api_version))"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_8f96cc92","line":333,"updated":"2015-06-22 17:28:54.000000000","message":"Can we just specify here self.request_api_version as default value instead of None?","commit_id":"dc3777406120b64c4fd36ad65cadcf8459f9e3fb"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"b17768cd1c305132038aab1f024ef8a3bab2141e","unresolved":false,"context_lines":[{"line_number":330,"context_line":"                                  api_version\u003d(api_version or"},{"line_number":331,"context_line":"                                               self.request_api_version))"},{"line_number":332,"context_line":""},{"line_number":333,"context_line":"    def _do_put(self, url, name, subs, api_version\u003dNone):"},{"line_number":334,"context_line":"        return self._do_post(url, name, subs, method\u003d\u0027PUT\u0027,"},{"line_number":335,"context_line":"                             api_version\u003d(api_version or"},{"line_number":336,"context_line":"                                          self.request_api_version))"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_133bb7d2","line":333,"in_reply_to":"fa32b979_8f96cc92","updated":"2015-06-23 02:00:15.000000000","message":"Actually I try the way you mentioned, it can\u0027t work. self.request_api_version should been init in subclass to different value with superclass, but the _do_* functions were defined in the TestBase class, so the funcation args default value will be request_api_version \u003d None forever. It can\u0027t been modified to different value in subclass.\n\nSee details:\n\nhttp://docs.python-guide.org/en/latest/writing/gotchas/#mutable-default-arguments","commit_id":"dc3777406120b64c4fd36ad65cadcf8459f9e3fb"}]}
