test: add unit and e2e tests

Signed-off-by: dosi <dosi.kolev@limechain.tech>

Author: Dosik13

tests/integration/account_allowance_e2e_test.py

@@ -15,7 +15,9 @@
 from hiero_sdk_python.hbar import Hbar
 from hiero_sdk_python.response_code import ResponseCode
 from hiero_sdk_python.tokens.nft_id import NftId
-from hiero_sdk_python.tokens.token_associate_transaction import TokenAssociateTransaction
+from hiero_sdk_python.tokens.token_associate_transaction import (
+    TokenAssociateTransaction,
+)
 from hiero_sdk_python.tokens.token_mint_transaction import TokenMintTransaction
 from hiero_sdk_python.transaction.transaction_id import TransactionId
 from hiero_sdk_python.transaction.transfer_transaction import TransferTransaction
@@ -61,7 +63,9 @@ def _mint_nft(env, token_id, metadata):
 
 
 @pytest.mark.integration
-def test_integration_cannot_transfer_on_behalf_of_spender_without_allowance_approval(env):
+def test_integration_cannot_transfer_on_behalf_of_spender_without_allowance_approval(
+    env,
+):
     """Test that a spender cannot transfer NFTs on behalf of account without allowance approval."""
     spender_account, receiver_account = _create_spender_and_receiver_accounts(env)
 
@@ -264,7 +268,9 @@ def test_integration_fungible_token_allowance(env):
 
 
 @pytest.mark.integration
-def test_integration_cant_transfer_on_behalf_of_spender_after_removing_the_allowance_approval(env):
+def test_integration_cant_transfer_on_behalf_of_spender_after_removing_the_allowance_approval(
+    env,
+):
     """Test that a spender cannot transfer NFTs after the allowance approval is removed."""
     spender_account, receiver_account = _create_spender_and_receiver_accounts(env)
 
@@ -486,3 +492,58 @@ def test_integration_cannot_send_deleted_token_nft_serials(env):
         f"Transfer should have failed with SPENDER_DOES_NOT_HAVE_ALLOWANCE"
         f"status but got: {ResponseCode(transfer_receipt.status).name}"
     )
+
+
+@pytest.mark.integration
+def test_integration_can_approve_serial_and_delete_all_serials_in_one_transaction(env):
+    """Test that approve-serial + delete-all-serials in one transaction preserves the per-serial allowance."""
+    spender_account, receiver_account = _create_spender_and_receiver_accounts(env)
+
+    token_id = create_nft_token(env)
+    assert token_id is not None
+
+    _associate_token_with_account(env, receiver_account, token_id)
+
+    nft_ids = _mint_nft(env, token_id, [b"\x01", b"\x02"])
+    nft1 = nft_ids[0]
+    nft2 = nft_ids[1]
+
+    # Approve nft1 specifically and delete-all-serials for the same (token, spender)
+    # pair in the same transaction. The per-serial approval is preserved because the
+    # two operations produce independent NftAllowance entries.
+    receipt = (
+        AccountAllowanceApproveTransaction()
+        .approve_token_nft_allowance(nft1, env.operator_id, spender_account.id)
+        .delete_token_nft_allowance_all_serials(token_id, env.operator_id, spender_account.id)
+        .execute(env.client)
+    )
+    assert receipt.status == ResponseCode.SUCCESS, (
+        f"Allowance approval failed with status: {ResponseCode(receipt.status).name}"
+    )
+
+    # Transfer nft1 (should succeed - per-serial allowance preserved)
+    transfer_receipt = (
+        TransferTransaction()
+        .set_transaction_id(TransactionId.generate(spender_account.id))
+        .add_approved_nft_transfer(nft1, env.operator_id, receiver_account.id)
+        .freeze_with(env.client)
+        .sign(spender_account.key)
+        .execute(env.client)
+    )
+    assert transfer_receipt.status == ResponseCode.SUCCESS, (
+        f"Transfer failed with status: {ResponseCode(transfer_receipt.status).name}"
+    )
+
+    # Transfer nft2 (should fail - delete-all-serials revoked any blanket allowance)
+    transfer_receipt2 = (
+        TransferTransaction()
+        .set_transaction_id(TransactionId.generate(spender_account.id))
+        .add_approved_nft_transfer(nft2, env.operator_id, receiver_account.id)
+        .freeze_with(env.client)
+        .sign(spender_account.key)
+        .execute(env.client)
+    )
+    assert transfer_receipt2.status == ResponseCode.SPENDER_DOES_NOT_HAVE_ALLOWANCE, (
+        f"Transfer should have failed with SPENDER_DOES_NOT_HAVE_ALLOWANCE"
+        f"status but got: {ResponseCode(transfer_receipt2.status).name}"
+    )

tests/unit/account_allowance_approve_transaction_test.py

@@ -217,6 +217,55 @@ def test_delete_token_nft_allowance_all_serials(account_allowance_transaction, s
     assert allowance.approved_for_all is False
 
 
+def test_delete_token_nft_allowance_all_serials_preserves_prior_serial_approval(
+    account_allowance_transaction, sample_accounts, sample_tokens
+):
+    """Test that delete-all-serials does not fold into a prior per-serial approval."""
+    token_id = sample_tokens["token1"]
+    owner = sample_accounts["owner"]
+    spender = sample_accounts["spender"]
+    nft_id = NftId(token_id, 1)
+
+    account_allowance_transaction.approve_token_nft_allowance(nft_id, owner, spender)
+    account_allowance_transaction.delete_token_nft_allowance_all_serials(token_id, owner, spender)
+
+    assert len(account_allowance_transaction.nft_allowances) == 2
+
+    first = account_allowance_transaction.nft_allowances[0]
+    assert first.token_id == token_id
+    assert first.owner_account_id == owner
+    assert first.spender_account_id == spender
+    assert first.serial_numbers == [1]
+    assert first.approved_for_all is False
+
+    second = account_allowance_transaction.nft_allowances[1]
+    assert second.token_id == token_id
+    assert second.owner_account_id == owner
+    assert second.spender_account_id == spender
+    assert second.serial_numbers == []
+    assert second.approved_for_all is False
+
+
+def test_delete_token_nft_allowance_all_serials_appends_per_call(
+    account_allowance_transaction, sample_accounts, sample_tokens
+):
+    """Test that repeated delete-all-serials calls append one entry per call."""
+    token_id = sample_tokens["token1"]
+    owner = sample_accounts["owner"]
+    spender = sample_accounts["spender"]
+
+    account_allowance_transaction.delete_token_nft_allowance_all_serials(token_id, owner, spender)
+    account_allowance_transaction.delete_token_nft_allowance_all_serials(token_id, owner, spender)
+
+    assert len(account_allowance_transaction.nft_allowances) == 2
+    for allowance in account_allowance_transaction.nft_allowances:
+        assert allowance.token_id == token_id
+        assert allowance.owner_account_id == owner
+        assert allowance.spender_account_id == spender
+        assert allowance.serial_numbers == []
+        assert allowance.approved_for_all is False
+
+
 def test_add_all_token_nft_approval(account_allowance_transaction, sample_accounts, sample_tokens):
     """Test adding all token NFT approval"""
     token_id = sample_tokens["token1"]