Remove unused dependencies from test files

OpenZeppelin · Aug 11, 2024 · 652eeac · 652eeac
1 parent 7e5b7b3
commit 652eeac
Show file tree

Hide file tree

Showing 21 changed files with 17 additions and 55 deletions.
diff --git a/packages/access/src/tests/test_ownable_twostep.cairo b/packages/access/src/tests/test_ownable_twostep.cairo
@@ -6,7 +6,6 @@ use openzeppelin_access::tests::mocks::ownable_mocks::DualCaseTwoStepOwnableMock
 use openzeppelin_test_common::ownable::OwnableSpyHelpers;
 use openzeppelin_testing::constants::{ZERO, OWNER, OTHER, NEW_OWNER};
 use openzeppelin_testing::events::EventSpyExt;
-use openzeppelin_utils::serde::SerializedAppend;
 use snforge_std::{EventSpy, spy_events, start_cheat_caller_address, test_address};
 use starknet::ContractAddress;
 

diff --git a/packages/account/src/tests/test_account.cairo b/packages/account/src/tests/test_account.cairo
@@ -24,7 +24,6 @@ use snforge_std::{
 };
 use snforge_std::{spy_events, declare, test_address, start_cheat_caller_address};
 use starknet::account::Call;
-use starknet::{contract_address_const, ContractAddress, ClassHash};
 
 //
 // Setup

diff --git a/packages/account/src/tests/test_dual_account.cairo b/packages/account/src/tests/test_dual_account.cairo
@@ -2,12 +2,12 @@ use openzeppelin_account::dual_account::{DualCaseAccountABI, DualCaseAccount};
 use openzeppelin_account::interface::{AccountABIDispatcherTrait, AccountABIDispatcher};
 use openzeppelin_introspection::interface::ISRC5_ID;
 
-use openzeppelin_test_common::account::{SIGNED_TX_DATA, get_accept_ownership_signature};
+use openzeppelin_test_common::account::{get_accept_ownership_signature};
 use openzeppelin_testing as utils;
 use openzeppelin_testing::constants::TRANSACTION_HASH;
 use openzeppelin_testing::constants::stark::{KEY_PAIR, KEY_PAIR_2};
 use openzeppelin_testing::signing::{StarkKeyPair, StarkSerializedSigning};
-use snforge_std::{declare, start_cheat_caller_address};
+use snforge_std::{start_cheat_caller_address};
 
 //
 // Setup

diff --git a/packages/account/src/tests/test_dual_eth_account.cairo b/packages/account/src/tests/test_dual_eth_account.cairo
@@ -1,7 +1,6 @@
 use openzeppelin_account::dual_eth_account::{DualCaseEthAccountABI, DualCaseEthAccount};
 use openzeppelin_account::interface::{EthAccountABIDispatcherTrait, EthAccountABIDispatcher};
 use openzeppelin_account::utils::secp256k1::{DebugSecp256k1Point, Secp256k1PointPartialEq};
-use openzeppelin_account::utils::signature::EthSignature;
 use openzeppelin_introspection::interface::ISRC5_ID;
 
 use openzeppelin_test_common::eth_account::get_accept_ownership_signature;

diff --git a/packages/account/src/tests/test_eth_account.cairo b/packages/account/src/tests/test_eth_account.cairo
@@ -1,4 +1,3 @@
-use core::starknet::SyscallResultTrait;
 use openzeppelin_account::EthAccountComponent::{InternalTrait, SRC6CamelOnlyImpl};
 use openzeppelin_account::EthAccountComponent::{PublicKeyCamelImpl, PublicKeyImpl};
 use openzeppelin_account::EthAccountComponent;
@@ -27,7 +26,6 @@ use snforge_std::{
     start_cheat_caller_address
 };
 use snforge_std::{spy_events, test_address};
-use starknet::ContractAddress;
 use starknet::account::Call;
 
 //

diff --git a/packages/governance/src/tests/test_timelock.cairo b/packages/governance/src/tests/test_timelock.cairo
@@ -1,22 +1,16 @@
 use core::hash::{HashStateTrait, HashStateExTrait};
-use core::num::traits::Zero;
 use core::pedersen::PedersenTrait;
 use openzeppelin_access::accesscontrol::AccessControlComponent::{
     AccessControlImpl, InternalImpl as AccessControlInternalImpl
 };
 use openzeppelin_access::accesscontrol::DEFAULT_ADMIN_ROLE;
 use openzeppelin_access::accesscontrol::interface::IACCESSCONTROL_ID;
 use openzeppelin_access::accesscontrol::interface::IAccessControl;
-use openzeppelin_governance::tests::mocks::timelock_mocks::MockContract;
 use openzeppelin_governance::tests::mocks::timelock_mocks::{
     IMockContractDispatcher, IMockContractDispatcherTrait
 };
-use openzeppelin_governance::tests::mocks::timelock_mocks::{
-    ITimelockAttackerDispatcher, ITimelockAttackerDispatcherTrait
-};
-use openzeppelin_governance::tests::mocks::timelock_mocks::{
-    TimelockControllerMock, TimelockAttackerMock
-};
+use openzeppelin_governance::tests::mocks::timelock_mocks::{ITimelockAttackerDispatcher};
+use openzeppelin_governance::tests::mocks::timelock_mocks::{TimelockControllerMock};
 use openzeppelin_governance::timelock::OperationState;
 use openzeppelin_governance::timelock::TimelockControllerComponent::{
     CallScheduled, CallExecuted, CallSalt, CallCancelled, MinDelayChanged

diff --git a/packages/presets/src/tests/test_universal_deployer.cairo b/packages/presets/src/tests/test_universal_deployer.cairo
@@ -9,7 +9,7 @@ use openzeppelin_utils::interfaces::{
     IUniversalDeployerDispatcher, IUniversalDeployerDispatcherTrait
 };
 use openzeppelin_utils::serde::SerializedAppend;
-use snforge_std::{EventSpy, spy_events, declare, start_cheat_caller_address};
+use snforge_std::{EventSpy, spy_events, start_cheat_caller_address};
 use starknet::{ClassHash, ContractAddress};
 
 fn ERC20_CLASS_HASH() -> ClassHash {

diff --git a/packages/test_common/src/account.cairo b/packages/test_common/src/account.cairo
@@ -5,7 +5,6 @@ use openzeppelin_account::AccountComponent;
 use openzeppelin_testing::constants::TRANSACTION_HASH;
 use openzeppelin_testing::events::EventSpyExt;
 use openzeppelin_testing::signing::StarkKeyPair;
-use openzeppelin_token::erc20::interface::IERC20Dispatcher;
 use snforge_std::EventSpy;
 use snforge_std::signature::stark_curve::StarkCurveSignerImpl;
 use starknet::ContractAddress;

diff --git a/packages/test_common/src/erc20.cairo b/packages/test_common/src/erc20.cairo
@@ -3,7 +3,7 @@ use openzeppelin_testing::constants::{NAME, SYMBOL};
 use openzeppelin_testing::events::EventSpyExt;
 use openzeppelin_token::erc20::ERC20Component::{Approval, Transfer};
 use openzeppelin_token::erc20::ERC20Component;
-use openzeppelin_token::erc20::interface::{IERC20Dispatcher, IERC20DispatcherTrait};
+use openzeppelin_token::erc20::interface::{IERC20Dispatcher};
 use openzeppelin_utils::serde::SerializedAppend;
 use snforge_std::EventSpy;
 use starknet::ContractAddress;

diff --git a/packages/test_common/src/ownable.cairo b/packages/test_common/src/ownable.cairo
@@ -1,7 +1,6 @@
 use openzeppelin_access::ownable::OwnableComponent::OwnershipTransferred;
 use openzeppelin_access::ownable::OwnableComponent;
 use openzeppelin_testing::EventSpyExt;
-use openzeppelin_utils::serde::SerializedAppend;
 use snforge_std::EventSpy;
 use starknet::ContractAddress;
 

diff --git a/packages/test_common/src/upgrades.cairo b/packages/test_common/src/upgrades.cairo
@@ -1,7 +1,7 @@
 use openzeppelin_testing::events::EventSpyExt;
 use openzeppelin_upgrades::UpgradeableComponent::Upgraded;
 use openzeppelin_upgrades::UpgradeableComponent;
-use snforge_std::{EventSpy, EventSpyAssertionsTrait};
+use snforge_std::{EventSpy};
 use starknet::{ContractAddress, ClassHash};
 
 #[generate_trait]

diff --git a/packages/testing/src/deployment.cairo b/packages/testing/src/deployment.cairo
@@ -1,7 +1,5 @@
-use core::starknet::SyscallResultTrait;
 use openzeppelin_testing::panic_data_to_byte_array;
-use snforge_std::{declare, get_class_hash, ContractClass, ContractClassTrait};
-use snforge_std::{start_cheat_caller_address, stop_cheat_caller_address};
+use snforge_std::{ContractClass, ContractClassTrait};
 use starknet::ContractAddress;
 
 /// Declares a contract with a `snforge` `declare` call and unwraps the result.

diff --git a/packages/testing/src/signing.cairo b/packages/testing/src/signing.cairo
@@ -1,6 +1,6 @@
 use snforge_std::signature::secp256k1_curve::{Secp256k1CurveSignerImpl, Secp256k1CurveKeyPairImpl};
 use snforge_std::signature::stark_curve::{StarkCurveSignerImpl, StarkCurveKeyPairImpl};
-use snforge_std::signature::{KeyPair, KeyPairTrait};
+use snforge_std::signature::{KeyPair};
 use starknet::secp256k1::Secp256k1Point;
 
 pub type StarkKeyPair = KeyPair<felt252, felt252>;

diff --git a/packages/token/src/tests/erc1155/test_dual1155_receiver.cairo b/packages/token/src/tests/erc1155/test_dual1155_receiver.cairo
@@ -4,12 +4,8 @@ use openzeppelin_token::erc1155::dual1155_receiver::{
     DualCaseERC1155Receiver, DualCaseERC1155ReceiverTrait
 };
 use openzeppelin_token::erc1155::interface::IERC1155_RECEIVER_ID;
-use openzeppelin_token::erc1155::interface::{
-    IERC1155ReceiverCamelDispatcher, IERC1155ReceiverCamelDispatcherTrait
-};
-use openzeppelin_token::erc1155::interface::{
-    IERC1155ReceiverDispatcher, IERC1155ReceiverDispatcherTrait
-};
+use openzeppelin_token::erc1155::interface::{IERC1155ReceiverCamelDispatcher};
+use openzeppelin_token::erc1155::interface::{IERC1155ReceiverDispatcher};
 
 //
 // Setup

diff --git a/packages/token/src/tests/erc1155/test_erc1155.cairo b/packages/token/src/tests/erc1155/test_erc1155.cairo
@@ -14,11 +14,9 @@ use openzeppelin_token::erc1155::ERC1155Component::ERC1155CamelImpl;
 use openzeppelin_token::erc1155::ERC1155Component::{
     ERC1155Impl, ERC1155MetadataURIImpl, InternalImpl
 };
-use openzeppelin_token::erc1155::ERC1155Component::{TransferBatch, ApprovalForAll, TransferSingle};
 use openzeppelin_token::erc1155::ERC1155Component;
 use openzeppelin_token::erc1155;
 use openzeppelin_token::tests::mocks::erc1155_mocks::DualCaseERC1155Mock;
-use openzeppelin_utils::serde::SerializedAppend;
 use snforge_std::{spy_events, test_address, start_cheat_caller_address};
 use starknet::ContractAddress;
 

diff --git a/packages/token/src/tests/erc20/test_erc20.cairo b/packages/token/src/tests/erc20/test_erc20.cairo
@@ -3,14 +3,11 @@ use openzeppelin_test_common::erc20::ERC20SpyHelpers;
 use openzeppelin_testing::constants::{
     ZERO, OWNER, SPENDER, RECIPIENT, NAME, SYMBOL, DECIMALS, SUPPLY, VALUE
 };
-use openzeppelin_token::erc20::ERC20Component::{Approval, Transfer};
 use openzeppelin_token::erc20::ERC20Component::{ERC20CamelOnlyImpl, ERC20Impl};
 use openzeppelin_token::erc20::ERC20Component::{ERC20MetadataImpl, InternalImpl};
 use openzeppelin_token::erc20::ERC20Component;
 use openzeppelin_token::tests::mocks::erc20_mocks::DualCaseERC20Mock;
-use openzeppelin_utils::serde::SerializedAppend;
 use snforge_std::{spy_events, test_address, start_cheat_caller_address};
-use starknet::ContractAddress;
 
 //
 // Setup

diff --git a/packages/token/src/tests/erc20/test_erc20_votes.cairo b/packages/token/src/tests/erc20/test_erc20_votes.cairo
@@ -13,15 +13,14 @@ use openzeppelin_token::erc20::extensions::erc20_votes::Delegation;
 use openzeppelin_token::tests::mocks::erc20_votes_mocks::DualCaseERC20VotesMock::SNIP12MetadataImpl;
 use openzeppelin_token::tests::mocks::erc20_votes_mocks::DualCaseERC20VotesMock;
 use openzeppelin_utils::cryptography::snip12::OffchainMessageHash;
-use openzeppelin_utils::serde::SerializedAppend;
 use openzeppelin_utils::structs::checkpoint::{Checkpoint, TraceTrait};
 use snforge_std::signature::KeyPairTrait;
 use snforge_std::signature::stark_curve::{StarkCurveKeyPairImpl, StarkCurveSignerImpl};
 use snforge_std::{
     cheat_block_timestamp_global, start_cheat_caller_address, spy_events, cheat_chain_id_global,
     test_address
 };
-use snforge_std::{EventSpy, EventSpyAssertionsTrait};
+use snforge_std::{EventSpy};
 use starknet::ContractAddress;
 use starknet::contract_address_const;
 

diff --git a/packages/token/src/tests/erc721/test_dual721.cairo b/packages/token/src/tests/erc721/test_dual721.cairo
@@ -9,7 +9,7 @@ use openzeppelin_token::erc721::interface::{
 };
 use openzeppelin_token::erc721::interface::{IERC721Dispatcher, IERC721DispatcherTrait};
 use openzeppelin_utils::serde::SerializedAppend;
-use snforge_std::{test_address, start_cheat_caller_address};
+use snforge_std::{start_cheat_caller_address};
 use starknet::ContractAddress;
 
 //

diff --git a/packages/token/src/tests/erc721/test_dual721_receiver.cairo b/packages/token/src/tests/erc721/test_dual721_receiver.cairo
@@ -4,19 +4,8 @@ use openzeppelin_token::erc721::dual721_receiver::{
     DualCaseERC721Receiver, DualCaseERC721ReceiverTrait
 };
 use openzeppelin_token::erc721::interface::IERC721_RECEIVER_ID;
-use openzeppelin_token::erc721::interface::{
-    IERC721ReceiverCamelDispatcher, IERC721ReceiverCamelDispatcherTrait
-};
-use openzeppelin_token::erc721::interface::{
-    IERC721ReceiverDispatcher, IERC721ReceiverDispatcherTrait
-};
-use openzeppelin_token::tests::mocks::erc721_receiver_mocks::{
-    CamelERC721ReceiverMock, CamelERC721ReceiverPanicMock
-};
-use openzeppelin_token::tests::mocks::erc721_receiver_mocks::{
-    SnakeERC721ReceiverMock, SnakeERC721ReceiverPanicMock
-};
-use openzeppelin_token::tests::mocks::non_implementing_mock::NonImplementingMock;
+use openzeppelin_token::erc721::interface::{IERC721ReceiverCamelDispatcher};
+use openzeppelin_token::erc721::interface::{IERC721ReceiverDispatcher};
 
 //
 // Setup

diff --git a/packages/upgrades/src/tests/test_upgradeable.cairo b/packages/upgrades/src/tests/test_upgradeable.cairo
@@ -1,8 +1,6 @@
 use openzeppelin_test_common::upgrades::UpgradeableSpyHelpers;
-use openzeppelin_testing::constants::{CLASS_HASH_ZERO, ZERO, FELT_VALUE as VALUE};
-use openzeppelin_testing::events::EventSpyExt;
+use openzeppelin_testing::constants::{CLASS_HASH_ZERO, FELT_VALUE as VALUE};
 use openzeppelin_testing::{declare_class, deploy};
-use openzeppelin_upgrades::UpgradeableComponent;
 use openzeppelin_upgrades::tests::mocks::upgrades_mocks::{
     IUpgradesV1Dispatcher, IUpgradesV1DispatcherTrait
 };

diff --git a/packages/utils/src/tests/test_snip12.cairo b/packages/utils/src/tests/test_snip12.cairo
@@ -1,4 +1,4 @@
-use core::hash::{Hash, HashStateTrait, HashStateExTrait};
+use core::hash::{HashStateTrait, HashStateExTrait};
 use core::poseidon::{PoseidonTrait, poseidon_hash_span};
 use openzeppelin_testing::constants::{OWNER, RECIPIENT};
 use openzeppelin_utils::cryptography::snip12::{