)]}'
{"openstackclient/tests/volume/v2/test_backup.py":[{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"3c6420af57f50ec438b43a1f3420d9c9c2b3e61f","unresolved":false,"context_lines":[{"line_number":148,"context_line":""},{"line_number":149,"context_line":"class TestBackupList(TestBackup):"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"    backups \u003d volume_fakes.FakeBackup.create_backups(count\u003d3)"},{"line_number":152,"context_line":"    volumes \u003d volume_fakes.FakeVolume.create_volumes(count\u003d3)"},{"line_number":153,"context_line":"    columns \u003d ["},{"line_number":154,"context_line":"        \u0027ID\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"dab17558_d65f323f","line":151,"range":{"start_line":151,"start_character":38,"end_line":151,"end_character":61},"updated":"2016-05-16 05:40:31.000000000","message":"I think you forgot to set volume_id for the backups.","commit_id":"5e7d3dd878fe349c8f682003e79f4027cfb9769d"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"3c6420af57f50ec438b43a1f3420d9c9c2b3e61f","unresolved":false,"context_lines":[{"line_number":159,"context_line":"    ]"},{"line_number":160,"context_line":"    datalist \u003d ("},{"line_number":161,"context_line":"        ("},{"line_number":162,"context_line":"            backups[0].id,"},{"line_number":163,"context_line":"            backups[0].name,"},{"line_number":164,"context_line":"            backups[0].description,"},{"line_number":165,"context_line":"            backups[0].status,"},{"line_number":166,"context_line":"            backups[0].size"},{"line_number":167,"context_line":"        ),"},{"line_number":168,"context_line":"    )"},{"line_number":169,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"dab17558_d678d2c6","line":166,"range":{"start_line":162,"start_character":0,"end_line":166,"end_character":27},"updated":"2016-05-16 05:40:31.000000000","message":"If you created 3 backups, you can add all of them to the datalist. And BTW, name it \"data\" if you don\u0027t mind.","commit_id":"5e7d3dd878fe349c8f682003e79f4027cfb9769d"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"3c6420af57f50ec438b43a1f3420d9c9c2b3e61f","unresolved":false,"context_lines":[{"line_number":170,"context_line":"    def setUp(self):"},{"line_number":171,"context_line":"        super(TestBackupList, self).setUp()"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"        self.volumes_mock.list.return_value \u003d [self.volumes[0]]"},{"line_number":174,"context_line":"        self.backups_mock.list.return_value \u003d [self.backups[0]]"},{"line_number":175,"context_line":"        # Get the command to test"},{"line_number":176,"context_line":"        self.cmd \u003d backup.ListBackup(self.app, None)"},{"line_number":177,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"dab17558_167ffaab","line":174,"range":{"start_line":173,"start_character":0,"end_line":174,"end_character":63},"updated":"2016-05-16 05:40:31.000000000","message":"Use side_effect to fake multiple return values.","commit_id":"5e7d3dd878fe349c8f682003e79f4027cfb9769d"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"3c6420af57f50ec438b43a1f3420d9c9c2b3e61f","unresolved":false,"context_lines":[{"line_number":200,"context_line":""},{"line_number":201,"context_line":"        self.assertEqual(columns, columns)"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"        datalist \u003d (("},{"line_number":204,"context_line":"            self.backups[0].id,"},{"line_number":205,"context_line":"            self.backups[0].name,"},{"line_number":206,"context_line":"            self.backups[0].description,"},{"line_number":207,"context_line":"            self.backups[0].status,"},{"line_number":208,"context_line":"            self.backups[0].size,"},{"line_number":209,"context_line":"            self.backups[0].availability_zone,"},{"line_number":210,"context_line":"            self.backups[0].volume_id,"},{"line_number":211,"context_line":"            self.backups[0].container"},{"line_number":212,"context_line":"        ),)"},{"line_number":213,"context_line":"        self.assertEqual(datalist, tuple(data))"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"dab17558_36503e32","line":212,"range":{"start_line":203,"start_character":0,"end_line":212,"end_character":11},"updated":"2016-05-16 05:40:31.000000000","message":"Please also move this to the beginning of the class. You can name it data_long or something similar.","commit_id":"5e7d3dd878fe349c8f682003e79f4027cfb9769d"}]}
