)]}'
{"tempest/tests/test_list_images.py":[{"author":{"_account_id":97,"name":"Daryl Walleck","email":"daryl.walleck@rackspace.com","username":"dwalleck"},"change_message_id":"62e0a39458a93686ed4f4d187c9cd3d7df8ff2ce","unresolved":false,"context_lines":[{"line_number":45,"context_line":"                self.fail(\u0027GET on non existant image should fail\u0027)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    @attr(type\u003d\u0027negative\u0027)"},{"line_number":48,"context_line":"    def test_list_nonexistant_images_with_detail(self):"},{"line_number":49,"context_line":"        \"\"\"Negative test: LIST details on non existant images should fail\"\"\""},{"line_number":50,"context_line":"        resp, images \u003d self.client.list_images_with_detail()"},{"line_number":51,"context_line":"        found \u003d any([i for i in images if i[\u0027id\u0027] \u003d\u003d 999])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAC3%2F%2F678%3D","line":48,"updated":"2011-12-22 20:45:35.000000000","message":"I\u0027m not sure what this test accomplishes as the server id is arbitrary. I could see a test case where checking to see if a deleted server no longer appears in the list of servers or to make sure that a server that does not match a filter is not in the list. Otherwise, I don\u0027t think this test accomplishes anything.","commit_id":"6d62deab6dfb1e95b877072908648201c5fcc6e0"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"dcce83c1ef7c62dd7bdcdb855e966ebafd9067ee","unresolved":false,"context_lines":[{"line_number":45,"context_line":"                self.fail(\u0027GET on non existant image should fail\u0027)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    @attr(type\u003d\u0027negative\u0027)"},{"line_number":48,"context_line":"    def test_list_nonexistant_images_with_detail(self):"},{"line_number":49,"context_line":"        \"\"\"Negative test: LIST details on non existant images should fail\"\"\""},{"line_number":50,"context_line":"        resp, images \u003d self.client.list_images_with_detail()"},{"line_number":51,"context_line":"        found \u003d any([i for i in images if i[\u0027id\u0027] \u003d\u003d 999])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2FvY%3D","line":48,"in_reply_to":"AAAAC3%2F%2F678%3D","updated":"2011-12-29 13:36:22.000000000","message":"Agreed. A more appropriate negative test would be to get the list of images and then do a GET on an image ID that was not in the list...","commit_id":"6d62deab6dfb1e95b877072908648201c5fcc6e0"}]}