)]}'
{"aodhclient/tests/unit/test_alarm_cli.py":[{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"38f2d4b0dc3b535d59046b61f24eb5ea5e851fad","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    @mock.patch.object(argparse.ArgumentParser, \u0027error\u0027)"},{"line_number":50,"context_line":"    def test_validate_args_gnocchi_resources_threshold(self, mock_arg):"},{"line_number":51,"context_line":"        # Cover the test case of the method _validate_args for"},{"line_number":52,"context_line":"        # threshold"},{"line_number":53,"context_line":"        cli_alarm_create \u003d CliAlarmCreateForTest(self.app, self.parser)"},{"line_number":54,"context_line":"        test_parser \u003d argparse.ArgumentParser()"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_17a2944b","line":51,"updated":"2016-01-31 16:08:43.000000000","message":"probably, it would be better to change to \"gnocchi_resources_threshold\" because now two tests have the same descriptions","commit_id":"89183d2968885556711eb3dc3960910e53033823"},{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"03da5ec03542aac54a9c7149075d830df24f4591","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    @mock.patch.object(argparse.ArgumentParser, \u0027error\u0027)"},{"line_number":50,"context_line":"    def test_validate_args_gnocchi_resources_threshold(self, mock_arg):"},{"line_number":51,"context_line":"        # Cover the test case of the method _validate_args for"},{"line_number":52,"context_line":"        # threshold"},{"line_number":53,"context_line":"        cli_alarm_create \u003d CliAlarmCreateForTest(self.app, self.parser)"},{"line_number":54,"context_line":"        test_parser \u003d argparse.ArgumentParser()"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_3258f087","line":51,"in_reply_to":"7a5de9d1_17a2944b","updated":"2016-02-01 09:28:38.000000000","message":"Hi Nadya\n\nThanks for your comments. I will fix it.","commit_id":"89183d2968885556711eb3dc3960910e53033823"},{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"38f2d4b0dc3b535d59046b61f24eb5ea5e851fad","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        test_parser.add_argument("},{"line_number":56,"context_line":"            \u0027--type\u0027,"},{"line_number":57,"context_line":"            default\u003d\u0027gnocchi_resources_threshold\u0027)"},{"line_number":58,"context_line":"        test_parser.add_argument(\u0027--metric\u0027, default\u003d\u0027cpu\u0027)"},{"line_number":59,"context_line":"        test_parser.add_argument(\u0027--resource_id\u0027, )"},{"line_number":60,"context_line":"        test_parser.add_argument(\u0027--aggregation_method\u0027, default\u003d\u0027last\u0027)"},{"line_number":61,"context_line":"        test_parser.add_argument(\u0027--resource_type\u0027, default\u003d\u0027generic\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_3765501d","line":58,"updated":"2016-01-31 16:08:43.000000000","message":"nit: comma","commit_id":"89183d2968885556711eb3dc3960910e53033823"},{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"03da5ec03542aac54a9c7149075d830df24f4591","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        test_parser.add_argument("},{"line_number":56,"context_line":"            \u0027--type\u0027,"},{"line_number":57,"context_line":"            default\u003d\u0027gnocchi_resources_threshold\u0027)"},{"line_number":58,"context_line":"        test_parser.add_argument(\u0027--metric\u0027, default\u003d\u0027cpu\u0027)"},{"line_number":59,"context_line":"        test_parser.add_argument(\u0027--resource_id\u0027, )"},{"line_number":60,"context_line":"        test_parser.add_argument(\u0027--aggregation_method\u0027, default\u003d\u0027last\u0027)"},{"line_number":61,"context_line":"        test_parser.add_argument(\u0027--resource_type\u0027, default\u003d\u0027generic\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_b2caa019","line":58,"in_reply_to":"7a5de9d1_3765501d","updated":"2016-02-01 09:28:38.000000000","message":"Hi Nadya\n\nThanks for your comments. I will fix it.","commit_id":"89183d2968885556711eb3dc3960910e53033823"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"8b3d85c300d0530b38767c3a8aa033f205833adb","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import argparse"},{"line_number":17,"context_line":"import mock"},{"line_number":18,"context_line":"import testtools"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from aodhclient.v2 import alarm_cli"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a5de9d1_6c98c11c","line":17,"updated":"2016-02-01 13:28:47.000000000","message":"separate argparse in one group... mock and testtools in another group","commit_id":"65d43884ed7cafee6178997e54588d5870f42b3e"},{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"ae0bed294b9d38f464ce4655768e8a7e6ca43608","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import argparse"},{"line_number":17,"context_line":"import mock"},{"line_number":18,"context_line":"import testtools"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from aodhclient.v2 import alarm_cli"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a5de9d1_35b3ec0b","line":17,"in_reply_to":"7a5de9d1_6c98c11c","updated":"2016-02-01 14:23:15.000000000","message":"Hi Gordon\n\nThanks for your comments.","commit_id":"65d43884ed7cafee6178997e54588d5870f42b3e"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"8b3d85c300d0530b38767c3a8aa033f205833adb","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class CliAlarmListTest(testtools.TestCase):"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    def setUp(self):"},{"line_number":26,"context_line":"        super(CliAlarmListTest, self).setUp()"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    def test_get_columns(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a5de9d1_4cd385d5","line":25,"updated":"2016-02-01 13:28:47.000000000","message":"not necessary","commit_id":"65d43884ed7cafee6178997e54588d5870f42b3e"},{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"ae0bed294b9d38f464ce4655768e8a7e6ca43608","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class CliAlarmListTest(testtools.TestCase):"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    def setUp(self):"},{"line_number":26,"context_line":"        super(CliAlarmListTest, self).setUp()"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    def test_get_columns(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a5de9d1_b5a91c26","line":25,"in_reply_to":"7a5de9d1_4cd385d5","updated":"2016-02-01 14:23:15.000000000","message":"Hi Gordon\n\nThanks for your comments.","commit_id":"65d43884ed7cafee6178997e54588d5870f42b3e"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"8b3d85c300d0530b38767c3a8aa033f205833adb","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        self.assertIn(expected_value, result)"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"class CliAlarmCreateForTest(alarm_cli.CliAlarmCreate):"},{"line_number":36,"context_line":"    # Wrap the class CliALarmCreate for testing"},{"line_number":37,"context_line":"    def __init__(self, app, parser):"},{"line_number":38,"context_line":"        self.app \u003d app"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a5de9d1_ec611106","line":35,"updated":"2016-02-01 13:28:47.000000000","message":"never really used","commit_id":"65d43884ed7cafee6178997e54588d5870f42b3e"},{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"ae0bed294b9d38f464ce4655768e8a7e6ca43608","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        self.assertIn(expected_value, result)"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"class CliAlarmCreateForTest(alarm_cli.CliAlarmCreate):"},{"line_number":36,"context_line":"    # Wrap the class CliALarmCreate for testing"},{"line_number":37,"context_line":"    def __init__(self, app, parser):"},{"line_number":38,"context_line":"        self.app \u003d app"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a5de9d1_d5c29852","line":35,"in_reply_to":"7a5de9d1_ec611106","updated":"2016-02-01 14:23:15.000000000","message":"Hi Gordon\n\nThanks. I will use Mock.","commit_id":"65d43884ed7cafee6178997e54588d5870f42b3e"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"550875e1f7ec170535dcb797f0e0a9a93eaccabd","unresolved":false,"context_lines":[{"line_number":60,"context_line":"        test_parser.add_argument(\u0027--aggregation_method\u0027, default\u003d\u0027last\u0027)"},{"line_number":61,"context_line":"        test_parser.add_argument(\u0027--resource_type\u0027, default\u003d\u0027generic\u0027)"},{"line_number":62,"context_line":"        test_parser.add_argument(\u0027--threshold\u0027, default\u003d80)"},{"line_number":63,"context_line":"        test_parsed_args \u003d test_parser.parse_args()"},{"line_number":64,"context_line":"        cli_alarm_create._validate_args(test_parsed_args)"},{"line_number":65,"context_line":"        mock_arg.assert_called_with("},{"line_number":66,"context_line":"            \u0027gnocchi_resources_threshold requires --metric\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a5de9d1_a5e1dc2b","line":63,"updated":"2016-02-01 17:51:18.000000000","message":"i would recommend not doing this. there is no reason for you to define a new parser. this is not testing input. this is testing what would happen with a completely unrelated parser.\n\nyou should use get_parser and pass in actually arguments to parse.. then you are actually properly testnig input.","commit_id":"65d43884ed7cafee6178997e54588d5870f42b3e"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"014f9558f6e05da9034e04204196d4a8a492ab05","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    def setUp(self):"},{"line_number":36,"context_line":"        super(CliAlarmCreateTest, self).setUp()"},{"line_number":37,"context_line":"        self.app \u003d mock.Mock()"},{"line_number":38,"context_line":"        self.parser \u003d argparse.ArgumentParser()"},{"line_number":39,"context_line":"        self.cli_alarm_create \u003d \\"},{"line_number":40,"context_line":"            alarm_cli.CliAlarmCreate(self.app, self.parser)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"7a5de9d1_a7e3d932","line":38,"range":{"start_line":38,"start_character":22,"end_line":38,"end_character":47},"updated":"2016-02-02 21:53:07.000000000","message":"can\u0027t this be mocked as well?","commit_id":"2cce3f48f40951a1b8dfe96947bf5bf9c59e5cbc"},{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"52a48fc69ba3e4865e94ccc92b19b7d29db80225","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    def setUp(self):"},{"line_number":36,"context_line":"        super(CliAlarmCreateTest, self).setUp()"},{"line_number":37,"context_line":"        self.app \u003d mock.Mock()"},{"line_number":38,"context_line":"        self.parser \u003d argparse.ArgumentParser()"},{"line_number":39,"context_line":"        self.cli_alarm_create \u003d \\"},{"line_number":40,"context_line":"            alarm_cli.CliAlarmCreate(self.app, self.parser)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"5a5ae5dd_bfa3074f","line":38,"range":{"start_line":38,"start_character":22,"end_line":38,"end_character":47},"in_reply_to":"7a5de9d1_a7e3d932","updated":"2016-02-03 02:55:30.000000000","message":"Hi Gordon\n\nThanks for your review. I will try it. :)","commit_id":"2cce3f48f40951a1b8dfe96947bf5bf9c59e5cbc"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"014f9558f6e05da9034e04204196d4a8a492ab05","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        super(CliAlarmCreateTest, self).setUp()"},{"line_number":37,"context_line":"        self.app \u003d mock.Mock()"},{"line_number":38,"context_line":"        self.parser \u003d argparse.ArgumentParser()"},{"line_number":39,"context_line":"        self.cli_alarm_create \u003d \\"},{"line_number":40,"context_line":"            alarm_cli.CliAlarmCreate(self.app, self.parser)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @mock.patch.object(argparse.ArgumentParser, \u0027error\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a5de9d1_477305b5","line":39,"updated":"2016-02-02 21:53:07.000000000","message":"use parantheses to wrap lines not \\","commit_id":"2cce3f48f40951a1b8dfe96947bf5bf9c59e5cbc"},{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"52a48fc69ba3e4865e94ccc92b19b7d29db80225","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        super(CliAlarmCreateTest, self).setUp()"},{"line_number":37,"context_line":"        self.app \u003d mock.Mock()"},{"line_number":38,"context_line":"        self.parser \u003d argparse.ArgumentParser()"},{"line_number":39,"context_line":"        self.cli_alarm_create \u003d \\"},{"line_number":40,"context_line":"            alarm_cli.CliAlarmCreate(self.app, self.parser)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @mock.patch.object(argparse.ArgumentParser, \u0027error\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a5ae5dd_3f9077ec","line":39,"in_reply_to":"7a5de9d1_477305b5","updated":"2016-02-03 02:55:30.000000000","message":"Hi Gordon\n\nThanks for your review. I will do it.","commit_id":"2cce3f48f40951a1b8dfe96947bf5bf9c59e5cbc"}],"aodhclient/tests/unit/test_alarm_manager.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"0474d48c28df28a4d9f44172515a30b5150b65f4","unresolved":false,"context_lines":[{"line_number":11,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":12,"context_line":"#    under the License."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"from aodhclient.v2 import alarm"},{"line_number":15,"context_line":"import mock"},{"line_number":16,"context_line":"from oslo_serialization import jsonutils"},{"line_number":17,"context_line":"import testtools"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a5de9d1_816829bc","line":14,"updated":"2016-01-27 13:21:00.000000000","message":"aodhclient should be in it\u0027s own import grouping below third party libs","commit_id":"474616464ba288b5ef981c2fb142984e57ea03e9"},{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"a557703f350479ffe1df1bf550ed9b066bd0ed27","unresolved":false,"context_lines":[{"line_number":11,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":12,"context_line":"#    under the License."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"from aodhclient.v2 import alarm"},{"line_number":15,"context_line":"import mock"},{"line_number":16,"context_line":"from oslo_serialization import jsonutils"},{"line_number":17,"context_line":"import testtools"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a5de9d1_4c700173","line":14,"in_reply_to":"7a5de9d1_816829bc","updated":"2016-01-28 07:29:11.000000000","message":"Hi Gordon\n\nThanks for your review.\n\nI change it.","commit_id":"474616464ba288b5ef981c2fb142984e57ea03e9"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"0474d48c28df28a4d9f44172515a30b5150b65f4","unresolved":false,"context_lines":[{"line_number":74,"context_line":"        am \u003d alarm.AlarmManager(self.client)"},{"line_number":75,"context_line":"        am.search()"},{"line_number":76,"context_line":"        url \u003d \u0027v2/query/alarms\u0027"},{"line_number":77,"context_line":"        query_value \u003d jsonutils.dumps({})"},{"line_number":78,"context_line":"        headers_value \u003d {\u0027Content-Type\u0027: \"application/json\"}"},{"line_number":79,"context_line":"        mock_am.assert_called_with("},{"line_number":80,"context_line":"            url,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a5de9d1_e15b1d2d","line":77,"updated":"2016-01-27 13:21:00.000000000","message":"isn\u0027t this just \u0027{}\u0027?","commit_id":"474616464ba288b5ef981c2fb142984e57ea03e9"},{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"a557703f350479ffe1df1bf550ed9b066bd0ed27","unresolved":false,"context_lines":[{"line_number":74,"context_line":"        am \u003d alarm.AlarmManager(self.client)"},{"line_number":75,"context_line":"        am.search()"},{"line_number":76,"context_line":"        url \u003d \u0027v2/query/alarms\u0027"},{"line_number":77,"context_line":"        query_value \u003d jsonutils.dumps({})"},{"line_number":78,"context_line":"        headers_value \u003d {\u0027Content-Type\u0027: \"application/json\"}"},{"line_number":79,"context_line":"        mock_am.assert_called_with("},{"line_number":80,"context_line":"            url,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a5de9d1_8c46f921","line":77,"in_reply_to":"7a5de9d1_e15b1d2d","updated":"2016-01-28 07:29:11.000000000","message":"Hi Gordon\n\nThanks for your review.\n\nYes, if query\u003d\u003dnone, data should be {}.\n\n(Pdb) self._post(url, headers\u003d{\u0027Content-Type\u0027: \"application/json\"}, data\u003djsonutils.dumps({})).json()\n[{u\u0027alarm_actions\u0027: [], u\u0027project_id\u0027: u\u00278173087c18c64d4fa5ac58d007d80e0f\u0027, u\u0027ok_actions\u0027: [], u\u0027name\u0027: u\u0027alarm_event\u0027, u\u0027timestamp\u0027: u\u00272016-01-28T03:41:42.764807\u0027, u\u0027description\u0027: u\u0027Alarm when * event occurred.\u0027, u\u0027time_constraints\u0027: [], u\u0027enabled\u0027: True, u\u0027state_timestamp\u0027: u\u00272016-01-28T03:41:42.764807\u0027, u\u0027alarm_id\u0027: u\u0027373d6671-150c-4737-bdb8-f95a2d13dcf3\u0027, u\u0027state\u0027: u\u0027insufficient data\u0027, u\u0027insufficient_data_actions\u0027: [], u\u0027repeat_actions\u0027: False, u\u0027user_id\u0027: u\u0027a1b4e0796581456ead710af4f751bdba\u0027, u\u0027event_rule\u0027: {u\u0027query\u0027: [], u\u0027event_type\u0027: u\u0027*\u0027}, u\u0027type\u0027: u\u0027event\u0027, u\u0027severity\u0027: u\u0027low\u0027}]\n\nBut based on our discussion on IRC, the event type is must. So the query should contain the one field \u0027type\u0027 at least. I will change the corresponding test case.\n\nThanks","commit_id":"474616464ba288b5ef981c2fb142984e57ea03e9"},{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"38f2d4b0dc3b535d59046b61f24eb5ea5e851fad","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        }"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    @mock.patch.object(alarm.AlarmManager, \u0027_get\u0027)"},{"line_number":44,"context_line":"    def test_list(self, mock_am):"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"        am \u003d alarm.AlarmManager(self.client)"},{"line_number":47,"context_line":"        am.list(\u0027event\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_57380cef","line":44,"updated":"2016-01-31 16:08:43.000000000","message":"The following line is not needed :)","commit_id":"89183d2968885556711eb3dc3960910e53033823"},{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"03da5ec03542aac54a9c7149075d830df24f4591","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        }"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    @mock.patch.object(alarm.AlarmManager, \u0027_get\u0027)"},{"line_number":44,"context_line":"    def test_list(self, mock_am):"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"        am \u003d alarm.AlarmManager(self.client)"},{"line_number":47,"context_line":"        am.list(\u0027event\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_92ea0473","line":44,"in_reply_to":"7a5de9d1_57380cef","updated":"2016-02-01 09:28:38.000000000","message":"Hi Nadya\n\nThanks for your comments.","commit_id":"89183d2968885556711eb3dc3960910e53033823"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"014f9558f6e05da9034e04204196d4a8a492ab05","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        query \u003d \u0027{\"\u003d\": {\"type\": \"event\"}}\u0027"},{"line_number":77,"context_line":"        am.search(query)"},{"line_number":78,"context_line":"        url \u003d \u0027v2/query/alarms\u0027"},{"line_number":79,"context_line":"        expected_value \u003d \u0027{\"filter\": \"{\\\\\"\u003d\\\\\": {\\\\\"type\\\\\":\u0027 + \\"},{"line_number":80,"context_line":"                         \u0027 \\\\\"event\\\\\"}}\"}\u0027"},{"line_number":81,"context_line":"        headers_value \u003d {\u0027Content-Type\u0027: \"application/json\"}"},{"line_number":82,"context_line":"        mock_am.assert_called_with("}],"source_content_type":"text/x-python","patch_set":5,"id":"7a5de9d1_472b0534","line":79,"updated":"2016-02-02 21:53:07.000000000","message":"use parantheses","commit_id":"2cce3f48f40951a1b8dfe96947bf5bf9c59e5cbc"},{"author":{"_account_id":9526,"name":"Zi Lian Ji","email":"jizilian@cn.ibm.com","username":"jizilian"},"change_message_id":"52a48fc69ba3e4865e94ccc92b19b7d29db80225","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        query \u003d \u0027{\"\u003d\": {\"type\": \"event\"}}\u0027"},{"line_number":77,"context_line":"        am.search(query)"},{"line_number":78,"context_line":"        url \u003d \u0027v2/query/alarms\u0027"},{"line_number":79,"context_line":"        expected_value \u003d \u0027{\"filter\": \"{\\\\\"\u003d\\\\\": {\\\\\"type\\\\\":\u0027 + \\"},{"line_number":80,"context_line":"                         \u0027 \\\\\"event\\\\\"}}\"}\u0027"},{"line_number":81,"context_line":"        headers_value \u003d {\u0027Content-Type\u0027: \"application/json\"}"},{"line_number":82,"context_line":"        mock_am.assert_called_with("}],"source_content_type":"text/x-python","patch_set":5,"id":"5a5ae5dd_bfba276d","line":79,"in_reply_to":"7a5de9d1_472b0534","updated":"2016-02-03 02:55:30.000000000","message":"Hi Gordon\n\nThanks for your review. I will do it.","commit_id":"2cce3f48f40951a1b8dfe96947bf5bf9c59e5cbc"}]}