)]}'
{"keystoneclient/tests/v2_0/test_shell.py":[{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"7e41a38d67c18afac5378e67619da0d2b254cca2","unresolved":false,"context_lines":[{"line_number":274,"context_line":"        self.run_command(\u0027service-create --type compute\u0027)"},{"line_number":275,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/OS-KSADM/services\u0027)"},{"line_number":276,"context_line":"        json \u003d {\"OS-KSADM:service\": {\"type\": \"compute\","},{"line_number":277,"context_line":"                                     \"name\": None,"},{"line_number":278,"context_line":"                                     \"description\": None}}"},{"line_number":279,"context_line":"        self.assertRequestBodyIs(json\u003djson)"},{"line_number":280,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_24de74da","line":277,"updated":"2014-12-24 10:51:58.000000000","message":"It means here that we leave \"--name\" argument untested. Is it ok?","commit_id":"af2978799e20f6747738cd38de6aedb071296fd0"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"dc30626a30ae2f4dc36e1711d74f5b39b8f46a5a","unresolved":false,"context_lines":[{"line_number":274,"context_line":"        self.run_command(\u0027service-create --type compute\u0027)"},{"line_number":275,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/OS-KSADM/services\u0027)"},{"line_number":276,"context_line":"        json \u003d {\"OS-KSADM:service\": {\"type\": \"compute\","},{"line_number":277,"context_line":"                                     \"name\": None,"},{"line_number":278,"context_line":"                                     \"description\": None}}"},{"line_number":279,"context_line":"        self.assertRequestBodyIs(json\u003djson)"},{"line_number":280,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_c1391041","line":277,"in_reply_to":"3a961159_24de74da","updated":"2014-12-25 11:17:32.000000000","message":"The optional field description is not tested as well at the moment, only the mandatory fields are being tested. I just removed the name for consistency.\n\nI could add another test that passes both the optional arguments: name and description. What do you think?","commit_id":"af2978799e20f6747738cd38de6aedb071296fd0"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"09c16e2a36c0b1b6ffea91e44fb61a23c48466f5","unresolved":false,"context_lines":[{"line_number":274,"context_line":"        self.run_command(\u0027service-create --type compute\u0027)"},{"line_number":275,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/OS-KSADM/services\u0027)"},{"line_number":276,"context_line":"        json \u003d {\"OS-KSADM:service\": {\"type\": \"compute\","},{"line_number":277,"context_line":"                                     \"name\": None,"},{"line_number":278,"context_line":"                                     \"description\": None}}"},{"line_number":279,"context_line":"        self.assertRequestBodyIs(json\u003djson)"},{"line_number":280,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_61393cf2","line":277,"in_reply_to":"3a961159_c1391041","updated":"2014-12-25 11:36:46.000000000","message":"it\u0027d be great!","commit_id":"af2978799e20f6747738cd38de6aedb071296fd0"}]}