)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6983,"name":"YangLei","email":"iamhappg@aliyun.com","username":"YangLei"},"change_message_id":"77c7ff180c69afd0f4fa426dbf048ca4dbde8ae1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Abhijeet Malawade \u003cAbhijeet.Malawade@nttdata.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-09-17 22:48:52 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"GET Volume tests for APIs that accept params (p1)"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"First patch of a series that adds tests for those GET API"},{"line_number":10,"context_line":"functions that accept params"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAS3%2F%2F2Ko%3D","line":7,"updated":"2013-09-18 10:17:35.000000000","message":"I think the follow summary is better.\nAdd testcases for volume","commit_id":"8dbe01047926f7f4f1280ebe04ca8adc0d85892d"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"f04df6a05046043d3a1cbef7b59fcc6c4c589875","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Abhijeet Malawade \u003cAbhijeet.Malawade@nttdata.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-09-17 22:48:52 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"GET Volume tests for APIs that accept params (p1)"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"First patch of a series that adds tests for those GET API"},{"line_number":10,"context_line":"functions that accept params"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAS3%2F%2F1Cw%3D","line":7,"in_reply_to":"AAAAS3%2F%2F2Ko%3D","updated":"2013-09-18 16:07:35.000000000","message":"+1 agree, you\u0027re actually adding tests around the list actions","commit_id":"8dbe01047926f7f4f1280ebe04ca8adc0d85892d"},{"author":{"_account_id":6983,"name":"YangLei","email":"iamhappg@aliyun.com","username":"YangLei"},"change_message_id":"77c7ff180c69afd0f4fa426dbf048ca4dbde8ae1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"GET Volume tests for APIs that accept params (p1)"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"First patch of a series that adds tests for those GET API"},{"line_number":10,"context_line":"functions that accept params"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"* list_volumes and list_volumes_with_detail for nova volume extension"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAS3%2F%2F2I0%3D","line":9,"updated":"2013-09-18 10:17:35.000000000","message":"\"First patch of a series that\" is not needed.","commit_id":"8dbe01047926f7f4f1280ebe04ca8adc0d85892d"}],"tempest/api/compute/volumes/test_volumes_list.py":[{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"f04df6a05046043d3a1cbef7b59fcc6c4c589875","unresolved":false,"context_lines":[{"line_number":117,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":118,"context_line":"        volume_id_list \u003d map(lambda v: v[\u0027id\u0027], fetched_vol_list)"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"        for volume_id in volume_id_list:"},{"line_number":121,"context_line":"            self.assertIn(volume_id, self.volume_id_list)"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"        self.assertEqual(len(fetched_vol_list), params[\u0027limit\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2F1GM%3D","line":120,"updated":"2013-09-18 16:07:35.000000000","message":"you don\u0027t need to perform this check here; same applies to below tests","commit_id":"8dbe01047926f7f4f1280ebe04ca8adc0d85892d"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"f04df6a05046043d3a1cbef7b59fcc6c4c589875","unresolved":false,"context_lines":[{"line_number":135,"context_line":"        for volume_id in volume_id_list:"},{"line_number":136,"context_line":"            self.assertIn(volume_id, self.volume_id_list)"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"        self.assertEqual(len(fetched_vol_list), 2,"},{"line_number":139,"context_line":"                         \"Number of volumes fetched is not equal to limit %s\" %"},{"line_number":140,"context_line":"                         params[\u0027limit\u0027])"},{"line_number":141,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2F1FY%3D","line":138,"updated":"2013-09-18 16:07:35.000000000","message":"you should probably use params[\u0027limit\u0027] rather than \"2\"","commit_id":"8dbe01047926f7f4f1280ebe04ca8adc0d85892d"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"f04df6a05046043d3a1cbef7b59fcc6c4c589875","unresolved":false,"context_lines":[{"line_number":152,"context_line":"        self.assertEqual(len(fetched_vol_list), params[\u0027limit\u0027],"},{"line_number":153,"context_line":"                         \"Number of volumes fetched is not equal to limit %s\" %"},{"line_number":154,"context_line":"                         params[\u0027limit\u0027])"},{"line_number":155,"context_line":"        self.assertEqual(volume_id_list[0], self.volume_id_list[1],"},{"line_number":156,"context_line":"                         \"Fetched volume id %s does not match with actual \\"},{"line_number":157,"context_line":"                         volume id %s\" %"},{"line_number":158,"context_line":"                         (volume_id_list[0], self.volume_id_list[1]))"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2F1EU%3D","line":155,"updated":"2013-09-18 16:07:35.000000000","message":"do we have any guarantee that the output order for the listing action will be identical to what is in self.volume_id_list?","commit_id":"8dbe01047926f7f4f1280ebe04ca8adc0d85892d"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"77b5e83532cd4266d237a69236df6acae5b4543e","unresolved":false,"context_lines":[{"line_number":142,"context_line":""},{"line_number":143,"context_line":"        # Validating length of the fetched volumes"},{"line_number":144,"context_line":"        self.assertEqual(len(fetched_vol_list), params[\u0027limit\u0027],"},{"line_number":145,"context_line":"                         \"Failed to list volumes by offset and limit\")"},{"line_number":146,"context_line":"        # Validating offset of fetched volume"},{"line_number":147,"context_line":"        for index, volume in enumerate(fetched_vol_list):"},{"line_number":148,"context_line":"            self.assertEqual(volume[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F%2FIig%3D","line":145,"updated":"2013-11-14 01:25:32.000000000","message":"maybe update the message saying it failed the limit assertion","commit_id":"bf15451976fefccf05970674a4c91d4d25ed1a0d"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"77b5e83532cd4266d237a69236df6acae5b4543e","unresolved":false,"context_lines":[{"line_number":147,"context_line":"        for index, volume in enumerate(fetched_vol_list):"},{"line_number":148,"context_line":"            self.assertEqual(volume[\u0027id\u0027],"},{"line_number":149,"context_line":"                             all_vol_list[index + params[\u0027offset\u0027]][\u0027id\u0027],"},{"line_number":150,"context_line":"                             \"Failed to list volumes by offset and limit\")"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":153,"context_line":"    def test_volume_list_with_detail_param_offset_and_limit(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F%2FIiY%3D","line":150,"updated":"2013-11-14 01:25:32.000000000","message":"as a consequence of previous comment, update the message saying it failed the offset assertion","commit_id":"bf15451976fefccf05970674a4c91d4d25ed1a0d"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"77b5e83532cd4266d237a69236df6acae5b4543e","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        resp, fetched_vol_list \u003d \\"},{"line_number":159,"context_line":"            self.client.list_volumes_with_detail(params\u003dparams)"},{"line_number":160,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"        # Validating length of the fetched volumes"},{"line_number":163,"context_line":"        self.assertEqual(len(fetched_vol_list), params[\u0027limit\u0027],"},{"line_number":164,"context_line":"                         \"Failed to list volume details by offset and limit\")"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F%2FIiI%3D","line":161,"updated":"2013-11-14 01:25:32.000000000","message":"I\u0027d apply previous comments to the next two asserts too","commit_id":"bf15451976fefccf05970674a4c91d4d25ed1a0d"}],"tempest/api/volume/test_volumes_list.py":[{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"f04df6a05046043d3a1cbef7b59fcc6c4c589875","unresolved":false,"context_lines":[{"line_number":115,"context_line":""},{"line_number":116,"context_line":"    def test_volume_list_with_detail_param_display_name(self):"},{"line_number":117,"context_line":"        # List volume detail by display name"},{"line_number":118,"context_line":"        volume \u003d self.volume_list[0]"},{"line_number":119,"context_line":"        params \u003d {\u0027display_name\u0027: volume[\u0027display_name\u0027]}"},{"line_number":120,"context_line":"        resp, fetched_vol \u003d self.client.list_volumes_with_detail(params\u003dparams)"},{"line_number":121,"context_line":"        self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2F1Bk%3D","line":118,"updated":"2013-09-18 16:07:35.000000000","message":"I\u0027d take anything which isn\u0027t [0] here","commit_id":"8dbe01047926f7f4f1280ebe04ca8adc0d85892d"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"f04df6a05046043d3a1cbef7b59fcc6c4c589875","unresolved":false,"context_lines":[{"line_number":175,"context_line":""},{"line_number":176,"context_line":"    def test_volume_list_param_status_and_metadata(self):"},{"line_number":177,"context_line":"        # Test to list volume when metadata and status param is given"},{"line_number":178,"context_line":"        volume \u003d self.volume_list[0]"},{"line_number":179,"context_line":"        params \u003d {\u0027metadata\u0027: {\u0027Type\u0027: \u0027work\u0027}, \u0027status\u0027: \u0027deleted\u0027}"},{"line_number":180,"context_line":"        resp, fetched_vol_list \u003d self.client.list_volumes(params\u003dparams)"},{"line_number":181,"context_line":"        self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2F1B8%3D","line":178,"updated":"2013-09-18 16:07:35.000000000","message":"what is this needed for ?","commit_id":"8dbe01047926f7f4f1280ebe04ca8adc0d85892d"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"f04df6a05046043d3a1cbef7b59fcc6c4c589875","unresolved":false,"context_lines":[{"line_number":176,"context_line":"    def test_volume_list_param_status_and_metadata(self):"},{"line_number":177,"context_line":"        # Test to list volume when metadata and status param is given"},{"line_number":178,"context_line":"        volume \u003d self.volume_list[0]"},{"line_number":179,"context_line":"        params \u003d {\u0027metadata\u0027: {\u0027Type\u0027: \u0027work\u0027}, \u0027status\u0027: \u0027deleted\u0027}"},{"line_number":180,"context_line":"        resp, fetched_vol_list \u003d self.client.list_volumes(params\u003dparams)"},{"line_number":181,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":182,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2F1Ac%3D","line":179,"updated":"2013-09-18 16:07:35.000000000","message":"do we have any \u0027deleted\u0027 volume at this point? wouldn\u0027t it be better to use \u0027available\u0027 ?","commit_id":"8dbe01047926f7f4f1280ebe04ca8adc0d85892d"},{"author":{"_account_id":8336,"name":"QianLin","email":"lin.qian@huawei.com","username":"qianlin"},"change_message_id":"b2a1c7399a6cc7375b73ef25db002c257d9e59fa","unresolved":false,"context_lines":[{"line_number":165,"context_line":"            self.assertEqual(zone, volume[\u0027availability_zone\u0027])"},{"line_number":166,"context_line":"        self.assertVolumesIn(fetched_list, self.volume_list)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    def test_volume_list_with_param_metadata(self):"},{"line_number":169,"context_line":"        # Test to list volumes when metadata param is given"},{"line_number":170,"context_line":"        params \u003d {\u0027metadata\u0027: {\u0027Type\u0027: \u0027work\u0027}}"},{"line_number":171,"context_line":"        resp, fetched_vol_list \u003d self.client.list_volumes(params\u003dparams)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FVK0%3D","line":168,"updated":"2013-11-10 07:48:13.000000000","message":"I think you miss \"@attr(type\u003d\u0027gate\u0027)\"","commit_id":"3dac77434419e7430b1f0ec4793226132628d7bb"},{"author":{"_account_id":8336,"name":"QianLin","email":"lin.qian@huawei.com","username":"qianlin"},"change_message_id":"b2a1c7399a6cc7375b73ef25db002c257d9e59fa","unresolved":false,"context_lines":[{"line_number":176,"context_line":"                             params[\u0027metadata\u0027][\u0027Type\u0027],"},{"line_number":177,"context_line":"                             \"Failed to list volumes by metadata\")"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"    def test_volume_list_with_detail_param_metadata(self):"},{"line_number":180,"context_line":"        # Test to list volumes details when metadata param is given"},{"line_number":181,"context_line":"        params \u003d {\u0027metadata\u0027: {\u0027Type\u0027: \u0027work\u0027}}"},{"line_number":182,"context_line":"        resp, fetched_vol_list \u003d \\"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FVKw%3D","line":179,"updated":"2013-11-10 07:48:13.000000000","message":"ditto","commit_id":"3dac77434419e7430b1f0ec4793226132628d7bb"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"984544c07393f0f57faaac5529784829e09b1b0a","unresolved":false,"context_lines":[{"line_number":189,"context_line":"                             \"Failed to list volume details by metadata\")"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":192,"context_line":"    def test_volume_list_param_status_and_metadata(self):"},{"line_number":193,"context_line":"        # Test to list volume when metadata and status param is given"},{"line_number":194,"context_line":"        params \u003d {\u0027metadata\u0027: {\u0027Type\u0027: \u0027work\u0027},"},{"line_number":195,"context_line":"                  \u0027status\u0027: \u0027available\u0027}"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2FYVk%3D","line":192,"updated":"2013-10-16 21:51:13.000000000","message":"I think this test and the one @208 are not really needed, we have tests which list by status already","commit_id":"3dac77434419e7430b1f0ec4793226132628d7bb"},{"author":{"_account_id":8336,"name":"QianLin","email":"lin.qian@huawei.com","username":"qianlin"},"change_message_id":"b2a1c7399a6cc7375b73ef25db002c257d9e59fa","unresolved":false,"context_lines":[{"line_number":189,"context_line":"                             \"Failed to list volume details by metadata\")"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":192,"context_line":"    def test_volume_list_param_status_and_metadata(self):"},{"line_number":193,"context_line":"        # Test to list volume when metadata and status param is given"},{"line_number":194,"context_line":"        params \u003d {\u0027metadata\u0027: {\u0027Type\u0027: \u0027work\u0027},"},{"line_number":195,"context_line":"                  \u0027status\u0027: \u0027available\u0027}"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FVKQ%3D","line":192,"in_reply_to":"AAAATH%2F%2FTM4%3D","updated":"2013-11-10 07:48:13.000000000","message":"I agree with Giulio, it is not necessary when \"Failed to list volumes by metadata\".if you want to verify multiple search filters,i think \"name\" and \"status\" are better.","commit_id":"3dac77434419e7430b1f0ec4793226132628d7bb"},{"author":{"_account_id":8300,"name":"Abhijeet Malawade","email":"abhijeet.malawade@gmail.com","username":"abhijeet"},"change_message_id":"9af309a163d2c2b12255fb2485f26d7cc289a256","unresolved":false,"context_lines":[{"line_number":189,"context_line":"                             \"Failed to list volume details by metadata\")"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":192,"context_line":"    def test_volume_list_param_status_and_metadata(self):"},{"line_number":193,"context_line":"        # Test to list volume when metadata and status param is given"},{"line_number":194,"context_line":"        params \u003d {\u0027metadata\u0027: {\u0027Type\u0027: \u0027work\u0027},"},{"line_number":195,"context_line":"                  \u0027status\u0027: \u0027available\u0027}"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2FTM4%3D","line":192,"in_reply_to":"AAAATH%2F%2FYVk%3D","updated":"2013-10-18 05:49:31.000000000","message":"We have tests for separate search filters like status, metadata etc. But we don\u0027t have test cases which  test multiple search filters. This test case and and bellow test case verifies multiple search filters.","commit_id":"3dac77434419e7430b1f0ec4793226132628d7bb"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"77b5e83532cd4266d237a69236df6acae5b4543e","unresolved":false,"context_lines":[{"line_number":168,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":169,"context_line":"    def test_volume_list_with_param_metadata(self):"},{"line_number":170,"context_line":"        # Test to list volumes when metadata param is given"},{"line_number":171,"context_line":"        params \u003d {\u0027metadata\u0027: {\u0027Type\u0027: \u0027work\u0027}}"},{"line_number":172,"context_line":"        resp, fetched_vol_list \u003d self.client.list_volumes(params\u003dparams)"},{"line_number":173,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":174,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F%2FIhM%3D","line":171,"updated":"2013-11-14 01:25:32.000000000","message":"this could fail if someone changes the metadata passed in setUpClass ; could you please save that in self. and reuse it here?","commit_id":"bf15451976fefccf05970674a4c91d4d25ed1a0d"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"77b5e83532cd4266d237a69236df6acae5b4543e","unresolved":false,"context_lines":[{"line_number":180,"context_line":"    @attr(type\u003d\u0027gate\u0027)"},{"line_number":181,"context_line":"    def test_volume_list_with_detail_param_metadata(self):"},{"line_number":182,"context_line":"        # Test to list volumes details when metadata param is given"},{"line_number":183,"context_line":"        params \u003d {\u0027metadata\u0027: {\u0027Type\u0027: \u0027work\u0027}}"},{"line_number":184,"context_line":"        resp, fetched_vol_list \u003d \\"},{"line_number":185,"context_line":"            self.client.list_volumes_with_detail(params\u003dparams)"},{"line_number":186,"context_line":"        self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F%2FIhI%3D","line":183,"updated":"2013-11-14 01:25:32.000000000","message":"ditto","commit_id":"bf15451976fefccf05970674a4c91d4d25ed1a0d"}],"tempest/api/volume/test_volumes_snapshots.py":[{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"77b5e83532cd4266d237a69236df6acae5b4543e","unresolved":false,"context_lines":[{"line_number":98,"context_line":"            self.assertEqual(snap[\u0027display_name\u0027], params[\u0027display_name\u0027],"},{"line_number":99,"context_line":"                             \"Failed to list snapshots by display name\")"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"        # Verify list snapshots by volume_id filter"},{"line_number":102,"context_line":"        params \u003d {\u0027volume_id\u0027: snapshot[\u0027volume_id\u0027]}"},{"line_number":103,"context_line":"        resp, snapshots \u003d self.snapshots_client.list_snapshots(params\u003dparams)"},{"line_number":104,"context_line":"        self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F%2FIgc%3D","line":101,"updated":"2013-11-14 01:25:32.000000000","message":"I\u0027d drop this, it seems overkill","commit_id":"bf15451976fefccf05970674a4c91d4d25ed1a0d"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"77b5e83532cd4266d237a69236df6acae5b4543e","unresolved":false,"context_lines":[{"line_number":114,"context_line":"            self.assertEqual(snap[\u0027status\u0027], params[\u0027status\u0027],"},{"line_number":115,"context_line":"                             \"Failed to list snapshots by status\")"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"        # Verify list snapshots by status and volume id filter"},{"line_number":118,"context_line":"        params \u003d {\u0027status\u0027: \u0027available\u0027,"},{"line_number":119,"context_line":"                  \u0027display_name\u0027: snapshot[\u0027display_name\u0027]}"},{"line_number":120,"context_line":"        resp, snapshots \u003d self.snapshots_client.list_snapshots(params\u003dparams)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F%2FIgU%3D","line":117,"updated":"2013-11-14 01:25:32.000000000","message":"if you want to test two params, maybe use status and display_name here","commit_id":"bf15451976fefccf05970674a4c91d4d25ed1a0d"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"77b5e83532cd4266d237a69236df6acae5b4543e","unresolved":false,"context_lines":[{"line_number":145,"context_line":"                             \"Failed to list snapshot detail by display name\")"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"        # Verify list snapshot details by volume_id filter"},{"line_number":148,"context_line":"        params \u003d {\u0027volume_id\u0027: snapshot[\u0027volume_id\u0027]}"},{"line_number":149,"context_line":"        resp, snapshots \u003d \\"},{"line_number":150,"context_line":"            self.snapshots_client.list_snapshots_with_detail(params\u003dparams)"},{"line_number":151,"context_line":"        self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F%2FIgM%3D","line":148,"updated":"2013-11-14 01:25:32.000000000","message":"same as @101","commit_id":"bf15451976fefccf05970674a4c91d4d25ed1a0d"}]}