)]}'
{"test/unit/common/test_db_replicator.py":[{"author":{"_account_id":1531,"name":"Ilya Kharin","email":"akscram@gmail.com","username":"akscram"},"change_message_id":"10cbf4905b9c84f476cf70765cb2fb8ee25048dc","unresolved":false,"context_lines":[{"line_number":671,"context_line":"    def test_repl_to_node_usync_success(self):"},{"line_number":672,"context_line":"        rinfo \u003d {\"id\": 3, \"point\": -1, \"max_row\": 5, \"hash\": \"c\"}"},{"line_number":673,"context_line":"        http \u003d ReplHttp(simplejson.dumps(rinfo))"},{"line_number":674,"context_line":"        broker \u003d FakeBroker()"},{"line_number":675,"context_line":"        local_sync \u003d broker.get_sync()"},{"line_number":676,"context_line":"        self.replicator._http_connect \u003d lambda *args: http"},{"line_number":677,"context_line":"        self.assertEquals(self.replicator._repl_to_node("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPH%2F%2F0QU%3D","line":674,"updated":"2013-05-23 08:37:28.000000000","message":"You create instance of FakeBroker class in an each test of this class, so, it would be great to move it in the setUp method.","commit_id":"2e2c8727ef8a3fd0ebadac8c251b290af28f3ef7"},{"author":{"_account_id":1531,"name":"Ilya Kharin","email":"akscram@gmail.com","username":"akscram"},"change_message_id":"10cbf4905b9c84f476cf70765cb2fb8ee25048dc","unresolved":false,"context_lines":[{"line_number":788,"context_line":"            self.assertEquals("},{"line_number":789,"context_line":"                None,"},{"line_number":790,"context_line":"                self.replicator._replicate_object(\u00270\u0027, \u0027file\u0027, \u0027node_id\u0027))"},{"line_number":791,"context_line":""},{"line_number":792,"context_line":""},{"line_number":793,"context_line":""},{"line_number":794,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPH%2F%2F0QM%3D","line":791,"updated":"2013-05-23 08:37:28.000000000","message":"Unnecessary blank lines.","commit_id":"2e2c8727ef8a3fd0ebadac8c251b290af28f3ef7"},{"author":{"_account_id":1531,"name":"Ilya Kharin","email":"akscram@gmail.com","username":"akscram"},"change_message_id":"10cbf4905b9c84f476cf70765cb2fb8ee25048dc","unresolved":false,"context_lines":[{"line_number":795,"context_line":"    def test_replicate_primary_success(self):"},{"line_number":796,"context_line":"        self.replicator._repl_to_node \u003d mock.Mock(return_value\u003dTrue)"},{"line_number":797,"context_line":"        self.replicator._replicate_object(\u00270\u0027, \u0027/path/to/file\u0027, 1)"},{"line_number":798,"context_line":"        nodes \u003d \\"},{"line_number":799,"context_line":"            [call[0][0] for call in \\"},{"line_number":800,"context_line":"                self.replicator._repl_to_node.call_args_list]"},{"line_number":801,"context_line":"        self.assertEquals(nodes, db_replicator.ring.Ring.devs[1:3])"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPH%2F%2Fx2Q%3D","line":798,"updated":"2013-05-23 08:37:28.000000000","message":"You can replace definition with backslash on a next form:\n\n    nodes \u003d [call[0][0]\n             for call in self.replicator._repl_to_node.call_args_list]","commit_id":"2e2c8727ef8a3fd0ebadac8c251b290af28f3ef7"},{"author":{"_account_id":1531,"name":"Ilya Kharin","email":"akscram@gmail.com","username":"akscram"},"change_message_id":"10cbf4905b9c84f476cf70765cb2fb8ee25048dc","unresolved":false,"context_lines":[{"line_number":800,"context_line":"                self.replicator._repl_to_node.call_args_list]"},{"line_number":801,"context_line":"        self.assertEquals(nodes, db_replicator.ring.Ring.devs[1:3])"},{"line_number":802,"context_line":"        self.assertEquals(self.replicator.delete_db.call_count, 0)"},{"line_number":803,"context_line":"        self.replicator.logger.increment.assert_has_calls("},{"line_number":804,"context_line":"            [mock.call.increment(\u0027attempts\u0027),"},{"line_number":805,"context_line":"             mock.call.increment(\u0027successes\u0027),"},{"line_number":806,"context_line":"             mock.call.increment(\u0027successes\u0027)])"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPH%2F%2Fx18%3D","line":803,"updated":"2013-05-23 08:37:28.000000000","message":"Looks good:\n\n    self.replicator.logger.increment.assert_has_calls([\n        mock.call.increment(\u0027attempts\u0027),\n        mock.call.increment(\u0027successes\u0027),\n        mock.call.increment(\u0027successes\u0027),\n    ])","commit_id":"2e2c8727ef8a3fd0ebadac8c251b290af28f3ef7"}]}