API: call() is broken for view functions with an array as the last parameter

[mikec]

Executing a "call" on a view function in the smart contract is broken for functions with an array as the last input param.

aragon.js/packages/aragon-wrapper/src/core/proxy/index.js

Line 53 in 1dea869

? this.contract.methods[method](...params.slice(0, -1)).call(lastParam)

It's checking to see if the last parameter is an object (which is true if the last param is an array) and it's making the assumption that if it's an object, it's the call options object.

[sohkai]

We should likely use the ABI to check the number of arguments expected, as implemented here: https://github.com/aragon/aragon.js/blob/master/packages/aragon-wrapper/src/index.js#L1110