From e9ae8c212150203ab9c83c03fca2fcee6a3b3995 Mon Sep 17 00:00:00 2001 From: Balaji Arun Date: Tue, 22 Aug 2023 14:36:43 -0700 Subject: [PATCH] [dag] e2e integration test (#9457) --- consensus/src/dag/tests/integration_tests.rs | 236 +++++++++++++++++++ consensus/src/dag/tests/mod.rs | 1 + 2 files changed, 237 insertions(+) create mode 100644 consensus/src/dag/tests/integration_tests.rs diff --git a/consensus/src/dag/tests/integration_tests.rs b/consensus/src/dag/tests/integration_tests.rs new file mode 100644 index 0000000000000..9fab57de6026d --- /dev/null +++ b/consensus/src/dag/tests/integration_tests.rs @@ -0,0 +1,236 @@ +// Copyright © Aptos Foundation + +use super::dag_test; +use crate::{ + dag::{bootstrap::bootstrap_dag, CertifiedNode}, + network::{DAGNetworkSenderImpl, IncomingDAGRequest, NetworkSender}, + network_interface::{ConsensusMsg, ConsensusNetworkClient, DIRECT_SEND, RPC}, + network_tests::{NetworkPlayground, TwinId}, + test_utils::{consensus_runtime, MockPayloadManager, MockStorage}, +}; +use aptos_channels::{aptos_channel, message_queues::QueueStyle}; +use aptos_config::network_id::{NetworkId, PeerNetworkId}; +use aptos_consensus_types::common::Author; +use aptos_logger::debug; +use aptos_network::{ + application::interface::NetworkClient, + peer_manager::{conn_notifs_channel, ConnectionRequestSender, PeerManagerRequestSender}, + protocols::{ + network::{self, Event, NetworkEvents, NewNetworkEvents, NewNetworkSender}, + wire::handshake::v1::ProtocolIdSet, + }, + transport::ConnectionMetadata, + ProtocolId, +}; +use aptos_time_service::TimeService; +use aptos_types::{ + epoch_state::EpochState, + validator_signer::ValidatorSigner, + validator_verifier::{random_validator_verifier, ValidatorVerifier}, +}; +use claims::assert_gt; +use futures::{ + stream::{select, AbortHandle, Select}, + StreamExt, +}; +use futures_channel::mpsc::UnboundedReceiver; +use maplit::hashmap; +use std::sync::Arc; + +struct DagBootstrapUnit { + nh_abort_handle: AbortHandle, + df_abort_handle: AbortHandle, + dag_rpc_tx: aptos_channel::Sender, + network_events: + Box, aptos_channels::Receiver>>>, +} + +impl DagBootstrapUnit { + fn make( + self_peer: Author, + epoch: u64, + signer: ValidatorSigner, + storage: Arc, + network: NetworkSender, + time_service: TimeService, + network_events: Box< + Select, aptos_channels::Receiver>>, + >, + ) -> (Self, UnboundedReceiver>>) { + let epoch_state = EpochState { + epoch, + verifier: storage.get_validator_set().into(), + }; + let dag_storage = dag_test::MockStorage::new(); + + let network = Arc::new(DAGNetworkSenderImpl::new(Arc::new(network))); + + let payload_client = Arc::new(MockPayloadManager::new(None)); + + let (nh_abort_handle, df_abort_handle, dag_rpc_tx, ordered_nodes_rx) = bootstrap_dag( + self_peer, + signer, + Arc::new(epoch_state), + storage.get_ledger_info(), + Arc::new(dag_storage), + network.clone(), + network.clone(), + time_service, + payload_client, + ); + + ( + Self { + nh_abort_handle, + df_abort_handle, + dag_rpc_tx, + network_events, + }, + ordered_nodes_rx, + ) + } + + async fn start(mut self) { + loop { + match self.network_events.next().await.unwrap() { + Event::RpcRequest(sender, msg, protocol, response_sender) => match msg { + ConsensusMsg::DAGMessage(msg) => { + debug!("handling RPC..."); + self.dag_rpc_tx.push(sender, IncomingDAGRequest { + req: msg, + sender, + protocol, + response_sender, + }) + }, + _ => unreachable!("expected only DAG-related messages"), + }, + _ => panic!("Unexpected Network Event"), + } + .unwrap() + } + } +} + +fn create_network( + playground: &mut NetworkPlayground, + id: usize, + author: Author, + validators: ValidatorVerifier, +) -> ( + NetworkSender, + Box, aptos_channels::Receiver>>>, +) { + let (network_reqs_tx, network_reqs_rx) = aptos_channel::new(QueueStyle::FIFO, 8, None); + let (connection_reqs_tx, _) = aptos_channel::new(QueueStyle::FIFO, 8, None); + let (consensus_tx, consensus_rx) = aptos_channel::new(QueueStyle::FIFO, 8, None); + let (_conn_mgr_reqs_tx, conn_mgr_reqs_rx) = aptos_channels::new_test(8); + let (_, conn_status_rx) = conn_notifs_channel::new(); + let network_sender = network::NetworkSender::new( + PeerManagerRequestSender::new(network_reqs_tx), + ConnectionRequestSender::new(connection_reqs_tx), + ); + let network_client = NetworkClient::new( + DIRECT_SEND.into(), + RPC.into(), + hashmap! {NetworkId::Validator => network_sender}, + playground.peer_protocols(), + ); + let consensus_network_client = ConsensusNetworkClient::new(network_client); + let network_events = NetworkEvents::new(consensus_rx, conn_status_rx, None); + + let (self_sender, self_receiver) = aptos_channels::new_test(1000); + let network = NetworkSender::new(author, consensus_network_client, self_sender, validators); + + let twin_id = TwinId { id, author }; + + playground.add_node(twin_id, consensus_tx, network_reqs_rx, conn_mgr_reqs_rx); + + let all_network_events = Box::new(select(network_events, self_receiver)); + + (network, all_network_events) +} + +fn bootstrap_nodes( + playground: &mut NetworkPlayground, + signers: Vec, + validators: ValidatorVerifier, +) -> ( + Vec, + Vec>>>, +) { + let peers_and_metadata = playground.peer_protocols(); + let (nodes, ordered_node_receivers) = signers + .iter() + .enumerate() + .map(|(id, signer)| { + let peer_id = signer.author(); + let mut conn_meta = ConnectionMetadata::mock(peer_id); + conn_meta.application_protocols = ProtocolIdSet::from_iter([ + ProtocolId::ConsensusDirectSendJson, + ProtocolId::ConsensusDirectSendBcs, + ProtocolId::ConsensusRpcBcs, + ]); + let peer_network_id = PeerNetworkId::new(NetworkId::Validator, peer_id); + peers_and_metadata + .insert_connection_metadata(peer_network_id, conn_meta) + .unwrap(); + + let (_, storage) = MockStorage::start_for_testing((&validators).into()); + let (network, network_events) = + create_network(playground, id, signer.author(), validators.clone()); + + DagBootstrapUnit::make( + signer.author(), + 1, + signer.clone(), + storage, + network, + aptos_time_service::TimeService::real(), + network_events, + ) + }) + .unzip(); + + (nodes, ordered_node_receivers) +} + +#[tokio::test] +async fn test_dag_e2e() { + let num_nodes = 7; + let runtime = consensus_runtime(); + let mut playground = NetworkPlayground::new(runtime.handle().clone()); + let (signers, validators) = random_validator_verifier(num_nodes, None, false); + let author_indexes = validators.address_to_validator_index().clone(); + + let (nodes, mut ordered_node_receivers) = bootstrap_nodes(&mut playground, signers, validators); + for node in nodes { + runtime.spawn(node.start()); + } + + runtime.spawn(playground.start()); + + let display = |node: &Arc| { + ( + node.metadata().round(), + *author_indexes.get(node.metadata().author()).unwrap(), + ) + }; + + for _ in 1..10 { + let mut all_ordered = vec![]; + for receiver in &mut ordered_node_receivers { + let block = receiver.next().await.unwrap(); + all_ordered.push(block) + } + let first: Vec<_> = all_ordered.first().unwrap().iter().map(display).collect(); + assert_gt!(first.len(), 0, "must order nodes"); + debug!("Nodes: {:?}", first); + for ordered in all_ordered.iter() { + let a: Vec<_> = ordered.iter().map(display).collect(); + assert_eq!(a.len(), first.len(), "length should match"); + assert_eq!(a, first); + } + } + runtime.shutdown_background(); +} diff --git a/consensus/src/dag/tests/mod.rs b/consensus/src/dag/tests/mod.rs index bbe46c627fdae..115bcd045144a 100644 --- a/consensus/src/dag/tests/mod.rs +++ b/consensus/src/dag/tests/mod.rs @@ -6,6 +6,7 @@ mod dag_network_test; mod dag_test; mod fetcher_test; mod helpers; +mod integration_tests; mod order_rule_tests; mod rb_handler_tests; mod types_test;