Skip to content

Commit

Permalink
remove warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
Jrigada committed Jul 9, 2024
1 parent b7b2a88 commit 4059706
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 13 deletions.
9 changes: 2 additions & 7 deletions crates/evm/traces/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -305,16 +305,11 @@ pub fn load_contracts<'a>(
let decoder = CallTraceDecoder::new();
let mut contracts = ContractsByAddress::new();
for trace in traces {
let a = decoder.trace_addresses(trace);
for (address, _) in a.clone() {
warn!("address in trace: {:?}", address);
}
let identified_addresses = local_identifier.identify_addresses(a);
let identified_addresses =
local_identifier.identify_addresses(decoder.trace_addresses(trace));
for address in identified_addresses {
let contract = address.contract;
let abi = address.abi;
warn!("contract: {:?}", contract);
warn!("abi: {:?}", abi);
if let (Some(contract), Some(abi)) = (contract, abi) {
contracts.insert(address.address, (contract, abi.into_owned()));
}
Expand Down
1 change: 0 additions & 1 deletion crates/forge/src/multi_runner.rs
Original file line number Diff line number Diff line change
Expand Up @@ -453,7 +453,6 @@ impl MultiContractRunnerBuilder {
immutable_references: BTreeMap::new(),
}),
};
warn!("id: {:?}", id);
zk_contracts_map.insert(id.clone(), compact_contract);
}
}
Expand Down
2 changes: 0 additions & 2 deletions crates/forge/src/runner.rs
Original file line number Diff line number Diff line change
Expand Up @@ -339,7 +339,6 @@ impl<'a> ContractRunner<'a> {

// Invariant testing requires tracing to figure out what contracts were created.
let has_invariants = self.contract.abi.functions().any(|func| func.is_invariant_test());
warn!("has_invariants: {:?}", has_invariants);
let tmp_tracing = self.executor.inspector.tracer.is_none() && has_invariants && call_setup;
if tmp_tracing {
self.executor.set_tracing(true);
Expand Down Expand Up @@ -397,7 +396,6 @@ impl<'a> ContractRunner<'a> {
&setup.deployments,
)
});
warn!("identified_contracts: {:?}", identified_contracts);
let test_results = functions
.par_iter()
.map(|&func| {
Expand Down
2 changes: 1 addition & 1 deletion zk-tests/foundry.toml
Original file line number Diff line number Diff line change
Expand Up @@ -19,4 +19,4 @@ testnet = "https://testnet.era.zksync.dev:443"

[profile.default.invariant]
runs = 10
depth = 1
depth = 1
2 changes: 1 addition & 1 deletion zk-tests/src/Deposit.sol
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,4 @@ contract Deposit {
(bool s, ) = msg.sender.call{value: amount}("");
require(s, "failed to send");
}
}
}
2 changes: 1 addition & 1 deletion zk-tests/src/InvariantDeposit.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -23,4 +23,4 @@ contract InvariantDeposit is Test {
}

receive() external payable {}
}
}

0 comments on commit 4059706

Please sign in to comment.