ContractModuleFactory expects only three arguments, receives four.

[levino]

https://github.com/ethereum/web3.js/blob/441c309d3304ffbc36fca4ef483adf314b5f4801/packages/web3-eth-contract/src/factories/ContractModuleFactory.js#L41-L49

https://github.com/ethereum/web3.js/blob/441c309d3304ffbc36fca4ef483adf314b5f4801/packages/web3-eth-contract/src/index.js#L49

I came across this because my IDE is screaming at me "This makes no sense" (static code analysis):

Should not be very difficult to add some sort of static code analysis for this kind of stuff.

[princesinha19]

@levino Yes, I think accounts is unnecessary parameter over there. we will remove it.