)]}'
{"magnumclient/tests/test_shell_args.py":[{"author":{"_account_id":15311,"name":"Surojit Pathak","email":"surojit.pathak@gmail.com","username":"suropathak"},"change_message_id":"b283636bb5695d40e3d23c2cc8daafd43c7d9bdc","unresolved":false,"context_lines":[{"line_number":628,"context_line":""},{"line_number":629,"context_line":"    @mock.patch(\u0027magnumclient.v1.services.ServiceManager.delete\u0027)"},{"line_number":630,"context_line":"    def test_coe_service_delete_success(self, mock_delete):"},{"line_number":631,"context_line":"        self._test_arg_success(\u0027coe-service-delete xxx zzz\u0027)"},{"line_number":632,"context_line":"        self.assertTrue(mock_delete.called)"},{"line_number":633,"context_line":"        self.assertEqual(1, mock_delete.call_count)"},{"line_number":634,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a1a9d01_0fb90779","line":631,"updated":"2015-09-29 21:59:06.000000000","message":"I think here it makes sense to pass more readable arguments, and to reflect their positional dependency","commit_id":"bdbde5abb9eb4133314c90fc3ecb0eec5641bc8b"}]}