)]}'
{"tests/api/v2/test_alarm_scenarios.py":[{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"f551776078981d3a0396e7db51d8039e6d6634d6","unresolved":false,"context_lines":[{"line_number":568,"context_line":""},{"line_number":569,"context_line":"        resp \u003d self.delete(\u0027/alarms/%s\u0027 % data[0][\u0027alarm_id\u0027],"},{"line_number":570,"context_line":"                           headers\u003dself.auth_headers,"},{"line_number":571,"context_line":"                           expect_errors\u003dTrue)"},{"line_number":572,"context_line":"        self.assertEqual(resp.status_code, 204)"},{"line_number":573,"context_line":"        self.assertEqual(resp.body, \u0027\u0027)"},{"line_number":574,"context_line":"        alarms \u003d list(self.conn.get_alarms())"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2Flrk%3D","line":571,"updated":"2013-09-24 09:29:44.000000000","message":"I\u0027m confused by the expect_errors\u003dTrue here, as it looks like the delete should succeed.\n\nAlso the previous form already asserted the status code is 204 (though not of course that the response entity body is empty).\n\nSeems wrong to imply that this API call fails, or?","commit_id":"9ff9fe2aaa74a36266086a78517ff7d39a8eba7c"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"e9b419194efe20acbe56996c60e9dbee49854bac","unresolved":false,"context_lines":[{"line_number":568,"context_line":""},{"line_number":569,"context_line":"        resp \u003d self.delete(\u0027/alarms/%s\u0027 % data[0][\u0027alarm_id\u0027],"},{"line_number":570,"context_line":"                           headers\u003dself.auth_headers,"},{"line_number":571,"context_line":"                           expect_errors\u003dTrue)"},{"line_number":572,"context_line":"        self.assertEqual(resp.status_code, 204)"},{"line_number":573,"context_line":"        self.assertEqual(resp.body, \u0027\u0027)"},{"line_number":574,"context_line":"        alarms \u003d list(self.conn.get_alarms())"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2Flfg%3D","line":571,"in_reply_to":"AAAAS3%2F%2Flrk%3D","updated":"2013-09-24 10:30:47.000000000","message":"You have right, I have thinked that the delete method return only the response.json like the get method but it return the full webtest response. So I can keep the previous behavior and just check the body.","commit_id":"9ff9fe2aaa74a36266086a78517ff7d39a8eba7c"}]}