From abba0cee08105c8721bcd493b163c888d2601c65 Mon Sep 17 00:00:00 2001 From: Omar Abdulla Date: Fri, 11 Jul 2025 11:26:08 +0300 Subject: [PATCH 1/5] Introduce a custom kitchensink network --- .gitignore | 4 + Cargo.lock | 1 + Cargo.toml | 3 + crates/node/Cargo.toml | 1 + crates/node/src/kitchensink.rs | 457 ++++++++++++++++++++++++++++++++- 5 files changed, 456 insertions(+), 10 deletions(-) diff --git a/.gitignore b/.gitignore index 10a2f71..cf26eca 100644 --- a/.gitignore +++ b/.gitignore @@ -3,3 +3,7 @@ .DS_Store node_modules /*.json + +# We do not want to commit any log files that we produce from running the code locally so this is +# added to the .gitignore file. +*.log \ No newline at end of file diff --git a/Cargo.lock b/Cargo.lock index 9d729db..0cff4c8 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -3998,6 +3998,7 @@ dependencies = [ "anyhow", "revive-dt-config", "revive-dt-node-interaction", + "serde", "serde_json", "sp-core", "sp-runtime", diff --git a/Cargo.toml b/Cargo.toml index a28e194..5d1d6a7 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -68,6 +68,9 @@ features = [ "rpc-types", "signer-local", "std", + "network", + "serde", + "rpc-types-eth", ] [profile.bench] diff --git a/crates/node/Cargo.toml b/crates/node/Cargo.toml index 1a9dcb0..1b54fc9 100644 --- a/crates/node/Cargo.toml +++ b/crates/node/Cargo.toml @@ -16,6 +16,7 @@ tracing = { workspace = true } revive-dt-node-interaction = { workspace = true } revive-dt-config = { workspace = true } +serde = { workspace = true } serde_json = { workspace = true } sp-core = { workspace = true } diff --git a/crates/node/src/kitchensink.rs b/crates/node/src/kitchensink.rs index 4f14a18..de6d216 100644 --- a/crates/node/src/kitchensink.rs +++ b/crates/node/src/kitchensink.rs @@ -12,16 +12,15 @@ use std::{ }; use alloy::{ - hex, - network::EthereumWallet, - primitives::Address, - providers::{Provider, ProviderBuilder, ext::DebugApi}, - rpc::types::{ - TransactionReceipt, - trace::geth::{DiffMode, GethDebugTracingOptions, PreStateConfig, PreStateFrame}, - }, + consensus::{BlockHeader, TxEnvelope}, hex, network::{ + Ethereum, EthereumWallet, Network, TransactionBuilder, TransactionBuilderError, + UnbuiltTransactionError, + }, primitives::{Address, BlockNumber, Bloom, Bytes, B256, B64, U256}, providers::{ext::DebugApi, Provider, ProviderBuilder}, rpc::types::{ + eth::{Block, Header, Transaction}, trace::geth::{DiffMode, GethDebugTracingOptions, PreStateConfig, PreStateFrame}, TransactionReceipt + } }; use serde_json::{Value as JsonValue, json}; +use serde::{Serialize, Deserialize}; use sp_core::crypto::Ss58Codec; use sp_runtime::AccountId32; @@ -254,8 +253,10 @@ impl EthereumNode for KitchensinkNode { tracing::debug!("Submitting transaction: {transaction:#?}"); - execute_transaction(Box::pin(async move { + tracing::info!("Submitting tx to kitchensink"); + let receipt = execute_transaction(Box::pin(async move { Ok(ProviderBuilder::new() + .network::() .wallet(wallet) .connect(&url) .await? @@ -263,7 +264,9 @@ impl EthereumNode for KitchensinkNode { .await? .get_receipt() .await?) - })) + })); + tracing::info!(?receipt, "Submitted tx to kitchensink"); + receipt } fn trace_transaction( @@ -281,6 +284,7 @@ impl EthereumNode for KitchensinkNode { trace_transaction(Box::pin(async move { Ok(ProviderBuilder::new() + .network::() .wallet(wallet) .connect(&url) .await? @@ -374,6 +378,439 @@ impl Drop for KitchensinkNode { } } +#[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Hash)] +struct KitchenSinkNetwork; + +impl Network for KitchenSinkNetwork { + type TxType = ::TxType; + + type TxEnvelope = ::TxEnvelope; + + type UnsignedTx = ::UnsignedTx; + + type ReceiptEnvelope = ::ReceiptEnvelope; + + type Header = KitchenSinkHeader; + + type TransactionRequest = ::TransactionRequest; + + type TransactionResponse = ::TransactionResponse; + + type ReceiptResponse = ::ReceiptResponse; + + type HeaderResponse = Header; + + type BlockResponse = Block, Header>; +} + +impl TransactionBuilder for ::TransactionRequest { + fn chain_id(&self) -> Option { + <::TransactionRequest as TransactionBuilder>::chain_id(self) + } + + fn set_chain_id(&mut self, chain_id: alloy::primitives::ChainId) { + <::TransactionRequest as TransactionBuilder>::set_chain_id( + self, chain_id, + ) + } + + fn nonce(&self) -> Option { + <::TransactionRequest as TransactionBuilder>::nonce(self) + } + + fn set_nonce(&mut self, nonce: u64) { + <::TransactionRequest as TransactionBuilder>::set_nonce( + self, nonce, + ) + } + + fn input(&self) -> Option<&alloy::primitives::Bytes> { + <::TransactionRequest as TransactionBuilder>::input(self) + } + + fn set_input>(&mut self, input: T) { + <::TransactionRequest as TransactionBuilder>::set_input( + self, input, + ) + } + + fn from(&self) -> Option
{ + <::TransactionRequest as TransactionBuilder>::from( + self, + ) + } + + fn set_from(&mut self, from: Address) { + <::TransactionRequest as TransactionBuilder>::set_from( + self, from + ) + } + + fn kind(&self) -> Option { + <::TransactionRequest as TransactionBuilder>::kind( + self, + ) + } + + fn clear_kind(&mut self) { + <::TransactionRequest as TransactionBuilder>::clear_kind( + self, + ) + } + + fn set_kind(&mut self, kind: alloy::primitives::TxKind) { + <::TransactionRequest as TransactionBuilder>::set_kind( + self, kind + ) + } + + fn value(&self) -> Option { + <::TransactionRequest as TransactionBuilder>::value( + self + ) + } + + fn set_value(&mut self, value: alloy::primitives::U256) { + <::TransactionRequest as TransactionBuilder>::set_value( + self, value + ) + } + + fn gas_price(&self) -> Option { + <::TransactionRequest as TransactionBuilder>::gas_price( + self, + ) + } + + fn set_gas_price(&mut self, gas_price: u128) { + <::TransactionRequest as TransactionBuilder>::set_gas_price( + self, gas_price + ) + } + + fn max_fee_per_gas(&self) -> Option { + <::TransactionRequest as TransactionBuilder>::max_fee_per_gas( + self, + ) + } + + fn set_max_fee_per_gas(&mut self, max_fee_per_gas: u128) { + <::TransactionRequest as TransactionBuilder>::set_max_fee_per_gas( + self, max_fee_per_gas + ) + } + + fn max_priority_fee_per_gas(&self) -> Option { + <::TransactionRequest as TransactionBuilder>::max_priority_fee_per_gas( + self, + ) + } + + fn set_max_priority_fee_per_gas(&mut self, max_priority_fee_per_gas: u128) { + <::TransactionRequest as TransactionBuilder>::set_max_priority_fee_per_gas( + self, max_priority_fee_per_gas + ) + } + + fn gas_limit(&self) -> Option { + <::TransactionRequest as TransactionBuilder>::gas_limit( + self, + ) + } + + fn set_gas_limit(&mut self, gas_limit: u64) { + <::TransactionRequest as TransactionBuilder>::set_gas_limit( + self, gas_limit + ) + } + + fn access_list(&self) -> Option<&alloy::rpc::types::AccessList> { + <::TransactionRequest as TransactionBuilder>::access_list( + self, + ) + } + + fn set_access_list(&mut self, access_list: alloy::rpc::types::AccessList) { + <::TransactionRequest as TransactionBuilder>::set_access_list( + self, access_list + ) + } + + fn complete_type( + &self, + ty: ::TxType, + ) -> Result<(), Vec<&'static str>> { + <::TransactionRequest as TransactionBuilder>::complete_type( + self, ty + ) + } + + fn can_submit(&self) -> bool { + <::TransactionRequest as TransactionBuilder>::can_submit( + self, + ) + } + + fn can_build(&self) -> bool { + <::TransactionRequest as TransactionBuilder>::can_build( + self, + ) + } + + fn output_tx_type(&self) -> ::TxType { + <::TransactionRequest as TransactionBuilder>::output_tx_type( + self, + ) + } + + fn output_tx_type_checked(&self) -> Option<::TxType> { + <::TransactionRequest as TransactionBuilder>::output_tx_type_checked( + self, + ) + } + + fn prep_for_submission(&mut self) { + <::TransactionRequest as TransactionBuilder>::prep_for_submission( + self, + ) + } + + fn build_unsigned( + self, + ) -> alloy::network::BuildResult<::UnsignedTx, KitchenSinkNetwork> + { + let result = <::TransactionRequest as TransactionBuilder>::build_unsigned( + self, + ); + match result { + Ok(unsigned_tx) => Ok(unsigned_tx), + Err(UnbuiltTransactionError { request, error }) => { + Err(UnbuiltTransactionError:: { + request: request, + error: match error { + TransactionBuilderError::InvalidTransactionRequest(tx_type, items) => { + TransactionBuilderError::InvalidTransactionRequest(tx_type, items) + } + TransactionBuilderError::UnsupportedSignatureType => { + TransactionBuilderError::UnsupportedSignatureType + } + TransactionBuilderError::Signer(error) => { + TransactionBuilderError::Signer(error) + } + TransactionBuilderError::Custom(error) => { + TransactionBuilderError::Custom(error) + } + }, + }) + } + } + } + + async fn build>( + self, + wallet: &W, + ) -> Result< + ::TxEnvelope, + TransactionBuilderError, + > { + Ok(wallet.sign_request(self).await?) + } +} + +#[derive(Clone, Debug, PartialEq, Eq, Hash)] +#[derive(Serialize, Deserialize)] +#[serde(rename_all = "camelCase")] +pub struct KitchenSinkHeader { + /// The Keccak 256-bit hash of the parent + /// block’s header, in its entirety; formally Hp. + pub parent_hash: B256, + /// The Keccak 256-bit hash of the ommers list portion of this block; formally Ho. + #[serde(rename = "sha3Uncles", alias = "ommersHash")] + pub ommers_hash: B256, + /// The 160-bit address to which all fees collected from the successful mining of this block + /// be transferred; formally Hc. + #[serde(rename = "miner", alias = "beneficiary")] + pub beneficiary: Address, + /// The Keccak 256-bit hash of the root node of the state trie, after all transactions are + /// executed and finalisations applied; formally Hr. + pub state_root: B256, + /// The Keccak 256-bit hash of the root node of the trie structure populated with each + /// transaction in the transactions list portion of the block; formally Ht. + pub transactions_root: B256, + /// The Keccak 256-bit hash of the root node of the trie structure populated with the receipts + /// of each transaction in the transactions list portion of the block; formally He. + pub receipts_root: B256, + /// The Bloom filter composed from indexable information (logger address and log topics) + /// contained in each log entry from the receipt of each transaction in the transactions list; + /// formally Hb. + pub logs_bloom: Bloom, + /// A scalar value corresponding to the difficulty level of this block. This can be calculated + /// from the previous block’s difficulty level and the timestamp; formally Hd. + pub difficulty: U256, + /// A scalar value equal to the number of ancestor blocks. The genesis block has a number of + /// zero; formally Hi. + #[serde(with = "alloy::serde::quantity")] + pub number: BlockNumber, + /// A scalar value equal to the current limit of gas expenditure per block; formally Hl. + // This is the main difference over the Ethereum network implementation. We use u128 here and + // not u64. + #[serde(with = "alloy::serde::quantity")] + pub gas_limit: u128, + /// A scalar value equal to the total gas used in transactions in this block; formally Hg. + #[serde(with = "alloy::serde::quantity")] + pub gas_used: u64, + /// A scalar value equal to the reasonable output of Unix’s time() at this block’s inception; + /// formally Hs. + #[serde(with = "alloy::serde::quantity")] + pub timestamp: u64, + /// An arbitrary byte array containing data relevant to this block. This must be 32 bytes or + /// fewer; formally Hx. + pub extra_data: Bytes, + /// A 256-bit hash which, combined with the + /// nonce, proves that a sufficient amount of computation has been carried out on this block; + /// formally Hm. + pub mix_hash: B256, + /// A 64-bit value which, combined with the mixhash, proves that a sufficient amount of + /// computation has been carried out on this block; formally Hn. + pub nonce: B64, + /// A scalar representing EIP1559 base fee which can move up or down each block according + /// to a formula which is a function of gas used in parent block and gas target + /// (block gas limit divided by elasticity multiplier) of parent block. + /// The algorithm results in the base fee per gas increasing when blocks are + /// above the gas target, and decreasing when blocks are below the gas target. The base fee per + /// gas is burned. + #[serde( + default, + with = "alloy::serde::quantity::opt", + skip_serializing_if = "Option::is_none" + )] + pub base_fee_per_gas: Option, + /// The Keccak 256-bit hash of the withdrawals list portion of this block. + /// + #[serde(default, skip_serializing_if = "Option::is_none")] + pub withdrawals_root: Option, + /// The total amount of blob gas consumed by the transactions within the block, added in + /// EIP-4844. + #[serde( + default, + with = "alloy::serde::quantity::opt", + skip_serializing_if = "Option::is_none" + )] + pub blob_gas_used: Option, + /// A running total of blob gas consumed in excess of the target, prior to the block. Blocks + /// with above-target blob gas consumption increase this value, blocks with below-target blob + /// gas consumption decrease it (bounded at 0). This was added in EIP-4844. + #[serde( + default, + with = "alloy::serde::quantity::opt", + skip_serializing_if = "Option::is_none" + )] + pub excess_blob_gas: Option, + /// The hash of the parent beacon block's root is included in execution blocks, as proposed by + /// EIP-4788. + /// + /// This enables trust-minimized access to consensus state, supporting staking pools, bridges, + /// and more. + /// + /// The beacon roots contract handles root storage, enhancing Ethereum's functionalities. + #[serde(default, skip_serializing_if = "Option::is_none")] + pub parent_beacon_block_root: Option, + /// The Keccak 256-bit hash of the an RLP encoded list with each + /// [EIP-7685] request in the block body. + /// + /// [EIP-7685]: https://eips.ethereum.org/EIPS/eip-7685 + #[serde(default, skip_serializing_if = "Option::is_none")] + pub requests_hash: Option, +} + +impl BlockHeader for KitchenSinkHeader { + fn parent_hash(&self) -> B256 { + self.parent_hash + } + + fn ommers_hash(&self) -> B256 { + self.ommers_hash + } + + fn beneficiary(&self) -> Address { + self.beneficiary + } + + fn state_root(&self) -> B256 { + self.state_root + } + + fn transactions_root(&self) -> B256 { + self.transactions_root + } + + fn receipts_root(&self) -> B256 { + self.receipts_root + } + + fn withdrawals_root(&self) -> Option { + self.withdrawals_root + } + + fn logs_bloom(&self) -> Bloom { + self.logs_bloom + } + + fn difficulty(&self) -> U256 { + self.difficulty + } + + fn number(&self) -> BlockNumber { + self.number + } + + // There's sadly nothing that we can do about this. We're required to implement this trait on + // any type that represents a header and the gas limit type used here is a u64. + fn gas_limit(&self) -> u64 { + self.gas_limit.try_into().unwrap_or(u64::MAX) + } + + fn gas_used(&self) -> u64 { + self.gas_used + } + + fn timestamp(&self) -> u64 { + self.timestamp + } + + fn mix_hash(&self) -> Option { + Some(self.mix_hash) + } + + fn nonce(&self) -> Option { + Some(self.nonce) + } + + fn base_fee_per_gas(&self) -> Option { + self.base_fee_per_gas + } + + fn blob_gas_used(&self) -> Option { + self.blob_gas_used + } + + fn excess_blob_gas(&self) -> Option { + self.excess_blob_gas + } + + fn parent_beacon_block_root(&self) -> Option { + self.parent_beacon_block_root + } + + fn requests_hash(&self) -> Option { + self.requests_hash + } + + fn extra_data(&self) -> &Bytes { + &self.extra_data + } +} + #[cfg(test)] mod tests { use revive_dt_config::Arguments; From f6374ad52aeab8c012b303a27842d6b9460911eb Mon Sep 17 00:00:00 2001 From: Omar Abdulla Date: Fri, 11 Jul 2025 14:49:05 +0300 Subject: [PATCH 2/5] fix formatting --- crates/node/src/kitchensink.rs | 79 ++++++++++++++++------------------ 1 file changed, 37 insertions(+), 42 deletions(-) diff --git a/crates/node/src/kitchensink.rs b/crates/node/src/kitchensink.rs index de6d216..a36a6d6 100644 --- a/crates/node/src/kitchensink.rs +++ b/crates/node/src/kitchensink.rs @@ -12,15 +12,22 @@ use std::{ }; use alloy::{ - consensus::{BlockHeader, TxEnvelope}, hex, network::{ + consensus::{BlockHeader, TxEnvelope}, + hex, + network::{ Ethereum, EthereumWallet, Network, TransactionBuilder, TransactionBuilderError, UnbuiltTransactionError, - }, primitives::{Address, BlockNumber, Bloom, Bytes, B256, B64, U256}, providers::{ext::DebugApi, Provider, ProviderBuilder}, rpc::types::{ - eth::{Block, Header, Transaction}, trace::geth::{DiffMode, GethDebugTracingOptions, PreStateConfig, PreStateFrame}, TransactionReceipt - } + }, + primitives::{Address, B64, B256, BlockNumber, Bloom, Bytes, U256}, + providers::{Provider, ProviderBuilder, ext::DebugApi}, + rpc::types::{ + TransactionReceipt, + eth::{Block, Header, Transaction}, + trace::geth::{DiffMode, GethDebugTracingOptions, PreStateConfig, PreStateFrame}, + }, }; +use serde::{Deserialize, Serialize}; use serde_json::{Value as JsonValue, json}; -use serde::{Serialize, Deserialize}; use sp_core::crypto::Ss58Codec; use sp_runtime::AccountId32; @@ -435,21 +442,17 @@ impl TransactionBuilder for ::Transacti } fn from(&self) -> Option
{ - <::TransactionRequest as TransactionBuilder>::from( - self, - ) + <::TransactionRequest as TransactionBuilder>::from(self) } fn set_from(&mut self, from: Address) { <::TransactionRequest as TransactionBuilder>::set_from( - self, from + self, from, ) } fn kind(&self) -> Option { - <::TransactionRequest as TransactionBuilder>::kind( - self, - ) + <::TransactionRequest as TransactionBuilder>::kind(self) } fn clear_kind(&mut self) { @@ -460,37 +463,33 @@ impl TransactionBuilder for ::Transacti fn set_kind(&mut self, kind: alloy::primitives::TxKind) { <::TransactionRequest as TransactionBuilder>::set_kind( - self, kind + self, kind, ) } fn value(&self) -> Option { - <::TransactionRequest as TransactionBuilder>::value( - self - ) + <::TransactionRequest as TransactionBuilder>::value(self) } fn set_value(&mut self, value: alloy::primitives::U256) { <::TransactionRequest as TransactionBuilder>::set_value( - self, value + self, value, ) } fn gas_price(&self) -> Option { - <::TransactionRequest as TransactionBuilder>::gas_price( - self, - ) + <::TransactionRequest as TransactionBuilder>::gas_price(self) } fn set_gas_price(&mut self, gas_price: u128) { <::TransactionRequest as TransactionBuilder>::set_gas_price( - self, gas_price + self, gas_price, ) } fn max_fee_per_gas(&self) -> Option { <::TransactionRequest as TransactionBuilder>::max_fee_per_gas( - self, + self, ) } @@ -502,7 +501,7 @@ impl TransactionBuilder for ::Transacti fn max_priority_fee_per_gas(&self) -> Option { <::TransactionRequest as TransactionBuilder>::max_priority_fee_per_gas( - self, + self, ) } @@ -513,26 +512,25 @@ impl TransactionBuilder for ::Transacti } fn gas_limit(&self) -> Option { - <::TransactionRequest as TransactionBuilder>::gas_limit( - self, - ) + <::TransactionRequest as TransactionBuilder>::gas_limit(self) } fn set_gas_limit(&mut self, gas_limit: u64) { <::TransactionRequest as TransactionBuilder>::set_gas_limit( - self, gas_limit + self, gas_limit, ) } fn access_list(&self) -> Option<&alloy::rpc::types::AccessList> { <::TransactionRequest as TransactionBuilder>::access_list( - self, + self, ) } fn set_access_list(&mut self, access_list: alloy::rpc::types::AccessList) { <::TransactionRequest as TransactionBuilder>::set_access_list( - self, access_list + self, + access_list, ) } @@ -541,37 +539,35 @@ impl TransactionBuilder for ::Transacti ty: ::TxType, ) -> Result<(), Vec<&'static str>> { <::TransactionRequest as TransactionBuilder>::complete_type( - self, ty + self, ty, ) } fn can_submit(&self) -> bool { <::TransactionRequest as TransactionBuilder>::can_submit( - self, + self, ) } fn can_build(&self) -> bool { - <::TransactionRequest as TransactionBuilder>::can_build( - self, - ) + <::TransactionRequest as TransactionBuilder>::can_build(self) } fn output_tx_type(&self) -> ::TxType { <::TransactionRequest as TransactionBuilder>::output_tx_type( - self, + self, ) } fn output_tx_type_checked(&self) -> Option<::TxType> { <::TransactionRequest as TransactionBuilder>::output_tx_type_checked( - self, + self, ) } fn prep_for_submission(&mut self) { <::TransactionRequest as TransactionBuilder>::prep_for_submission( - self, + self, ) } @@ -617,8 +613,7 @@ impl TransactionBuilder for ::Transacti } } -#[derive(Clone, Debug, PartialEq, Eq, Hash)] -#[derive(Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Hash, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct KitchenSinkHeader { /// The Keccak 256-bit hash of the parent @@ -652,7 +647,7 @@ pub struct KitchenSinkHeader { #[serde(with = "alloy::serde::quantity")] pub number: BlockNumber, /// A scalar value equal to the current limit of gas expenditure per block; formally Hl. - // This is the main difference over the Ethereum network implementation. We use u128 here and + // This is the main difference over the Ethereum network implementation. We use u128 here and // not u64. #[serde(with = "alloy::serde::quantity")] pub gas_limit: u128, @@ -764,8 +759,8 @@ impl BlockHeader for KitchenSinkHeader { self.number } - // There's sadly nothing that we can do about this. We're required to implement this trait on - // any type that represents a header and the gas limit type used here is a u64. + // There's sadly nothing that we can do about this. We're required to implement this trait on + // any type that represents a header and the gas limit type used here is a u64. fn gas_limit(&self) -> u64 { self.gas_limit.try_into().unwrap_or(u64::MAX) } From 4bab45711425db25b217347c0b522f03798280b5 Mon Sep 17 00:00:00 2001 From: Omar Abdulla Date: Fri, 11 Jul 2025 17:18:42 +0300 Subject: [PATCH 3/5] Added `--dev` to `substrate-node` arguments. This commit adds the `--dev` argument to the `substrate-node` to allow the chain to keep advancing as time goes own. We have found that if this option is not added then the chain won't advance forward. --- Cargo.lock | 1 + crates/node/Cargo.toml | 1 + crates/node/src/kitchensink.rs | 35 +++++++++++++++++++++++++++++++++- 3 files changed, 36 insertions(+), 1 deletion(-) diff --git a/Cargo.lock b/Cargo.lock index 0cff4c8..c125fee 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -4003,6 +4003,7 @@ dependencies = [ "sp-core", "sp-runtime", "temp-dir", + "tokio", "tracing", ] diff --git a/crates/node/Cargo.toml b/crates/node/Cargo.toml index 1b54fc9..8fd6150 100644 --- a/crates/node/Cargo.toml +++ b/crates/node/Cargo.toml @@ -24,3 +24,4 @@ sp-runtime = { workspace = true } [dev-dependencies] temp-dir = { workspace = true } +tokio = { workspace = true } diff --git a/crates/node/src/kitchensink.rs b/crates/node/src/kitchensink.rs index a36a6d6..f93d539 100644 --- a/crates/node/src/kitchensink.rs +++ b/crates/node/src/kitchensink.rs @@ -128,6 +128,7 @@ impl KitchensinkNode { // Start Substrate node let mut substrate_process = Command::new(&self.substrate_binary) + .arg("--dev") .arg("--chain") .arg(chainspec_path) .arg("--base-path") @@ -808,13 +809,14 @@ impl BlockHeader for KitchenSinkHeader { #[cfg(test)] mod tests { + use alloy::rpc::types::TransactionRequest; use revive_dt_config::Arguments; use std::path::PathBuf; use temp_dir::TempDir; use std::fs; - use super::KitchensinkNode; + use super::*; use crate::{GENESIS_JSON, Node}; fn test_config() -> (Arguments, TempDir) { @@ -829,6 +831,37 @@ mod tests { (config, temp_dir) } + #[tokio::test] + async fn node_mines_simple_transfer_transaction_and_returns_receipt() { + // Arrange + let (args, _temp_dir) = test_config(); + let mut node = KitchensinkNode::new(&args); + node.spawn(GENESIS_JSON.to_owned()) + .expect("Failed to spawn the node"); + + let provider = ProviderBuilder::new() + .network::() + .wallet(args.wallet()) + .connect(&node.rpc_url) + .await + .expect("Failed to create provider"); + + let account_address = args.wallet().default_signer().address(); + let transaction = TransactionRequest::default() + .to(account_address) + .value(U256::from(100_000_000_000_000u128)); + + // Act + let receipt = provider.send_transaction(transaction).await; + + // Assert + let _ = receipt + .expect("Failed to send the transfer transaction") + .get_receipt() + .await + .expect("Failed to get the receipt for the transfer"); + } + #[test] fn test_init_generates_chainspec_with_balances() { let genesis_content = r#" From 7664e9735e44809eb8d604acc01c89db480647fd Mon Sep 17 00:00:00 2001 From: Omar Abdulla Date: Fri, 11 Jul 2025 17:20:36 +0300 Subject: [PATCH 4/5] fix clippy warning --- crates/node/src/kitchensink.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/crates/node/src/kitchensink.rs b/crates/node/src/kitchensink.rs index f93d539..e621164 100644 --- a/crates/node/src/kitchensink.rs +++ b/crates/node/src/kitchensink.rs @@ -583,7 +583,7 @@ impl TransactionBuilder for ::Transacti Ok(unsigned_tx) => Ok(unsigned_tx), Err(UnbuiltTransactionError { request, error }) => { Err(UnbuiltTransactionError:: { - request: request, + request, error: match error { TransactionBuilderError::InvalidTransactionRequest(tx_type, items) => { TransactionBuilderError::InvalidTransactionRequest(tx_type, items) From 76c85f191ca3bd9f0ba3999556b36fdfe801dcfb Mon Sep 17 00:00:00 2001 From: Omar Abdulla Date: Fri, 11 Jul 2025 17:31:42 +0300 Subject: [PATCH 5/5] fix clippy warning --- crates/node/src/kitchensink.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/crates/node/src/kitchensink.rs b/crates/node/src/kitchensink.rs index a36a6d6..239fe13 100644 --- a/crates/node/src/kitchensink.rs +++ b/crates/node/src/kitchensink.rs @@ -582,7 +582,7 @@ impl TransactionBuilder for ::Transacti Ok(unsigned_tx) => Ok(unsigned_tx), Err(UnbuiltTransactionError { request, error }) => { Err(UnbuiltTransactionError:: { - request: request, + request, error: match error { TransactionBuilderError::InvalidTransactionRequest(tx_type, items) => { TransactionBuilderError::InvalidTransactionRequest(tx_type, items)