)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"23318a189e1f29d3295ecab14591b15113b58450","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ronald Bradford \u003cronald.bradford@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-03 22:38:57 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Corrected transposed AssertEqual(expected, actual)"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I9c96477ec36869e15d18c5ee26075f2aadd776a5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_5483a4cc","line":7,"updated":"2016-03-07 13:35:55.000000000","message":"Need a reference to justify this change. Is there a mention of that in the OpenStack coding style for example?","commit_id":"603b8c4983fff74e7ef547dbe12652cc63ef9de5"},{"author":{"_account_id":16051,"name":"Ronald Bradford","email":"ronald.bradford@gmail.com","username":"ronaldbradford"},"change_message_id":"d30e50b8f71d7df22b5a65582018112868ff78d1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ronald Bradford \u003cronald.bradford@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-03 22:38:57 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Corrected transposed AssertEqual(expected, actual)"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I9c96477ec36869e15d18c5ee26075f2aadd776a5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_c0951741","line":7,"in_reply_to":"5aef4532_5483a4cc","updated":"2016-03-07 20:03:31.000000000","message":"See http://lists.openstack.org/pipermail/openstack-dev/2015-September/075320.html","commit_id":"603b8c4983fff74e7ef547dbe12652cc63ef9de5"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"662f251a83043500c9e57024c6b2f4f79b749274","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ronald Bradford \u003cronald.bradford@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-03 22:38:57 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Corrected transposed AssertEqual(expected, actual)"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I9c96477ec36869e15d18c5ee26075f2aadd776a5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_8fa5f805","line":7,"in_reply_to":"5aef4532_c0951741","updated":"2016-03-07 21:40:43.000000000","message":"Its also documented with testtools but I don\u0027t think a reference is absolutely necessary.\n\nhttps://testtools.readthedocs.org/en/latest/api.html#testtools.TestCase.assertEqual","commit_id":"603b8c4983fff74e7ef547dbe12652cc63ef9de5"}],"oslo_log/tests/unit/test_log.py":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"1563f430ea12f496189eef35ba1af638159cb865","unresolved":false,"context_lines":[{"line_number":88,"context_line":"    def test_handles_context_kwarg(self):"},{"line_number":89,"context_line":"        self.log.info(\"foo\", context\u003d_fake_context())"},{"line_number":90,"context_line":"        self.assertTrue(True)  # didn\u0027t raise exception"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"    def test_will_be_verbose_if_verbose_flag_set(self):"},{"line_number":93,"context_line":"        self.config(verbose\u003dTrue)"},{"line_number":94,"context_line":"        logger_name \u003d \u0027test_is_verbose\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"3afc51ec_833fae24","line":91,"updated":"2016-03-11 17:46:45.000000000","message":"Why did you remove test_handles_context_kwarg_as_dict() and test_handles_context_kwarg_as_none()?","commit_id":"d13011e28dc7834c1de48903a36afe4265806e58"},{"author":{"_account_id":16051,"name":"Ronald Bradford","email":"ronald.bradford@gmail.com","username":"ronaldbradford"},"change_message_id":"5ccb8947d8c5e46228dc72716d45ea57ea814fef","unresolved":false,"context_lines":[{"line_number":88,"context_line":"    def test_handles_context_kwarg(self):"},{"line_number":89,"context_line":"        self.log.info(\"foo\", context\u003d_fake_context())"},{"line_number":90,"context_line":"        self.assertTrue(True)  # didn\u0027t raise exception"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"    def test_will_be_verbose_if_verbose_flag_set(self):"},{"line_number":93,"context_line":"        self.config(verbose\u003dTrue)"},{"line_number":94,"context_line":"        logger_name \u003d \u0027test_is_verbose\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"3afc51ec_0318be83","line":91,"in_reply_to":"3afc51ec_833fae24","updated":"2016-03-11 17:55:20.000000000","message":"See negative feedback of this separate review for adding these tests. I have abandoned this review until I can come up with more applicable tests around self.assertTrue(True). \n\nhttps://review.openstack.org/#/c/288755/3/oslo_log/tests/unit/test_log.py","commit_id":"d13011e28dc7834c1de48903a36afe4265806e58"},{"author":{"_account_id":9459,"name":".Haifeng Yan","email":"yanheven@qq.com","username":"yan"},"change_message_id":"8482d83c60bbe609ec60b4f2cd4fbdec2849cbe0","unresolved":false,"context_lines":[{"line_number":605,"context_line":""},{"line_number":606,"context_line":"        self.colorlog.info(\"foo\")"},{"line_number":607,"context_line":"        self.assertNotEqual(sys.stderr.getvalue().find(\"KeyError: \u0027missing\u0027\"),"},{"line_number":608,"context_line":"                            -1)"},{"line_number":609,"context_line":""},{"line_number":610,"context_line":"        sys.stderr \u003d error"},{"line_number":611,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"dada55a8_c5ef3be7","line":608,"range":{"start_line":608,"start_character":30,"end_line":608,"end_character":31},"updated":"2016-07-21 02:31:53.000000000","message":"this one","commit_id":"d13011e28dc7834c1de48903a36afe4265806e58"}]}