Move cumulus zombienet tests to aura & async backing

cumulus/client/collator/src/lib.rs

@@ -381,7 +381,7 @@ mod tests {
 			sproof.included_para_head = Some(HeadData(parent.encode()));
 			sproof.para_id = cumulus_test_runtime::PARACHAIN_ID.into();
 
-			let builder = self.client.init_block_builder_at(
+			let (builder, ..) = self.client.init_block_builder_at(
 				parent.hash(),
 				Some(validation_data.clone()),
 				sproof,

cumulus/client/consensus/common/src/tests.rs

@@ -274,7 +274,7 @@ fn build_block<B: InitBlockBuilder>(
 	timestamp: Option<u64>,
 	relay_parent: Option<PHash>,
 ) -> Block {
-	let builder = match at {
+	let (builder, ..) = match at {
 		Some(at) => match timestamp {
 			Some(ts) => builder.init_block_builder_with_timestamp(at, None, sproof, ts),
 			None => builder.init_block_builder_at(at, None, sproof),
@@ -503,7 +503,7 @@ fn follow_finalized_does_not_stop_on_unknown_block() {
 
 	let unknown_block = {
 		let sproof = sproof_with_parent_by_hash(&client, block.hash());
-		let block_builder = client.init_block_builder_at(block.hash(), None, sproof);
+		let block_builder = client.init_block_builder_at(block.hash(), None, sproof).0;
 		block_builder.build().unwrap().block
 	};
 
@@ -553,7 +553,7 @@ fn follow_new_best_sets_best_after_it_is_imported() {
 
 	let unknown_block = {
 		let sproof = sproof_with_parent_by_hash(&client, block.hash());
-		let block_builder = client.init_block_builder_at(block.hash(), None, sproof);
+		let block_builder = client.init_block_builder_at(block.hash(), None, sproof).0;
 		block_builder.build().unwrap().block
 	};
 
@@ -665,8 +665,8 @@ fn do_not_set_best_block_to_older_block() {
 fn prune_blocks_on_level_overflow() {
 	// Here we are using the timestamp value to generate blocks with different hashes.
 	const LEVEL_LIMIT: usize = 3;
-	const TIMESTAMP_MULTIPLIER: u64 = 60000;
 
+	let mut ts_producer = std::iter::successors(Some(0), |&x| Some(x + 6000));
 	let backend = Arc::new(Backend::new_test(1000, 3));
 	let client = Arc::new(TestClientBuilder::with_backend(backend.clone()).build());
 	let mut para_import = ParachainBlockImport::new_with_limit(
@@ -675,13 +675,14 @@ fn prune_blocks_on_level_overflow() {
 		LevelLimit::Some(LEVEL_LIMIT),
 	);
 
+	let best_hash = client.chain_info().best_hash;
 	let block0 = build_and_import_block_ext(
 		&client,
 		BlockOrigin::NetworkInitialSync,
 		true,
 		&mut para_import,
-		None,
-		None,
+		Some(best_hash),
+		ts_producer.next(),
 		None,
 	);
 	let id0 = block0.header.hash();
@@ -694,22 +695,22 @@ fn prune_blocks_on_level_overflow() {
 				i == 1,
 				&mut para_import,
 				Some(id0),
-				Some(i as u64 * TIMESTAMP_MULTIPLIER),
+				ts_producer.next(),
 				None,
 			)
 		})
 		.collect::<Vec<_>>();
 	let id10 = blocks1[0].header.hash();
 
 	let blocks2 = (0..2)
-		.map(|i| {
+		.map(|_| {
 			build_and_import_block_ext(
 				&client,
 				BlockOrigin::Own,
 				false,
 				&mut para_import,
 				Some(id10),
-				Some(i as u64 * TIMESTAMP_MULTIPLIER),
+				ts_producer.next(),
 				None,
 			)
 		})
@@ -738,7 +739,7 @@ fn prune_blocks_on_level_overflow() {
 		false,
 		&mut para_import,
 		Some(id0),
-		Some(LEVEL_LIMIT as u64 * TIMESTAMP_MULTIPLIER),
+		ts_producer.next(),
 		None,
 	);
 
@@ -758,7 +759,7 @@ fn prune_blocks_on_level_overflow() {
 		false,
 		&mut para_import,
 		Some(id0),
-		Some(2 * LEVEL_LIMIT as u64 * TIMESTAMP_MULTIPLIER),
+		ts_producer.next(),
 		None,
 	);
 
@@ -777,10 +778,11 @@ fn prune_blocks_on_level_overflow() {
 
 #[test]
 fn restore_limit_monitor() {
+	sp_tracing::try_init_simple();
 	// Here we are using the timestamp value to generate blocks with different hashes.
 	const LEVEL_LIMIT: usize = 2;
-	const TIMESTAMP_MULTIPLIER: u64 = 60000;
-
+	// Iterator that produces a new timestamp in the next slot
+	let mut ts_producer = std::iter::successors(Some(0), |&x| Some(x + 6000));
 	let backend = Arc::new(Backend::new_test(1000, 3));
 	let client = Arc::new(TestClientBuilder::with_backend(backend.clone()).build());
 
@@ -791,13 +793,14 @@ fn restore_limit_monitor() {
 		LevelLimit::Some(usize::MAX),
 	);
 
+	let best_hash = client.chain_info().best_hash;
 	let block00 = build_and_import_block_ext(
 		&client,
 		BlockOrigin::NetworkInitialSync,
 		true,
 		&mut para_import,
-		None,
-		None,
+		Some(best_hash),
+		ts_producer.next(),
 		None,
 	);
 	let id00 = block00.header.hash();
@@ -810,26 +813,24 @@ fn restore_limit_monitor() {
 				i == 1,
 				&mut para_import,
 				Some(id00),
-				Some(i as u64 * TIMESTAMP_MULTIPLIER),
+				ts_producer.next(),
 				None,
 			)
 		})
 		.collect::<Vec<_>>();
 	let id10 = blocks1[0].header.hash();
 
-	let _ = (0..LEVEL_LIMIT)
-		.map(|i| {
-			build_and_import_block_ext(
-				&client,
-				BlockOrigin::Own,
-				false,
-				&mut para_import,
-				Some(id10),
-				Some(i as u64 * TIMESTAMP_MULTIPLIER),
-				None,
-			)
-		})
-		.collect::<Vec<_>>();
+	for _ in 0..LEVEL_LIMIT {
+		build_and_import_block_ext(
+			&client,
+			BlockOrigin::Own,
+			false,
+			&mut para_import,
+			Some(id10),
+			ts_producer.next(),
+			None,
+		);
+	}
 
 	// Scenario before limit application (with B11 imported as best)
 	// Import order (freshness): B00, B10, B11, B12, B20, B21
@@ -860,7 +861,7 @@ fn restore_limit_monitor() {
 		false,
 		&mut para_import,
 		Some(id00),
-		Some(LEVEL_LIMIT as u64 * TIMESTAMP_MULTIPLIER),
+		ts_producer.next(),
 		None,
 	);
 

cumulus/pallets/parachain-system/Cargo.toml

@@ -59,10 +59,12 @@ sp-keyring = { path = "../../../substrate/primitives/keyring" }
 sp-crypto-hashing = { path = "../../../substrate/primitives/crypto/hashing" }
 sp-tracing = { path = "../../../substrate/primitives/tracing" }
 sp-version = { path = "../../../substrate/primitives/version" }
+sp-consensus-slots = { path = "../../../substrate/primitives/consensus/slots" }
 
 # Cumulus
 cumulus-test-client = { path = "../../test/client" }
 cumulus-test-relay-sproof-builder = { path = "../../test/relay-sproof-builder" }
+cumulus-test-runtime = { path = "../../test/runtime" }
 
 [features]
 default = ["std"]