)]}'
{"ceilometer/tests/api/v2/test_alarm_scenarios.py":[{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"191422933dee41d563cf16b44a2953ed2600b4c6","unresolved":false,"context_lines":[{"line_number":906,"context_line":""},{"line_number":907,"context_line":"        resp \u003d self.put_json(\u0027/alarms/%s\u0027 % alarm_id,"},{"line_number":908,"context_line":"                             params\u003djson,"},{"line_number":909,"context_line":"                             headers\u003dself.auth_headers)"},{"line_number":910,"context_line":"        self.assertEqual(resp.status_code, 200)"},{"line_number":911,"context_line":""},{"line_number":912,"context_line":"    def test_delete_alarm(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2BTL4%3D","line":909,"updated":"2014-01-28 08:39:45.000000000","message":"Optional: If we remove expect_errors\u003dTrue, the status can be check by put_json itself, with the argument state\u003d200","commit_id":"d14a58871a109cf6d2c7bc8f7037f24df8550aab"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"1ee29b30f25272f26046d2e5e242ac137afebe25","unresolved":false,"context_lines":[{"line_number":906,"context_line":""},{"line_number":907,"context_line":"        resp \u003d self.put_json(\u0027/alarms/%s\u0027 % alarm_id,"},{"line_number":908,"context_line":"                             params\u003djson,"},{"line_number":909,"context_line":"                             headers\u003dself.auth_headers)"},{"line_number":910,"context_line":"        self.assertEqual(resp.status_code, 200)"},{"line_number":911,"context_line":""},{"line_number":912,"context_line":"    def test_delete_alarm(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2BTEY%3D","line":909,"in_reply_to":"AAAAUH%2F%2BTL4%3D","updated":"2014-01-28 09:09:53.000000000","message":"Thanks, the hint is really helpful.\n\nSince I\u0027m a bit familiar with the testtools, so I keep the assertEqual when upload this patch. I will try the new style if another patchset is needed. Actually, a test for another patch showes that the failure message is as good as assertEqual\n\n```\nAppError: Bad response: 400 Bad Request (not 200 OK or 3xx redirect for http://localhost/v2/alarms/affc7386-7851-4971-b076-a2f99847ee0d)\n```\n\nThank you very much.","commit_id":"d14a58871a109cf6d2c7bc8f7037f24df8550aab"}]}