pytest-ethereum-old icon indicating copy to clipboard operation
pytest-ethereum-old copied to clipboard

Remove hacky deployment kwargs mechanism in contract factory

Open fubuloubu opened this issue 7 years ago • 1 comments

I have this kwarg rewrite hack here, but it's not great: https://github.com/fubuloubu/pytest-ethereum/blob/da7f9f19c10b6bb6bd50a3fc75836672efeccf3f/pytest_ethereum/contract.py#L96-L99

The contract factory does not inherent this easier API from ConciseContract/ImplicitContract, but I was trying to leverage them the same way so you didn't have to context switch the kwargs with deployment factories.

Solution may require an update to web3.py. I mentioned this to @carver in gitter as an inconsistency in the API, not sure what would come of it.

fubuloubu avatar May 21 '18 03:05 fubuloubu

My attention will mostly be on trinity for a while.

carver avatar May 21 '18 19:05 carver