)]}'
{"cinder/tests/unit/volume/drivers/test_pure.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"16b087f414b02343ea260df3e297d94844767ba4","unresolved":false,"context_lines":[{"line_number":1034,"context_line":"        mock_cgsnapshot \u003d mock.Mock()"},{"line_number":1035,"context_line":"        mock_snapshots \u003d [mock.Mock() for i in range(5)]"},{"line_number":1036,"context_line":"        mock_volumes \u003d [mock.Mock() for i in range(5)]"},{"line_number":1037,"context_line":"        self.driver.create_consistencygroup_from_src("},{"line_number":1038,"context_line":"            mock_context,"},{"line_number":1039,"context_line":"            mock_group,"},{"line_number":1040,"context_line":"            mock_volumes,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_351cb6c0","line":1037,"updated":"2016-09-07 08:34:57.000000000","message":"-1: You  are not checking this returned value","commit_id":"31ca02574538bf5c65fd1510a063800782ec4631"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"16b087f414b02343ea260df3e297d94844767ba4","unresolved":false,"context_lines":[{"line_number":1069,"context_line":"        mock_source_cg \u003d mock.MagicMock()"},{"line_number":1070,"context_line":"        mock_volumes \u003d [mock.MagicMock() for i in range(num_volumes)]"},{"line_number":1071,"context_line":"        mock_source_vols \u003d [mock.MagicMock() for i in range(num_volumes)]"},{"line_number":1072,"context_line":"        self.driver.create_consistencygroup_from_src("},{"line_number":1073,"context_line":"            mock_context,"},{"line_number":1074,"context_line":"            mock_group,"},{"line_number":1075,"context_line":"            mock_volumes,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_7580fec0","line":1072,"updated":"2016-09-07 08:34:57.000000000","message":"-1: You  are not checking this returned value","commit_id":"31ca02574538bf5c65fd1510a063800782ec4631"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ab476bf1811249a5d6804b25168b52491c454537","unresolved":false,"context_lines":[{"line_number":1295,"context_line":""},{"line_number":1296,"context_line":"        self.array.destroy_pgroup.assert_called_with(snap_name)"},{"line_number":1297,"context_line":"        self.assertFalse(self.array.eradicate_pgroup.called)"},{"line_number":1298,"context_line":"        self.assertIsNone(model_update)"},{"line_number":1299,"context_line":"        self.assertIsNone(snapshots)"},{"line_number":1300,"context_line":""},{"line_number":1301,"context_line":"        self.array.destroy_pgroup.side_effect \u003d \\"},{"line_number":1302,"context_line":"            self.purestorage_module.PureHTTPError("}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_36d63728","line":1299,"range":{"start_line":1298,"start_character":0,"end_line":1299,"end_character":36},"updated":"2016-09-08 10:29:42.000000000","message":"nit: For consistency you could have done the assert like in the other tests.","commit_id":"ffdb05f532393c3d50481cf4bd886f38d8f958d3"}],"cinder/volume/drivers/pure.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"16b087f414b02343ea260df3e297d94844767ba4","unresolved":false,"context_lines":[{"line_number":794,"context_line":"                \u0027status\u0027: \u0027deleted\u0027,"},{"line_number":795,"context_line":"            })"},{"line_number":796,"context_line":""},{"line_number":797,"context_line":"        model_update \u003d {\u0027status\u0027: cgsnapshot.status}"},{"line_number":798,"context_line":""},{"line_number":799,"context_line":"        return model_update, snapshot_updates"},{"line_number":800,"context_line":""},{"line_number":801,"context_line":"    def _validate_manage_existing_ref(self, existing_ref, is_snap\u003dFalse):"},{"line_number":802,"context_line":"        \"\"\"Ensure that an existing_ref is valid and return volume info"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_b501a66c","line":799,"range":{"start_line":797,"start_character":0,"end_line":799,"end_character":45},"updated":"2016-09-07 08:34:57.000000000","message":"-1: You missed this one","commit_id":"31ca02574538bf5c65fd1510a063800782ec4631"}]}