mirror of
https://github.com/pezkuwichain/revive-differential-tests.git
synced 2026-04-22 21:57:58 +00:00
Compare commits
4 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| ca7af278cb | |||
| 429f2e92a2 | |||
| 65f41f2038 | |||
| 3ed8a1ca1c |
@@ -54,7 +54,7 @@ pub struct Arguments {
|
|||||||
pub geth: PathBuf,
|
pub geth: PathBuf,
|
||||||
|
|
||||||
/// The maximum time in milliseconds to wait for geth to start.
|
/// The maximum time in milliseconds to wait for geth to start.
|
||||||
#[arg(long = "geth-start-timeout", default_value = "2000")]
|
#[arg(long = "geth-start-timeout", default_value = "5000")]
|
||||||
pub geth_start_timeout: u64,
|
pub geth_start_timeout: u64,
|
||||||
|
|
||||||
/// The test network chain ID.
|
/// The test network chain ID.
|
||||||
|
|||||||
@@ -3,6 +3,7 @@
|
|||||||
use std::collections::HashMap;
|
use std::collections::HashMap;
|
||||||
use std::fmt::Debug;
|
use std::fmt::Debug;
|
||||||
use std::marker::PhantomData;
|
use std::marker::PhantomData;
|
||||||
|
use std::str::FromStr;
|
||||||
|
|
||||||
use alloy::json_abi::JsonAbi;
|
use alloy::json_abi::JsonAbi;
|
||||||
use alloy::network::{Ethereum, TransactionBuilder};
|
use alloy::network::{Ethereum, TransactionBuilder};
|
||||||
@@ -100,12 +101,30 @@ where
|
|||||||
anyhow::bail!("unsupported solc version: {:?}", &mode.solc_version);
|
anyhow::bail!("unsupported solc version: {:?}", &mode.solc_version);
|
||||||
};
|
};
|
||||||
|
|
||||||
|
// Note: if the metadata is contained within a solidity file then this is the only file that
|
||||||
|
// we wish to compile since this is a self-contained test. Otherwise, if it's a JSON file
|
||||||
|
// then we need to compile all of the contracts that are in the directory since imports are
|
||||||
|
// allowed in there.
|
||||||
|
let Some(ref metadata_file_path) = metadata.file_path else {
|
||||||
|
anyhow::bail!("The metadata file path is not defined");
|
||||||
|
};
|
||||||
|
let mut files_to_compile = if metadata_file_path
|
||||||
|
.extension()
|
||||||
|
.is_some_and(|extension| extension.eq_ignore_ascii_case("sol"))
|
||||||
|
{
|
||||||
|
Box::new(std::iter::once(metadata_file_path.clone())) as Box<dyn Iterator<Item = _>>
|
||||||
|
} else {
|
||||||
|
Box::new(
|
||||||
|
FilesWithExtensionIterator::new(metadata.directory()?)
|
||||||
|
.with_allowed_extension("sol"),
|
||||||
|
)
|
||||||
|
};
|
||||||
|
|
||||||
let compiler = Compiler::<T::Compiler>::new()
|
let compiler = Compiler::<T::Compiler>::new()
|
||||||
.allow_path(metadata.directory()?)
|
.allow_path(metadata.directory()?)
|
||||||
.solc_optimizer(mode.solc_optimize());
|
.solc_optimizer(mode.solc_optimize());
|
||||||
let mut compiler = FilesWithExtensionIterator::new(metadata.directory()?)
|
let mut compiler =
|
||||||
.with_allowed_extension("sol")
|
files_to_compile.try_fold(compiler, |compiler, path| compiler.with_source(&path))?;
|
||||||
.try_fold(compiler, |compiler, path| compiler.with_source(&path))?;
|
|
||||||
for (library_instance, (library_address, _)) in self.deployed_libraries.iter() {
|
for (library_instance, (library_address, _)) in self.deployed_libraries.iter() {
|
||||||
let library_ident = &metadata
|
let library_ident = &metadata
|
||||||
.contracts
|
.contracts
|
||||||
@@ -439,8 +458,18 @@ where
|
|||||||
expected_events.iter().zip(execution_receipt.logs())
|
expected_events.iter().zip(execution_receipt.logs())
|
||||||
{
|
{
|
||||||
// Handling the emitter assertion.
|
// Handling the emitter assertion.
|
||||||
if let Some(expected_address) = expected_event.address {
|
if let Some(ref expected_address) = expected_event.address {
|
||||||
let expected = expected_address;
|
let expected = if let Some(contract_instance) = expected_address
|
||||||
|
.strip_suffix(".address")
|
||||||
|
.map(ContractInstance::new)
|
||||||
|
{
|
||||||
|
deployed_contracts
|
||||||
|
.get(&contract_instance)
|
||||||
|
.map(|(address, _)| *address)
|
||||||
|
} else {
|
||||||
|
Address::from_str(expected_address).ok()
|
||||||
|
}
|
||||||
|
.context("Failed to get the address of the event")?;
|
||||||
let actual = actual_event.address();
|
let actual = actual_event.address();
|
||||||
if actual != expected {
|
if actual != expected {
|
||||||
tracing::error!(
|
tracing::error!(
|
||||||
|
|||||||
@@ -51,7 +51,7 @@ pub struct ExpectedOutput {
|
|||||||
|
|
||||||
#[derive(Clone, Debug, Default, Deserialize, Eq, PartialEq)]
|
#[derive(Clone, Debug, Default, Deserialize, Eq, PartialEq)]
|
||||||
pub struct Event {
|
pub struct Event {
|
||||||
pub address: Option<Address>,
|
pub address: Option<String>,
|
||||||
pub topics: Vec<String>,
|
pub topics: Vec<String>,
|
||||||
pub values: Calldata,
|
pub values: Calldata,
|
||||||
}
|
}
|
||||||
@@ -1010,4 +1010,53 @@ mod tests {
|
|||||||
// Assert
|
// Assert
|
||||||
assert!(resolved.is_err())
|
assert!(resolved.is_err())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn expected_json_can_be_deserialized1() {
|
||||||
|
// Arrange
|
||||||
|
let str = r#"
|
||||||
|
{
|
||||||
|
"return_data": [
|
||||||
|
"1"
|
||||||
|
],
|
||||||
|
"events": [
|
||||||
|
{
|
||||||
|
"topics": [],
|
||||||
|
"values": []
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
"#;
|
||||||
|
|
||||||
|
// Act
|
||||||
|
let expected = serde_json::from_str::<Expected>(str);
|
||||||
|
|
||||||
|
// Assert
|
||||||
|
expected.expect("Failed to deserialize");
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn expected_json_can_be_deserialized2() {
|
||||||
|
// Arrange
|
||||||
|
let str = r#"
|
||||||
|
{
|
||||||
|
"return_data": [
|
||||||
|
"1"
|
||||||
|
],
|
||||||
|
"events": [
|
||||||
|
{
|
||||||
|
"address": "Main.address",
|
||||||
|
"topics": [],
|
||||||
|
"values": []
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
"#;
|
||||||
|
|
||||||
|
// Act
|
||||||
|
let expected = serde_json::from_str::<Expected>(str);
|
||||||
|
|
||||||
|
// Assert
|
||||||
|
expected.expect("Failed to deserialize");
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -193,7 +193,7 @@ impl Metadata {
|
|||||||
metadata.file_path = Some(path.to_path_buf());
|
metadata.file_path = Some(path.to_path_buf());
|
||||||
metadata.contracts = Some(
|
metadata.contracts = Some(
|
||||||
[(
|
[(
|
||||||
ContractInstance::new("test"),
|
ContractInstance::new("Test"),
|
||||||
ContractPathAndIdent {
|
ContractPathAndIdent {
|
||||||
contract_source_path: path.to_path_buf(),
|
contract_source_path: path.to_path_buf(),
|
||||||
contract_ident: ContractIdent::new("Test"),
|
contract_ident: ContractIdent::new("Test"),
|
||||||
|
|||||||
Reference in New Issue
Block a user