Calling with `sender=ContractInstance` doesn't work

Environment information

• ape and plugin versions:

$ ape --version
0.1.3

$ ape plugins list
Installed Plugins:
  ens          
  hardhat      
  infura       
  vyper        
  etherscan    
  solidity     
  tokens  

• Python Version: 3.9.9
• OS: linux

What went wrong?

When using a network provider that supports transactions without signatures (such as forked mode networks or local test networks), ape doesn’t support calling via a ContractInstance object as sender e.g.:

one_contract.mutableCall(..., sender=another_contract)

How can it be fixed?

The signature check in certain providers checks to see if sender is not of type AddressType before switching to using the signature check, which is missing ContractInstance (this is done because of a circular dependency). Just add the extra support for ContractInstace as well. Could even have it always convert the sender argument to AddressType, if signing is not required on that network.

[fubuloubu]

quick fix to ContractTransaction:

    def _get_type(self, kwarg_name: str) -> Any:
        return TransactionAPI.__fields__[kwarg_name].type_

    def serialize_transaction(self, *args, **kwargs) -> TransactionAPI:
        kwargs = {
            k: self.conversion_manager.convert(
                v,
                # TODO: Upstream, `TransactionAPI.sender` should be `AddressType` (not `str`)
                AddressType if k == "sender" else self._get_type(k),
            )
            for k, v in kwargs.items()
        }
        ...

[sabotagebeats]

https://github.com/ApeWorX/ape-hardhat/issues/38