)]}'
{"nova/tests/api/openstack/test_server_actions.py":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"aededbfcc6cce6814d80db93d17a06e8af2825c4","unresolved":false,"context_lines":[{"line_number":285,"context_line":""},{"line_number":286,"context_line":"        res \u003d req.get_response(fakes.wsgi_app())"},{"line_number":287,"context_line":"        self.assertEqual(res.status_int, 400)"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"    def test_revert_resize_server(self):"},{"line_number":290,"context_line":"        req \u003d self.webreq(\u0027/1/action\u0027, \u0027POST\u0027, dict(revertResize\u003dNone))"},{"line_number":291,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAACX%2F%2F9ME%3D","line":288,"updated":"2011-10-10 20:48:43.000000000","message":"Looks like this test would pass even with the old code, as the default exception catch also returned BadRequest.  Ie, it\u0027s essentially the same test as the one above it and not testing anything any further.  Maybe check the response for specific text?","commit_id":"261b4111d481562760321bfc83d64ba35e981b5b"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"4a136e81c372f0cc464c9dfe3da7360f4da0d117","unresolved":false,"context_lines":[{"line_number":285,"context_line":""},{"line_number":286,"context_line":"        res \u003d req.get_response(fakes.wsgi_app())"},{"line_number":287,"context_line":"        self.assertEqual(res.status_int, 400)"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"    def test_revert_resize_server(self):"},{"line_number":290,"context_line":"        req \u003d self.webreq(\u0027/1/action\u0027, \u0027POST\u0027, dict(revertResize\u003dNone))"},{"line_number":291,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAACX%2F%2F9K0%3D","line":288,"in_reply_to":"AAAACX%2F%2F9Lc%3D","updated":"2011-10-10 21:08:41.000000000","message":"Nah :)  Not going to worry about it.","commit_id":"261b4111d481562760321bfc83d64ba35e981b5b"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"1bd4053b9f5e5f5855bdfc204ee6d867b38aa14a","unresolved":false,"context_lines":[{"line_number":285,"context_line":""},{"line_number":286,"context_line":"        res \u003d req.get_response(fakes.wsgi_app())"},{"line_number":287,"context_line":"        self.assertEqual(res.status_int, 400)"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"    def test_revert_resize_server(self):"},{"line_number":290,"context_line":"        req \u003d self.webreq(\u0027/1/action\u0027, \u0027POST\u0027, dict(revertResize\u003dNone))"},{"line_number":291,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAACX%2F%2F9Lc%3D","line":288,"in_reply_to":"AAAACX%2F%2F9ME%3D","updated":"2011-10-10 20:57:53.000000000","message":"I would really prefer not to add a test that checks the text of an exception, however, I do see how this test would pass with the original code since we catch \u0027Exception\u0027. Would you prefer I remove it this test?","commit_id":"261b4111d481562760321bfc83d64ba35e981b5b"}]}