Remove skipping of certain tests (#1470)

* Remove skip 'test_get_declare_transaction'

* Remove skip 'test_get_invoke_transaction'

* Remove skip 'test_get_deploy_account_transaction'

* Remove skip 'test_get_block_by_hash'

* Remove skip 'test_multicall'

* Remove skipping of several tests

* Remove skipping of several tests

* Remove skipping of several tests

* Remove skipping of several tests

* Remove skipping of several tests

* Remove ledger from CI

* Update 'max_fee' in 'test_declare_v2'

* Skip ledger tests

* Update 'max_fee' in 'test_declaring_contracts'

* Add copy of map contract

* Add copy of account contract

* Add second copy of map contract

* Fix 'lib.cairo'

* Fix import

* Fix import

* Fix 'test_compute_sierra_class_hash'

* Revert "Remove ledger from CI"

This reverts commit a2efffa.

* Revert "Skip ledger tests"

This reverts commit 376e6fe.

* Rename 'account_copy.cario' -> 'account_copy1.cairo'

* Fix imports in 'lib.cairo'

* Revert "Fix 'test_compute_sierra_class_hash'"

This reverts commit 82a21e4.

* Reapply "Remove ledger from CI"

This reverts commit ec492e6.

* Reapply "Skip ledger tests"

This reverts commit dd560e7.

* Revert "Reapply "Skip ledger tests""

This reverts commit c88cb42.

* Revert "Reapply "Remove ledger from CI""

This reverts commit 203de83.

* Remove unnecessary line

* Rename copy fizture names

* Fix lint

* Rename contracts

Author: kkawula

starknet_py/tests/e2e/block_test.py

@@ -34,8 +34,6 @@ async def test_block_with_tx_hashes_pending(account):
     assert isinstance(blk.transactions, list)
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_block_with_tx_hashes_latest(account):
     blk = await account.client.get_block_with_tx_hashes(block_number="latest")
@@ -64,7 +62,6 @@ async def test_get_block_with_txs_pending(account):
     assert isinstance(blk.transactions, list)
 
 
-# TODO (#1419): Fix contract redeclaration
 @pytest.mark.asyncio
 async def test_get_block_with_txs_latest(account, map_class_hash):
     # pylint: disable=unused-argument

starknet_py/tests/e2e/client/client_test.py

@@ -41,8 +41,6 @@
 )
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_get_declare_transaction(
     client, declare_transaction_hash, class_hash, account
@@ -55,8 +53,6 @@ async def test_get_declare_transaction(
     assert transaction.sender_address == account.address
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_get_invoke_transaction(
     client,
@@ -69,8 +65,6 @@ async def test_get_invoke_transaction(
     assert transaction.hash == invoke_transaction_hash
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_get_deploy_account_transaction(client, deploy_account_transaction_hash):
     transaction = await client.get_transaction(deploy_account_transaction_hash)
@@ -89,8 +83,6 @@ async def test_get_transaction_raises_on_not_received(client):
         await client.get_transaction(tx_hash=0x9999)
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_get_block_by_hash(
     client,
@@ -104,8 +96,6 @@ async def test_get_block_by_hash(
     assert len(block.transactions) != 0
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_get_block_by_number(
     client,
@@ -119,8 +109,6 @@ async def test_get_block_by_number(
     assert len(block.transactions) != 0
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_get_storage_at(client, contract_address):
     storage = await client.get_storage_at(
@@ -132,8 +120,6 @@ async def test_get_storage_at(client, contract_address):
     assert storage == 1897
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_get_transaction_receipt(
     client, invoke_transaction_hash, block_with_invoke_number
@@ -145,8 +131,6 @@ async def test_get_transaction_receipt(
     assert receipt.type == TransactionType.INVOKE
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_estimate_fee_invoke(account, contract_address):
     invoke_tx = await account.sign_invoke_v1(
@@ -169,8 +153,6 @@ async def test_estimate_fee_invoke(account, contract_address):
     assert estimate_fee.data_gas_consumed > 0
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_estimate_fee_invoke_v3(account, contract_address):
     invoke_tx = await account.sign_invoke_v3(
@@ -228,8 +210,6 @@ async def test_estimate_fee_deploy_account(client, deploy_account_transaction):
     assert estimate_fee.data_gas_consumed > 0
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_estimate_fee_for_multiple_transactions(
     client, deploy_account_transaction, contract_address, account
@@ -260,8 +240,6 @@ async def test_estimate_fee_for_multiple_transactions(
         assert estimated_fee.data_gas_consumed > 0
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_call_contract(client, contract_address):
     call = Call(
@@ -292,8 +270,6 @@ async def test_add_transaction(map_contract, client, account):
     assert transaction_receipt.type == TransactionType.INVOKE
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_get_class_hash_at(client, contract_address, class_hash):
     received_class_hash = await client.get_class_hash_at(
@@ -302,8 +278,6 @@ async def test_get_class_hash_at(client, contract_address, class_hash):
     assert received_class_hash == class_hash
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_get_class_by_hash(client, class_hash):
     contract_class = await client.get_class_by_hash(class_hash=class_hash)
@@ -501,8 +475,6 @@ async def test_state_update_storage_diffs(
     assert isinstance(state_update, BlockStateUpdate)
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.run_on_devnet
 @pytest.mark.asyncio
 async def test_state_update_deployed_contracts(

starknet_py/tests/e2e/client/full_node_test.py

@@ -40,8 +40,6 @@ def _parse_event_name(event: str) -> str:
 EVENT_TWO_PARSED_NAME = _parse_event_name("AnotherPutCalled")
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.run_on_devnet
 @pytest.mark.asyncio
 async def test_node_get_declare_transaction_by_block_number_and_index(
@@ -57,8 +55,6 @@ async def test_node_get_declare_transaction_by_block_number_and_index(
     assert tx.version == 2
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.run_on_devnet
 @pytest.mark.asyncio
 async def test_get_class_at(
@@ -496,8 +492,6 @@ async def test_simulate_transactions_invoke(account, deployed_balance_contract):
     assert simulated_txs[0].transaction_trace.execution_resources is not None
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_simulate_transactions_two_txs(account, deployed_balance_contract):
     assert isinstance(deployed_balance_contract, Contract)

starknet_py/tests/e2e/docs/code_examples/test_contract.py

@@ -53,8 +53,6 @@ async def test_from_address(account, contract_address):
     # docs-end: from_address
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_declare_v2(account):
     compiled_contract = load_contract(
@@ -66,7 +64,7 @@ async def test_declare_v2(account):
         account,
         compiled_contract=compiled_contract["sierra"],
         compiled_contract_casm=compiled_contract["casm"],
-        max_fee=int(1e15),
+        max_fee=int(1e18),
     )
     # docs-end: declare_v2
     await declare_result.wait_for_acceptance()

starknet_py/tests/e2e/docs/code_examples/test_full_node_client.py

@@ -233,8 +233,6 @@ async def test_trace_transaction(client):
     # docs-end: trace_transaction
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_simulate_transactions(
     account, deployed_balance_contract, deploy_account_transaction

starknet_py/tests/e2e/docs/guide/test_declaring_contracts.py

@@ -1,13 +1,19 @@
+import sys
+
 import pytest
 
 from starknet_py.net.client_models import ResourceBounds
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
+@pytest.mark.skipif(
+    "--contract_dir=v2" not in sys.argv,
+    reason="Contract exists only in v2 directory",
+)
 @pytest.mark.asyncio
-async def test_declaring_contracts(account, map_compiled_contract_and_class_hash):
-    (compiled_contract, class_hash) = map_compiled_contract_and_class_hash
+async def test_declaring_contracts(
+    account, map_compiled_contract_and_class_hash_copy_1
+):
+    (compiled_contract, class_hash) = map_compiled_contract_and_class_hash_copy_1
 
     # docs: start
     # Account.sign_declare_v2 and Account.sign_declare_v3 take string containing a compiled contract (sierra)
@@ -18,7 +24,7 @@ async def test_declaring_contracts(account, map_compiled_contract_and_class_hash
         compiled_contract=compiled_contract,
         compiled_class_hash=class_hash,
         l1_resource_bounds=ResourceBounds(
-            max_amount=5000, max_price_per_unit=int(1e12)
+            max_amount=int(1e5), max_price_per_unit=int(1e13)
         ),
     )
 

starknet_py/tests/e2e/docs/guide/test_multicall.py

@@ -1,8 +1,6 @@
 import pytest
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
 @pytest.mark.asyncio
 async def test_multicall(account, deployed_balance_contract):
     # pylint: disable=import-outside-toplevel

starknet_py/tests/e2e/docs/guide/test_simple_declare_and_deploy_cairo1.py

@@ -1,18 +1,22 @@
+import sys
+
 import pytest
 
 from starknet_py.tests.e2e.fixtures.misc import load_contract
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
+@pytest.mark.skipif(
+    "--contract_dir=v2" not in sys.argv,
+    reason="Contract exists only in v2 directory",
+)
 @pytest.mark.asyncio
 async def test_simple_declare_and_deploy(account):
     # pylint: disable=import-outside-toplevel
     # docs: start
     from starknet_py.contract import Contract
 
     # docs: end
-    compiled_contract = load_contract("Account")
+    compiled_contract = load_contract("AccountCopy1")
     constructor_args = {"public_key": 0x123}
 
     # docs: start

starknet_py/tests/e2e/docs/quickstart/test_using_account.py

@@ -1,15 +1,18 @@
 import os
+import sys
 
 import pytest
 
 directory = os.path.dirname(__file__)
 
 
-# TODO (#1419): Fix contract redeclaration
-@pytest.mark.skip(reason="Redeclaration occurred")
+@pytest.mark.skipif(
+    "--contract_dir=v2" not in sys.argv,
+    reason="Contract exists only in v2 directory",
+)
 @pytest.mark.asyncio
-async def test_using_account(account, map_compiled_contract_and_class_hash):
-    (compiled_contract, class_hash) = map_compiled_contract_and_class_hash
+async def test_using_account(account, map_compiled_contract_and_class_hash_copy_2):
+    (compiled_contract, class_hash) = map_compiled_contract_and_class_hash_copy_2
     # pylint: disable=import-outside-toplevel, duplicate-code, too-many-locals
     # docs: start
     from starknet_py.contract import Contract

starknet_py/tests/e2e/fixtures/contracts_v1.py

@@ -240,6 +240,26 @@ def map_compiled_contract_and_class_hash() -> Tuple[str, int]:
     )
 
 
+@pytest.fixture(scope="package")
+def map_compiled_contract_and_class_hash_copy_1() -> Tuple[str, int]:
+    contract = load_contract("MapCopy1")
+
+    return (
+        contract["sierra"],
+        compute_casm_class_hash(create_casm_class(contract["casm"])),
+    )
+
+
+@pytest.fixture(scope="package")
+def map_compiled_contract_and_class_hash_copy_2() -> Tuple[str, int]:
+    contract = load_contract("MapCopy2")
+
+    return (
+        contract["sierra"],
+        compute_casm_class_hash(create_casm_class(contract["casm"])),
+    )
+
+
 @pytest.fixture(scope="package")
 def map_compiled_contract_casm() -> str:
     contract = load_contract("Map")