Skip to content

Commit

Permalink
Remove unnecessary clone
Browse files Browse the repository at this point in the history
  • Loading branch information
agostbiro committed Sep 4, 2024
1 parent 576d6fa commit 0258c35
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion crates/edr_napi/src/solidity_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ pub fn run_solidity_tests(

let runtime = runtime::Handle::current();
runtime.spawn(async move {
let runner = match build_runner(&known_contracts, test_suites, config_args).await {
let runner = match build_runner(known_contracts, test_suites, config_args).await {
Ok(runner) => runner,
Err(error) => {
let call_status =
Expand Down
4 changes: 2 additions & 2 deletions crates/edr_napi/src/solidity_tests/runner.rs
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ use foundry_common::ContractsByArtifact;
use crate::solidity_tests::config::SolidityTestRunnerConfigArgs;

pub(super) async fn build_runner(
known_contracts: &ContractsByArtifact,
known_contracts: ContractsByArtifact,
test_suites: Vec<foundry_common::ArtifactId>,
config_args: SolidityTestRunnerConfigArgs,
) -> napi::Result<MultiContractRunner> {
Expand Down Expand Up @@ -44,7 +44,7 @@ pub(super) async fn build_runner(
})
.collect::<napi::Result<TestContracts>>()?;

MultiContractRunner::new(config, contracts, known_contracts.clone(), revert_decoder)
MultiContractRunner::new(config, contracts, known_contracts, revert_decoder)
.await
.map_err(|err| {
napi::Error::new(
Expand Down

0 comments on commit 0258c35

Please sign in to comment.