)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"ab30cc19806a4e02e97c3564713dccd89fa8eeac","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Tao Li \u003clitao3721@126.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-09-17 11:12:56 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace assertEqual(True/Fase, expr) with assertTrue/assertFalse in tesa"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In some cases, If the result of expr is a boolen value, we shoud use"},{"line_number":10,"context_line":"assertTrue/assertFalse to instead. Beacause it is clear and simple."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_3ef5c951","line":7,"range":{"start_line":7,"start_character":25,"end_line":7,"end_character":29},"updated":"2018-09-17 07:16:41.000000000","message":"nit: False","commit_id":"550488c2d282c95c699847802bcb86138aa1dbc1"},{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"ab30cc19806a4e02e97c3564713dccd89fa8eeac","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Tao Li \u003clitao3721@126.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-09-17 11:12:56 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace assertEqual(True/Fase, expr) with assertTrue/assertFalse in tesa"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In some cases, If the result of expr is a boolen value, we shoud use"},{"line_number":10,"context_line":"assertTrue/assertFalse to instead. Beacause it is clear and simple."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_de9375ff","line":7,"range":{"start_line":7,"start_character":68,"end_line":7,"end_character":72},"updated":"2018-09-17 07:16:41.000000000","message":"nit: tests","commit_id":"550488c2d282c95c699847802bcb86138aa1dbc1"}],"ironic_python_agent/tests/unit/test_hardware.py":[{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"ab30cc19806a4e02e97c3564713dccd89fa8eeac","unresolved":false,"context_lines":[{"line_number":451,"context_line":"                (2, \u002702626172\u0027)],"},{"line_number":452,"context_line":"        }"},{"line_number":453,"context_line":"        result \u003d self.hardware.collect_lldp_data(if_names)"},{"line_number":454,"context_line":"        self.assertTrue(if_names[0] in result)"},{"line_number":455,"context_line":"        self.assertEqual(expected_lldp_data, result)"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":"    @mock.patch(\u0027ironic_python_agent.netutils.get_lldp_info\u0027, autospec\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_bee9d97f","line":454,"range":{"start_line":454,"start_character":8,"end_line":454,"end_character":46},"updated":"2018-09-17 07:16:41.000000000","message":"Actually I think it\u0027s better to use assertIn here, ditto below.","commit_id":"550488c2d282c95c699847802bcb86138aa1dbc1"},{"author":{"_account_id":9610,"name":"Tao Li","email":"litao3721@126.com","username":"Litao"},"change_message_id":"4293c53bf06cd44b6ceb5e833c4308db1e0092c3","unresolved":false,"context_lines":[{"line_number":451,"context_line":"                (2, \u002702626172\u0027)],"},{"line_number":452,"context_line":"        }"},{"line_number":453,"context_line":"        result \u003d self.hardware.collect_lldp_data(if_names)"},{"line_number":454,"context_line":"        self.assertTrue(if_names[0] in result)"},{"line_number":455,"context_line":"        self.assertEqual(expected_lldp_data, result)"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":"    @mock.patch(\u0027ironic_python_agent.netutils.get_lldp_info\u0027, autospec\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_28933750","line":454,"range":{"start_line":454,"start_character":8,"end_line":454,"end_character":46},"in_reply_to":"3f79a3b5_bee9d97f","updated":"2018-09-18 07:18:41.000000000","message":"Yes，I will try.","commit_id":"550488c2d282c95c699847802bcb86138aa1dbc1"}]}