)]}'
{"cinder/tests/test_emc_vmax.py":[{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"1b8adbffdbb439ae8c83f87bd22482bc8635b5d7","unresolved":false,"context_lines":[{"line_number":1750,"context_line":"        EMCVMAXUtils,"},{"line_number":1751,"context_line":"        \u0027_wait_for_job_complete\u0027,"},{"line_number":1752,"context_line":"        return_value\u003dNone)"},{"line_number":1753,"context_line":"    def test_delete_volume_failed("},{"line_number":1754,"context_line":"            self, _mock_volume_type, mock_storage_system, mock_wait):"},{"line_number":1755,"context_line":"        self.driver.create_volume(self.data.failed_delete_vol)"},{"line_number":1756,"context_line":"        self.assertRaises(exception.VolumeBackendAPIException,"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_2a106d58","line":1753,"updated":"2015-02-11 07:17:20.000000000","message":"The order of mocks passed into test case is the same ordering python applies to decorators, which is bottom upwards. All of your test cases in this file are wrong. See http://www.voidspace.org.uk/python/mock/patch.html#nesting-patch-decorators","commit_id":"20459e33309411da86c9f6c1576a6e5d1a794838"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"9c1404fa4649e29a197474a401abd2a419f91c35","unresolved":false,"context_lines":[{"line_number":1750,"context_line":"        EMCVMAXUtils,"},{"line_number":1751,"context_line":"        \u0027_wait_for_job_complete\u0027,"},{"line_number":1752,"context_line":"        return_value\u003dNone)"},{"line_number":1753,"context_line":"    def test_delete_volume_failed("},{"line_number":1754,"context_line":"            self, _mock_volume_type, mock_storage_system, mock_wait):"},{"line_number":1755,"context_line":"        self.driver.create_volume(self.data.failed_delete_vol)"},{"line_number":1756,"context_line":"        self.assertRaises(exception.VolumeBackendAPIException,"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_81385cd4","line":1753,"in_reply_to":"da86d52c_2a106d58","updated":"2015-02-11 09:47:04.000000000","message":"Mike, good catch!","commit_id":"20459e33309411da86c9f6c1576a6e5d1a794838"}]}