Compare commits

..

11 Commits

Author SHA1 Message Date
Omar Abdulla 63389ef199 Add information on the deployed contracts 2025-08-25 13:52:09 +03:00
Omar Abdulla 6443f52502 Cleanup report model 2025-08-25 05:06:11 +03:00
Omar Abdulla 3d198f4e71 Include contract compilation information in report 2025-08-25 05:02:19 +03:00
Omar Abdulla 7d87cb9ee8 Integrate the reporting infrastructure with the
CLI reporter used by the program.
2025-08-25 00:24:10 +03:00
Omar Abdulla 5d9075b389 Add case status information to the report 2025-08-24 23:38:50 +03:00
Omar Abdulla 4f55e8968d Update report to use better map data structures 2025-08-24 23:01:30 +03:00
Omar Abdulla 38e3915169 Add reporting infra for reporting ignored tests 2025-08-24 22:50:40 +03:00
Omar Abdulla 4d7d7ed843 Add more runner events to the reporter and refine the structure 2025-08-24 20:13:48 +03:00
Omar Abdulla cbb6a11404 Implement the initial set of reporter events 2025-08-24 19:45:01 +03:00
Omar Abdulla bd5ff6d75e Use the Test struct more in the code 2025-08-23 18:05:32 +03:00
Omar Abdulla c9303f0020 Remove the old reporting infra 2025-08-23 17:58:13 +03:00
2 changed files with 6 additions and 38 deletions
-12
View File
@@ -115,18 +115,6 @@ pub struct Arguments {
#[arg(short, long = "kitchensink", default_value = "substrate-node")]
pub kitchensink: PathBuf,
/// The path to the `revive-dev-node` executable.
///
/// By default it uses `revive-dev-node` binary found in `$PATH`.
#[arg(long = "revive-dev-node", default_value = "revive-dev-node")]
pub revive_dev_node: PathBuf,
/// By default the tool uses the revive-dev-node when it's running differential tests against
/// PolkaVM since the dev-node is much faster than kitchensink. This flag allows the caller to
/// configure the tool to use kitchensink rather than the dev-node.
#[arg(long)]
pub use_kitchensink_not_dev_node: bool,
/// The path to the `eth_proxy` executable.
///
/// By default it uses `eth-rpc` binary found in `$PATH`.
+6 -26
View File
@@ -54,7 +54,6 @@ static NODE_COUNT: AtomicU32 = AtomicU32::new(0);
pub struct KitchensinkNode {
id: u32,
substrate_binary: PathBuf,
dev_node_binary: PathBuf,
eth_proxy_binary: PathBuf,
rpc_url: String,
base_directory: PathBuf,
@@ -64,7 +63,6 @@ pub struct KitchensinkNode {
wallet: Arc<EthereumWallet>,
nonce_manager: CachedNonceManager,
chain_id_filler: ChainIdFiller,
use_kitchensink_not_dev_node: bool,
/// This vector stores [`File`] objects that we use for logging which we want to flush when the
/// node object is dropped. We do not store them in a structured fashion at the moment (in
/// separate fields) as the logic that we need to apply to them is all the same regardless of
@@ -103,21 +101,11 @@ impl KitchensinkNode {
// Note: we do not pipe the logs of this process to a separate file since this is just a
// once-off export of the default chain spec and not part of the long-running node process.
let output = if self.use_kitchensink_not_dev_node {
Command::new(&self.substrate_binary)
.arg("export-chain-spec")
.arg("--chain")
.arg("dev")
.output()
.context("Failed to export the chain-spec")?
} else {
Command::new(&self.dev_node_binary)
.arg("build-spec")
.arg("--chain")
.arg("dev")
.output()
.context("Failed to export the chain-spec")?
};
let output = Command::new(&self.substrate_binary)
.arg("export-chain-spec")
.arg("--chain")
.arg("dev")
.output()?;
if !output.status.success() {
anyhow::bail!(
@@ -200,12 +188,7 @@ impl KitchensinkNode {
let kitchensink_stderr_logs_file = open_options
.clone()
.open(self.kitchensink_stderr_log_file_path())?;
let node_binary_path = if self.use_kitchensink_not_dev_node {
self.substrate_binary.as_path()
} else {
self.dev_node_binary.as_path()
};
self.process_substrate = Command::new(node_binary_path)
self.process_substrate = Command::new(&self.substrate_binary)
.arg("--dev")
.arg("--chain")
.arg(chainspec_path)
@@ -550,7 +533,6 @@ impl Node for KitchensinkNode {
Self {
id,
substrate_binary: config.kitchensink.clone(),
dev_node_binary: config.revive_dev_node.clone(),
eth_proxy_binary: config.eth_proxy.clone(),
rpc_url: String::new(),
base_directory,
@@ -560,7 +542,6 @@ impl Node for KitchensinkNode {
wallet: Arc::new(wallet),
chain_id_filler: Default::default(),
nonce_manager: Default::default(),
use_kitchensink_not_dev_node: config.use_kitchensink_not_dev_node,
// We know that we only need to be storing 4 files so we can specify that when creating
// the vector. It's the stdout and stderr of the substrate-node and the eth-rpc.
logs_file_to_flush: Vec::with_capacity(4),
@@ -1078,7 +1059,6 @@ mod tests {
Arguments {
kitchensink: PathBuf::from("substrate-node"),
eth_proxy: PathBuf::from("eth-rpc"),
use_kitchensink_not_dev_node: true,
..Default::default()
}
}