)]}'
{"openstackclient/tests/volume/v2/test_volume.py":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"953eaf343e7f86b8ee49c150bef0bcb355998018","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        self._test_init_args()"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def _test_init_args(self):"},{"line_number":43,"context_line":"        self.collist \u003d ("},{"line_number":44,"context_line":"            \u0027attachments\u0027,"},{"line_number":45,"context_line":"            \u0027availability_zone\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_5f90d4ec","line":42,"updated":"2015-12-03 18:28:54.000000000","message":"Why not just set these up directly as class attributes?  I don\u0027t think they would be modified, requiring them to be reinitialized for each test.","commit_id":"a673ea748e0fc46a69a005f382d23def0ebf6f18"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"953eaf343e7f86b8ee49c150bef0bcb355998018","unresolved":false,"context_lines":[{"line_number":491,"context_line":""},{"line_number":492,"context_line":"        columns, data \u003d self.cmd.take_action(parsed_args)"},{"line_number":493,"context_line":""},{"line_number":494,"context_line":"        self.assertEqual(list(self.collist_list), columns)"},{"line_number":495,"context_line":""},{"line_number":496,"context_line":"        server \u003d volume_fakes.volume_attachment_server[\u0027server_id\u0027]"},{"line_number":497,"context_line":"        device \u003d volume_fakes.volume_attachment_server[\u0027device\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_ff9420e9","line":494,"updated":"2015-12-03 18:28:54.000000000","message":"I prefer the form used below on line 522 of using tuple(columns).  Either way we should do the same thing within tests for the same command.","commit_id":"a673ea748e0fc46a69a005f382d23def0ebf6f18"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"c501ddeb9b9e616bf60c6aa62c04ab67f5ea47f5","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        \u0027type\u0027,"},{"line_number":38,"context_line":"    )"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    collist_list \u003d ("},{"line_number":41,"context_line":"        \u0027ID\u0027,"},{"line_number":42,"context_line":"        \u0027Display Name\u0027,"},{"line_number":43,"context_line":"        \u0027Status\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_6a7be58c","line":40,"updated":"2015-12-08 08:22:09.000000000","message":"It is about the name again.\n\nI think this is column_headers.","commit_id":"005d78e56f85dca2520a1237f3c62b389ef653e4"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"c501ddeb9b9e616bf60c6aa62c04ab67f5ea47f5","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        \u0027Attached to\u0027,"},{"line_number":46,"context_line":"    )"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    collist_list_long \u003d ("},{"line_number":49,"context_line":"        \u0027ID\u0027,"},{"line_number":50,"context_line":"        \u0027Display Name\u0027,"},{"line_number":51,"context_line":"        \u0027Status\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_0a6ca1ce","line":48,"updated":"2015-12-08 08:22:09.000000000","message":"The same here.","commit_id":"005d78e56f85dca2520a1237f3c62b389ef653e4"}]}