)]}'
{"test/unit/account/test_backend.py":[{"author":{"_account_id":25720,"name":"Ngo Quoc Cuong","email":"cuongngo.ict@gmail.com","username":"cuongnq"},"change_message_id":"80409dc0b003bf0b1db64c85e86af0092709e306","unresolved":false,"context_lines":[{"line_number":87,"context_line":"                raise Exception(\u0027OMG\u0027)"},{"line_number":88,"context_line":"        except Exception:"},{"line_number":89,"context_line":"            pass"},{"line_number":90,"context_line":"        self.assertIsNone(broker.conn)"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"    def test_empty(self):"},{"line_number":93,"context_line":"        # Test AccountBroker.empty"}],"source_content_type":"text/x-python","patch_set":8,"id":"1f1a1f67_f21634a8","line":90,"range":{"start_line":90,"start_character":13,"end_line":90,"end_character":25},"updated":"2017-07-19 03:47:08.000000000","message":"For assertIsNone, I have one suggestion for you, that is enable H203\n\nhttps://github.com/openstack/swift/blob/master/tox.ini#L102\n\nSee:\nhttps://review.openstack.org/#/c/481698/3/tox.ini","commit_id":"3bdcfef8bc0733c779cbd94d1f96c45a5bbf58da"}],"test/unit/account/test_server.py":[{"author":{"_account_id":14766,"name":"Kazuhiro MIYAHARA","email":"miyahara.kazuhiro@lab.ntt.co.jp","username":"miyahara"},"change_message_id":"234539c531a2871db78710c470a6ed9e9c53242d","unresolved":false,"context_lines":[{"line_number":1644,"context_line":"            self.assertFalse(hasattr(method, \u0027replication\u0027))"},{"line_number":1645,"context_line":"        for method_name in repl_methods:"},{"line_number":1646,"context_line":"            method \u003d getattr(self.controller, method_name)"},{"line_number":1647,"context_line":"            self.assertTrue(method.replication)"},{"line_number":1648,"context_line":""},{"line_number":1649,"context_line":"    def test_correct_allowed_method(self):"},{"line_number":1650,"context_line":"        # Test correct work for allowed method using"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_dbda4f78","line":1647,"range":{"start_line":1647,"start_character":12,"end_line":1647,"end_character":47},"updated":"2016-09-05 05:28:45.000000000","message":"I don\u0027t think this change is reasonable, because I think \u0027self.assertEqual(foo, True)\u0027 is stricter than \u0027self.assertTrue(foo)\u0027. \u0027self.assertTrue(foo)\u0027 is equivalent to \u0027self.assertEqual(bool(foo), True)\u0027. It means that result of \u0027self.assertEqual(\"sample\", True) is failure and result of \u0027self.assertTrue(\"sample\") is OK.","commit_id":"2a475766480dab4427decb32ac836cc8f9fb113d"},{"author":{"_account_id":18603,"name":"Béla Vancsics","email":"vancsics@inf.u-szeged.hu","username":"vancsics"},"change_message_id":"f991f7b7360e1ffcf083fcabc933646128d9e89d","unresolved":false,"context_lines":[{"line_number":1729,"context_line":"            self.assertFalse(hasattr(method, \u0027replication\u0027))"},{"line_number":1730,"context_line":"        for method_name in repl_methods:"},{"line_number":1731,"context_line":"            method \u003d getattr(self.controller, method_name)"},{"line_number":1732,"context_line":"            self.assertEqual(method.replication, True)"},{"line_number":1733,"context_line":""},{"line_number":1734,"context_line":"    def test_correct_allowed_method(self):"},{"line_number":1735,"context_line":"        # Test correct work for allowed method using"}],"source_content_type":"text/x-python","patch_set":7,"id":"5f201791_80608534","line":1732,"range":{"start_line":1732,"start_character":12,"end_line":1732,"end_character":54},"updated":"2017-06-21 08:44:35.000000000","message":"self.assertTrue(method.replication)","commit_id":"d467a4a44c67e4a0d966a9ffa69566aba7c97b1f"}]}