Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Foundry-like tests #1231

Merged
merged 7 commits into from
Oct 10, 2023

clippy fix

e335988
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

Foundry-like tests #1231

clippy fix
e335988
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
GitHub Actions / clippy succeeded Oct 10, 2023 in 0s

reviewdog [clippy] report

reported by reviewdog 🐶

Findings (0)
Filtered Findings (15)

contracts/feature-tests/composability/transfer-role-features/tests/transfer_role_blackbox_test.rs|57 col 60| warning: redundant clone
--> contracts/feature-tests/composability/transfer-role-features/tests/transfer_role_blackbox_test.rs:57:60
|
57 | Account::new().nonce(1).code(vault_code.clone()),
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/feature-tests/composability/transfer-role-features/tests/transfer_role_blackbox_test.rs:57:50
|
57 | Account::new().nonce(1).code(vault_code.clone()),
| ^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
= note: #[warn(clippy::redundant_clone)] on by default
contracts/feature-tests/composability/transfer-role-features/tests/transfer_role_whitebox_test.rs|64 col 48| warning: redundant clone
--> contracts/feature-tests/composability/transfer-role-features/tests/transfer_role_whitebox_test.rs:64:48
|
64 | Account::new().nonce(1).code(vault_code.clone()),
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/feature-tests/composability/transfer-role-features/tests/transfer_role_whitebox_test.rs:64:38
|
64 | Account::new().nonce(1).code(vault_code.clone()),
| ^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
= note: #[warn(clippy::redundant_clone)] on by default
contracts/examples/rewards-distribution/tests/rewards_distribution_blackbox_test.rs|157 col 48| warning: redundant clone
--> contracts/examples/rewards-distribution/tests/rewards_distribution_blackbox_test.rs:157:48
|
157 | .code(rewards_distribution_code.clone())
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/examples/rewards-distribution/tests/rewards_distribution_blackbox_test.rs:157:23
|
157 | .code(rewards_distribution_code.clone())
| ^^^^^^^^^^^^^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
= note: #[warn(clippy::redundant_clone)] on by default
contracts/examples/rewards-distribution/tests/rewards_distribution_blackbox_test.rs|359 col 46| warning: redundant clone
--> contracts/examples/rewards-distribution/tests/rewards_distribution_blackbox_test.rs:359:46
|
359 | .multi_esdt_transfer(nft_payments.clone())
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/examples/rewards-distribution/tests/rewards_distribution_blackbox_test.rs:359:34
|
359 | .multi_esdt_transfer(nft_payments.clone())
| ^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
contracts/examples/multisig/tests/multisig_blackbox_test.rs|403 col 76| warning: redundant clone
--> contracts/examples/multisig/tests/multisig_blackbox_test.rs:403:76
|
403 | let action_id = state.propose_add_board_member(new_board_member_address.clone());
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/examples/multisig/tests/multisig_blackbox_test.rs:403:52
|
403 | let action_id = state.propose_add_board_member(new_board_member_address.clone());
| ^^^^^^^^^^^^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
= note: #[warn(clippy::redundant_clone)] on by default
contracts/examples/multisig/tests/multisig_blackbox_test.rs|458 col 37| warning: redundant clone
--> contracts/examples/multisig/tests/multisig_blackbox_test.rs:458:37
|
458 | new_user_address.clone(),
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/examples/multisig/tests/multisig_blackbox_test.rs:458:21
|
458 | new_user_address.clone(),
| ^^^^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
contracts/examples/multisig/tests/multisig_blackbox_test.rs|539 col 69| warning: redundant clone
--> contracts/examples/multisig/tests/multisig_blackbox_test.rs:539:69
|
539 | let action_id = state.propose_transfer_execute(new_adder_address.clone(), 0u64, adder_call);
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/examples/multisig/tests/multisig_blackbox_test.rs:539:52
|
539 | let action_id = state.propose_transfer_execute(new_adder_address.clone(), 0u64, adder_call);
| ^^^^^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
contracts/examples/multisig/tests/multisig_blackbox_test.rs|568 col 26| warning: redundant clone
--> contracts/examples/multisig/tests/multisig_blackbox_test.rs:568:26
|
568 | factorial_address.clone(),
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/examples/multisig/tests/multisig_blackbox_test.rs:568:9
|
568 | factorial_address.clone(),
| ^^^^^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs|139 col 47| warning: redundant clone
--> contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs:139:47
|
139 | .multi_esdt_transfer(nft_transfers.clone()),
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs:139:34
|
139 | .multi_esdt_transfer(nft_transfers.clone()),
| ^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
= note: #[warn(clippy::redundant_clone)] on by default
contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs|269 col 48| warning: redundant clone
--> contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs:269:48
|
269 | .multi_esdt_transfer(esdt_transfers.clone()),
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs:269:34
|
269 | .multi_esdt_transfer(esdt_transfers.clone()),
| ^^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs|334 col 52| warning: redundant clone
--> contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs:334:52
|
334 | .multi_esdt_transfer(combined_transfers.clone()),
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs:334:34
|
334 | .multi_esdt_transfer(combined_transfers.clone()),
| ^^^^^^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs|532 col 48| warning: redundant clone
--> contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs:532:48
|
532 | .multi_esdt_transfer(esdt_transfers.clone()),
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs:532:34
|
532 | .multi_esdt_transfer(esdt_transfers.clone()),
| ^^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs|750 col 47| warning: redundant clone
--> contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs:750:47
|
750 | .multi_esdt_transfer(nft_transfers.clone()),
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/feature-tests/use-module/tests/token_merge_module_whitebox_test.rs:750:34
|
750 | .multi_esdt_transfer(nft_transfers.clone()),
| ^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
contracts/feature-tests/composability/forwarder/tests/forwarder_whitebox_test.rs|43 col 41| warning: redundant clone
--> contracts/feature-tests/composability/forwarder/tests/forwarder_whitebox_test.rs:43:41
|
43 | .code(forwarder_code.clone())
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/feature-tests/composability/forwarder/tests/forwarder_whitebox_test.rs:43:27
|
43 | .code(forwarder_code.clone())
| ^^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
= note: #[warn(clippy::redundant_clone)] on by default
contracts/feature-tests/composability/forwarder/tests/forwarder_blackbox_test.rs|50 col 45| warning: redundant clone
--> contracts/feature-tests/composability/forwarder/tests/forwarder_blackbox_test.rs:50:45
|
50 | .code(forwarder_code.clone())
| ^^^^^^^^ help: remove this
|
note: this value is dropped without further use
--> contracts/feature-tests/composability/forwarder/tests/forwarder_blackbox_test.rs:50:31
|
50 | .code(forwarder_code.clone())
| ^^^^^^^^^^^^^^
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_clone
= note: #[warn(clippy::redundant_clone)] on by default