)]}'
{"magnumclient/tests/v1/test_baymodels_shell.py":[{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"0136a91e1d72e82acadde58500628871e1408ee4","unresolved":false,"context_lines":[{"line_number":323,"context_line":"    @mock.patch(\u0027magnumclient.v1.baymodels.BayModelManager.update\u0027)"},{"line_number":324,"context_line":"    def test_baymodel_update_success(self, mock_update):"},{"line_number":325,"context_line":"        self._test_arg_success(\u0027baymodel-update test add test\u003dtest\u0027)"},{"line_number":326,"context_line":"        self.assertTrue(mock_update.called)"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    @mock.patch(\u0027magnumclient.v1.baymodels.BayModelManager.update\u0027)"},{"line_number":329,"context_line":"    def test_baymodel_update_success_many_attribute(self, mock_update):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_f6cc076c","line":326,"range":{"start_line":326,"start_character":8,"end_line":326,"end_character":23},"updated":"2016-11-10 06:37:31.000000000","message":"yatin, can you please change it to assert_called_once_with?","commit_id":"7a38e426a3ea297c741794e3823e05f66a674349"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"ec4bff9b303609a17550f37dab139feba75175f5","unresolved":false,"context_lines":[{"line_number":323,"context_line":"    @mock.patch(\u0027magnumclient.v1.baymodels.BayModelManager.update\u0027)"},{"line_number":324,"context_line":"    def test_baymodel_update_success(self, mock_update):"},{"line_number":325,"context_line":"        self._test_arg_success(\u0027baymodel-update test add test\u003dtest\u0027)"},{"line_number":326,"context_line":"        self.assertTrue(mock_update.called)"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    @mock.patch(\u0027magnumclient.v1.baymodels.BayModelManager.update\u0027)"},{"line_number":329,"context_line":"    def test_baymodel_update_success_many_attribute(self, mock_update):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_a7fc1b73","line":326,"range":{"start_line":326,"start_character":8,"end_line":326,"end_character":23},"in_reply_to":"9a629dbe_f6cc076c","updated":"2016-11-10 08:04:50.000000000","message":"@Madhuri, yes i can change that but it will be inconsistent with other tests. \nIt only make sense if it is changed for every test. Please suggest. If this is the case i will update at all places in a seperate patch.\nI also think args to mock_method should also be validated instead of just checking the mock method is called.","commit_id":"7a38e426a3ea297c741794e3823e05f66a674349"},{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"0136a91e1d72e82acadde58500628871e1408ee4","unresolved":false,"context_lines":[{"line_number":329,"context_line":"    def test_baymodel_update_success_many_attribute(self, mock_update):"},{"line_number":330,"context_line":"        self._test_arg_success(\u0027baymodel-update test \u0027"},{"line_number":331,"context_line":"                               \u0027add test\u003dtest test1\u003dtest1\u0027)"},{"line_number":332,"context_line":"        self.assertTrue(mock_update.called)"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    @mock.patch(\u0027magnumclient.v1.baymodels.BayModelManager.update\u0027)"},{"line_number":335,"context_line":"    def test_baymodel_update_failure_wrong_op(self, mock_update):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_56c0532a","line":332,"range":{"start_line":332,"start_character":13,"end_line":332,"end_character":23},"updated":"2016-11-10 06:37:31.000000000","message":"ditto","commit_id":"7a38e426a3ea297c741794e3823e05f66a674349"}]}