)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f5d23522f9229197464e6f3b95e493f1342e66d1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     zhufl \u003czhu.fanglei@zte.com.cn\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-11-24 14:46:42 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"It is better not to use 1234 as non_existing_id"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In test_list_servers_by_non_existing_flavor, 1234 is used as"},{"line_number":10,"context_line":"non_existing_flavor, but it is not strange if someone happens"},{"line_number":11,"context_line":"to use 1234 as flavor id in his environment."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Besides, we can use something like \u0027invalid_server_name\u0027 instead"},{"line_number":14,"context_line":"of a hard-coded strange name for invalid names."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a74a57a_7c73c880","line":11,"range":{"start_line":7,"start_character":0,"end_line":11,"end_character":44},"updated":"2016-11-24 09:07:00.000000000","message":"make it more generic as it is not just changing flavor also others","commit_id":"8c5c051d43f13b38df0fa8f6014dd3e321e2d1a9"}],"tempest/api/compute/servers/test_list_servers_negative.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f5d23522f9229197464e6f3b95e493f1342e66d1","unresolved":false,"context_lines":[{"line_number":68,"context_line":"    def test_list_servers_by_non_existing_image(self):"},{"line_number":69,"context_line":"        # Listing servers for a non existing image returns empty list"},{"line_number":70,"context_line":"        non_existing_image \u003d data_utils.rand_uuid()"},{"line_number":71,"context_line":"        body \u003d self.client.list_servers(image\u003dnon_existing_image)"},{"line_number":72,"context_line":"        servers \u003d body[\u0027servers\u0027]"},{"line_number":73,"context_line":"        self.assertEqual([], servers)"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_1c3bb49d","line":71,"range":{"start_line":71,"start_character":46,"end_line":71,"end_character":64},"updated":"2016-11-24 09:07:00.000000000","message":"\u0027non_existing_image\u0027 would not work here?","commit_id":"8c5c051d43f13b38df0fa8f6014dd3e321e2d1a9"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f5d23522f9229197464e6f3b95e493f1342e66d1","unresolved":false,"context_lines":[{"line_number":76,"context_line":"    @test.idempotent_id(\u00275913660b-223b-44d4-a651-a0fbfd44ca75\u0027)"},{"line_number":77,"context_line":"    def test_list_servers_by_non_existing_flavor(self):"},{"line_number":78,"context_line":"        # Listing servers by non existing flavor returns empty list"},{"line_number":79,"context_line":"        non_existing_flavor \u003d data_utils.rand_uuid()"},{"line_number":80,"context_line":"        body \u003d self.client.list_servers(flavor\u003dnon_existing_flavor)"},{"line_number":81,"context_line":"        servers \u003d body[\u0027servers\u0027]"},{"line_number":82,"context_line":"        self.assertEqual([], servers)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_26993f20","line":79,"range":{"start_line":79,"start_character":30,"end_line":79,"end_character":52},"updated":"2016-11-24 09:07:00.000000000","message":"this can be \"non_existing_flavor\"","commit_id":"8c5c051d43f13b38df0fa8f6014dd3e321e2d1a9"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f5d23522f9229197464e6f3b95e493f1342e66d1","unresolved":false,"context_lines":[{"line_number":93,"context_line":"    @test.idempotent_id(\u0027fcdf192d-0f74-4d89-911f-1ec002b822c4\u0027)"},{"line_number":94,"context_line":"    def test_list_servers_status_non_existing(self):"},{"line_number":95,"context_line":"        # Return an empty list when invalid status is specified"},{"line_number":96,"context_line":"        body \u003d self.client.list_servers(status\u003d\u0027NON_EXISTING_STATUS\u0027)"},{"line_number":97,"context_line":"        servers \u003d body[\u0027servers\u0027]"},{"line_number":98,"context_line":"        self.assertEqual([], servers)"},{"line_number":99,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_8603f341","line":96,"range":{"start_line":96,"start_character":48,"end_line":96,"end_character":67},"updated":"2016-11-24 09:07:00.000000000","message":"you can keep this also as lower case","commit_id":"8c5c051d43f13b38df0fa8f6014dd3e321e2d1a9"}]}