)]}'
{"mistral/tests/unit/engine/test_policies.py":[{"author":{"_account_id":20863,"name":"Sharat Sharma","email":"sharat.sharma@in.ibm.com","username":"sharat.sharma"},"change_message_id":"3ad71d16417d53e5252f794dfd13c30115d40c18","unresolved":false,"context_lines":[{"line_number":516,"context_line":"    @mock.patch.object("},{"line_number":517,"context_line":"        requests,"},{"line_number":518,"context_line":"        \u0027request\u0027,"},{"line_number":519,"context_line":"        mock.MagicMock(side_effect\u003dException())"},{"line_number":520,"context_line":"    )"},{"line_number":521,"context_line":"    def test_retry_policy(self):"},{"line_number":522,"context_line":"        wb_service.create_workbook_v2(RETRY_WB)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_b648618c","line":519,"range":{"start_line":519,"start_character":13,"end_line":519,"end_character":18},"updated":"2016-11-14 05:11:27.000000000","message":"I am afraid that mock.MagicMock is being replaced by mock.Mock in other projects. Should we be using the same here too?","commit_id":"f7501a8716e02d33d9740c67361e429b99e79711"},{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"806d5f4b15f375210e449056f847cd8700afadf0","unresolved":false,"context_lines":[{"line_number":516,"context_line":"    @mock.patch.object("},{"line_number":517,"context_line":"        requests,"},{"line_number":518,"context_line":"        \u0027request\u0027,"},{"line_number":519,"context_line":"        mock.MagicMock(side_effect\u003dException())"},{"line_number":520,"context_line":"    )"},{"line_number":521,"context_line":"    def test_retry_policy(self):"},{"line_number":522,"context_line":"        wb_service.create_workbook_v2(RETRY_WB)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a77a97e_907049de","line":519,"range":{"start_line":519,"start_character":13,"end_line":519,"end_character":18},"in_reply_to":"7a77a97e_3c6fd993","updated":"2016-11-17 06:47:48.000000000","message":"I agree with Winson. Let\u0027s merge this patch and address your concern with a separate blueprint. Also, if something is used in other projects it doesn\u0027t necessarily mean that we have to use it too. I would rather be interested in technical details why we should be using mock.Mock.","commit_id":"f7501a8716e02d33d9740c67361e429b99e79711"},{"author":{"_account_id":5558,"name":"Winson Chan","email":"wcchan@stackstorm.com","username":"winson-c-chan"},"change_message_id":"b8dec79567a2906692c1c64a8ec2c0419e6b3f25","unresolved":false,"context_lines":[{"line_number":516,"context_line":"    @mock.patch.object("},{"line_number":517,"context_line":"        requests,"},{"line_number":518,"context_line":"        \u0027request\u0027,"},{"line_number":519,"context_line":"        mock.MagicMock(side_effect\u003dException())"},{"line_number":520,"context_line":"    )"},{"line_number":521,"context_line":"    def test_retry_policy(self):"},{"line_number":522,"context_line":"        wb_service.create_workbook_v2(RETRY_WB)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a77a97e_3c6fd993","line":519,"range":{"start_line":519,"start_character":13,"end_line":519,"end_character":18},"in_reply_to":"9a629dbe_b648618c","updated":"2016-11-14 18:51:29.000000000","message":"If it\u0027s so, it should be done as a separate blueprint.","commit_id":"f7501a8716e02d33d9740c67361e429b99e79711"}]}