)]}'
{"tempest/tests/test_server_actions.py":[{"author":{"_account_id":97,"name":"Daryl Walleck","email":"daryl.walleck@rackspace.com","username":"dwalleck"},"change_message_id":"e877461515cf1c0833e7edebe0018884ca0a0d00","unresolved":false,"context_lines":[{"line_number":118,"context_line":"        \"\"\"Negative Test: The server reboot on non"},{"line_number":119,"context_line":"                          existant server should return an error\"\"\""},{"line_number":120,"context_line":"        resp, body \u003d self.client.reboot(999, \u0027SOFT\u0027)"},{"line_number":121,"context_line":"        self.assertEqual(422, resp.status)"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    @attr(type\u003d\u0027negative\u0027)"},{"line_number":124,"context_line":"    def test_rebuild_nonexistant_server(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAC3%2F%2F688%3D","line":121,"updated":"2011-12-22 19:52:12.000000000","message":"This should not be excepting a 422. A 404 is what is expected, which is causing this test to fail.","commit_id":"ac94eba78bf51c18d983783a6da188a454982694"},{"author":{"_account_id":97,"name":"Daryl Walleck","email":"daryl.walleck@rackspace.com","username":"dwalleck"},"change_message_id":"e877461515cf1c0833e7edebe0018884ca0a0d00","unresolved":false,"context_lines":[{"line_number":134,"context_line":"                                       self.image_ref_alt, name\u003dnew_name,"},{"line_number":135,"context_line":"                                       meta\u003dmeta, personality\u003dpersonality,"},{"line_number":136,"context_line":"                                       adminPass\u003d\u0027rebuild\u0027)"},{"line_number":137,"context_line":"                print resp"},{"line_number":138,"context_line":"        except:"},{"line_number":139,"context_line":"                pass"},{"line_number":140,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAC3%2F%2F684%3D","line":137,"updated":"2011-12-22 19:52:12.000000000","message":"Please remove this print statement","commit_id":"ac94eba78bf51c18d983783a6da188a454982694"},{"author":{"_account_id":97,"name":"Daryl Walleck","email":"daryl.walleck@rackspace.com","username":"dwalleck"},"change_message_id":"e877461515cf1c0833e7edebe0018884ca0a0d00","unresolved":false,"context_lines":[{"line_number":135,"context_line":"                                       meta\u003dmeta, personality\u003dpersonality,"},{"line_number":136,"context_line":"                                       adminPass\u003d\u0027rebuild\u0027)"},{"line_number":137,"context_line":"                print resp"},{"line_number":138,"context_line":"        except:"},{"line_number":139,"context_line":"                pass"},{"line_number":140,"context_line":"        else:"},{"line_number":141,"context_line":"                self.fail(\u0027The server rebuild for a non \\"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAC3%2F%2F680%3D","line":138,"updated":"2011-12-22 19:52:12.000000000","message":"In the very near future, we should explicitly catch a 404 exception instead of any exception.","commit_id":"ac94eba78bf51c18d983783a6da188a454982694"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"435dc126b1e5b33c05f889028eabb604521767b5","unresolved":false,"context_lines":[{"line_number":118,"context_line":"        \"\"\"Negative Test: The server reboot on non"},{"line_number":119,"context_line":"                          existant server should return an error\"\"\""},{"line_number":120,"context_line":"        resp, body \u003d self.client.reboot(999, \u0027SOFT\u0027)"},{"line_number":121,"context_line":"        self.assertEqual(422, resp.status)"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    @attr(type\u003d\u0027negative\u0027)"},{"line_number":124,"context_line":"    def test_rebuild_nonexistant_server(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAADH%2F%2F%2FdE%3D","line":121,"updated":"2011-12-30 16:12:41.000000000","message":"This should be a 404. Spec says 404, so that\u0027s what it should test :)","commit_id":"39cc41fab371fd4548b097b558fef7bd0c22655d"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"435dc126b1e5b33c05f889028eabb604521767b5","unresolved":false,"context_lines":[{"line_number":134,"context_line":"                                       self.image_ref_alt, name\u003dnew_name,"},{"line_number":135,"context_line":"                                       meta\u003dmeta, personality\u003dpersonality,"},{"line_number":136,"context_line":"                                       adminPass\u003d\u0027rebuild\u0027)"},{"line_number":137,"context_line":"                print resp"},{"line_number":138,"context_line":"        except:"},{"line_number":139,"context_line":"                pass"},{"line_number":140,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAADH%2F%2F%2Fc0%3D","line":137,"updated":"2011-12-30 16:12:41.000000000","message":"Please fix the indentation to make sure all indentations are 4 space indents. Thanks!\n-jay","commit_id":"39cc41fab371fd4548b097b558fef7bd0c22655d"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"108f6fbd1b66fdebde720ec34f94666aaf64cd28","unresolved":false,"context_lines":[{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    @attr(type\u003d\u0027negative\u0027)"},{"line_number":121,"context_line":"    def test_reboot_nonexistant_server_soft(self):"},{"line_number":122,"context_line":"        \"\"\"Negative Test: The server reboot on non \\"},{"line_number":123,"context_line":" existant server should return an error\"\"\""},{"line_number":124,"context_line":"        resp, body \u003d self.client.reboot(999, \u0027SOFT\u0027)"},{"line_number":125,"context_line":"        self.assertEqual(404, resp.status)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAADX%2F%2F%2F9o%3D","line":122,"updated":"2012-01-05 18:38:04.000000000","message":"You don\u0027t need the \\ when using a \"\"\" string","commit_id":"2e6b7ec63adeb1d71efe5745ab6c2f6782e984dd"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"108f6fbd1b66fdebde720ec34f94666aaf64cd28","unresolved":false,"context_lines":[{"line_number":127,"context_line":"    @attr(type\u003d\u0027negative\u0027)"},{"line_number":128,"context_line":"    def test_rebuild_nonexistant_server(self):"},{"line_number":129,"context_line":"        \"\"\" Negative test: The server rebuild for a"},{"line_number":130,"context_line":"                               non existing server should not be allowed\"\"\""},{"line_number":131,"context_line":"        meta \u003d {\u0027rebuild\u0027: \u0027server\u0027}"},{"line_number":132,"context_line":"        new_name \u003d rand_name(\u0027server\u0027)"},{"line_number":133,"context_line":"        file_contents \u003d \u0027Test server rebuild.\u0027"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAADX%2F%2F%2F9k%3D","line":130,"updated":"2012-01-05 18:38:04.000000000","message":"Couple little style things:\n\na) Remove the extra whitespace between \"\"\" and Negatve\nb) Align the two lines of text with a common indentation","commit_id":"2e6b7ec63adeb1d71efe5745ab6c2f6782e984dd"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"108f6fbd1b66fdebde720ec34f94666aaf64cd28","unresolved":false,"context_lines":[{"line_number":137,"context_line":"            resp, rebuilt_server \u003d self.client.rebuild(999,"},{"line_number":138,"context_line":"                                   self.image_ref_alt, name\u003dnew_name,"},{"line_number":139,"context_line":"                                   meta\u003dmeta, personality\u003dpersonality,"},{"line_number":140,"context_line":"                                   adminPass\u003d\u0027rebuild\u0027)"},{"line_number":141,"context_line":"        except:"},{"line_number":142,"context_line":"            pass"},{"line_number":143,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAADX%2F%2F%2F9c%3D","line":140,"updated":"2012-01-05 18:38:04.000000000","message":"Please indent the lines after:\n\n self.client.rebuild(999,\n\nSo that is is easier to tell that the 4 lines are a single method call...","commit_id":"2e6b7ec63adeb1d71efe5745ab6c2f6782e984dd"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"d3bd317d5d789dd7c5913ec946092c241126d4e7","unresolved":false,"context_lines":[{"line_number":142,"context_line":"                                       self.image_ref_alt, name\u003dnew_name,"},{"line_number":143,"context_line":"                                       meta\u003dmeta, personality\u003dpersonality,"},{"line_number":144,"context_line":"                                       adminPass\u003d\u0027rebuild\u0027)"},{"line_number":145,"context_line":"        except:"},{"line_number":146,"context_line":"            pass"},{"line_number":147,"context_line":"        else:"},{"line_number":148,"context_line":"            self.fail(\u0027The server rebuild for a non existing server should not\u0027"}],"source_content_type":"text/x-python","patch_set":15,"id":"AAAAD3%2F%2F798%3D","line":145,"updated":"2012-01-20 21:15:56.000000000","message":"This needs to be except exceptions.NotFound...","commit_id":"3cd1fb87b5b09d012cee775fff1b93524bd955db"}]}