)]}'
{"tempest/cli/simple_read_only/test_cinder.py":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"81d782c0cce1a9e420cf332ac8d89e89a7371216","unresolved":false,"context_lines":[{"line_number":119,"context_line":"            region \u003d self.config.identity.region"},{"line_number":120,"context_line":"        self.cinder(\u0027list\u0027, flags\u003d\u0027--os-region-name \u0027 + region)"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    # Test based on v2-api"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"    def test_cinder_volumes_list_in_v2(self):"},{"line_number":125,"context_line":"        \"\"\"This CLI representation in v1/v2 is different.\"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2FZeA%3D","line":122,"updated":"2013-10-16 17:47:11.000000000","message":"Nit: It would be nice to split this out into a separate class","commit_id":"2076ff728a653c3530d616cfcff1f498fadc1481"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"81d782c0cce1a9e420cf332ac8d89e89a7371216","unresolved":false,"context_lines":[{"line_number":142,"context_line":"                                     flags\u003d\u0027--os-volume-api-version 2\u0027))"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"    def test_cinder_backup_list_in_v2(self):"},{"line_number":145,"context_line":"        self.assertNotEqual(\"\","},{"line_number":146,"context_line":"                            self.cinder(\u0027backup-list\u0027,"},{"line_number":147,"context_line":"                                        flags\u003d\u0027--os-volume-api-version 2\u0027))"},{"line_number":148,"context_line":"        self.assertEqual(self.cinder(\u0027backup-list\u0027),"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2FZdQ%3D","line":145,"updated":"2013-10-16 17:47:11.000000000","message":"It looks like this can be made DRYer you have a lot of duplicate code.\n\n assertNotEqual(\"\",cmd)\n assert*Equal(V1,V2)","commit_id":"2076ff728a653c3530d616cfcff1f498fadc1481"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"fa1ca077ac5e190981a8f1744e6dac9babf4e732","unresolved":false,"context_lines":[{"line_number":162,"context_line":"        self.assertNotEqual(\"\", v1)"},{"line_number":163,"context_line":"        self.assertNotEqual(\"\", v2)"},{"line_number":164,"context_line":"        # This CLI representation in v1/v2 is different."},{"line_number":165,"context_line":"        self.assertNotEqual(v1, v2)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"    def test_cinder_absolute_limit_list_in_v2(self):"},{"line_number":168,"context_line":"        cmd \u003d \u0027absolute-limits\u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F%2BaA0%3D","line":165,"updated":"2013-11-26 13:16:05.000000000","message":"Since these blocks are always repeated, can we move it into a function?","commit_id":"6e59a2c6372650a3d31c96e85ef239b0f008dd17"}]}