)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"6a9b2b5cca152da1cb9ea4efa52479f1684581fc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7dc05ca0_6a6f2b0e","updated":"2022-09-26 15:20:36.000000000","message":"Thanks for the patch! One nit inline.","commit_id":"e2032435ba9b81cb2fb02096cd2d475677ef6de7"}],"ovsdbapp/tests/unit/backend/ovs_idl/test_connection.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"6a9b2b5cca152da1cb9ea4efa52479f1684581fc","unresolved":true,"context_lines":[{"line_number":41,"context_line":"        mock_wait_for_change.assert_any_call(self.conn.idl, self.conn.timeout)"},{"line_number":42,"context_line":"        mock_poller.assert_called_once_with()"},{"line_number":43,"context_line":"        mock_thread.assert_called_once_with(target\u003dself.conn.run)"},{"line_number":44,"context_line":"        self.assertIs(True, mock_thread.return_value.daemon)"},{"line_number":45,"context_line":"        mock_thread.return_value.start.assert_called_once_with()"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def test_queue_txn(self, mock_thread):"}],"source_content_type":"text/x-python","patch_set":1,"id":"50995c7e_87a24e01","line":44,"range":{"start_line":44,"start_character":13,"end_line":44,"end_character":21},"updated":"2022-09-26 15:20:36.000000000","message":"Would assertTrue be better here?","commit_id":"e2032435ba9b81cb2fb02096cd2d475677ef6de7"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"e496c7766ec26156236a9a06a566cebe61e6386e","unresolved":true,"context_lines":[{"line_number":41,"context_line":"        mock_wait_for_change.assert_any_call(self.conn.idl, self.conn.timeout)"},{"line_number":42,"context_line":"        mock_poller.assert_called_once_with()"},{"line_number":43,"context_line":"        mock_thread.assert_called_once_with(target\u003dself.conn.run)"},{"line_number":44,"context_line":"        self.assertIs(True, mock_thread.return_value.daemon)"},{"line_number":45,"context_line":"        mock_thread.return_value.start.assert_called_once_with()"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def test_queue_txn(self, mock_thread):"}],"source_content_type":"text/x-python","patch_set":1,"id":"8ab0150d_5a7280fe","line":44,"range":{"start_line":44,"start_character":13,"end_line":44,"end_character":21},"in_reply_to":"50995c7e_87a24e01","updated":"2022-10-15 08:07:46.000000000","message":"assertTrue checks whether bool(mock_thread.return_value.daemon) is True or not.\nIf mock_thread.return_value.daemon is 1, assertTrue is success.\nSo assertIs is better.","commit_id":"e2032435ba9b81cb2fb02096cd2d475677ef6de7"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"7f35d01033a72c01aea22eea37daaf0e7a14895a","unresolved":true,"context_lines":[{"line_number":41,"context_line":"        mock_wait_for_change.assert_any_call(self.conn.idl, self.conn.timeout)"},{"line_number":42,"context_line":"        mock_poller.assert_called_once_with()"},{"line_number":43,"context_line":"        mock_thread.assert_called_once_with(target\u003dself.conn.run)"},{"line_number":44,"context_line":"        self.assertIs(True, mock_thread.return_value.daemon)"},{"line_number":45,"context_line":"        mock_thread.return_value.start.assert_called_once_with()"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def test_queue_txn(self, mock_thread):"}],"source_content_type":"text/x-python","patch_set":1,"id":"0e90325f_ddf798c7","line":44,"range":{"start_line":44,"start_character":13,"end_line":44,"end_character":21},"in_reply_to":"8ab0150d_5a7280fe","updated":"2022-10-15 15:19:42.000000000","message":"Got it, thanks for the explanation.","commit_id":"e2032435ba9b81cb2fb02096cd2d475677ef6de7"}]}