Added better errors when calling contracts with incorrect number of arguments.

ricmoo · ricmoo · commit 03f8c1db8682 · 2018-04-05T15:47:34.000-04:00
diff --git a/contracts/contract.js b/contracts/contract.js
@@ -76,6 +76,10 @@ function Contract(addressOrName, contractInterface, signerOrProvider) {
                         throw new Error('unknown transaction override ' + key);
                     }
                 }
+            } else if (params.length > method.inputs.types.length) {
+                throw new Error('too many parameters');
+            } else if (params.length < method.inputs.types.length) {
+                throw new Error('too few parameters');
             }
 
             // Check overrides make sense

Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,10 @@ function Contract(addressOrName, contractInterface, signerOrProvider) {`
`76`	`76`	`throw new Error('unknown transaction override ' + key);`
`77`	`77`	`}`
`78`	`78`	`}`
	`79`	`+ } else if (params.length > method.inputs.types.length) {`
	`80`	`+ throw new Error('too many parameters');`
	`81`	`+ } else if (params.length < method.inputs.types.length) {`
	`82`	`+ throw new Error('too few parameters');`
`79`	`83`	`}`
`80`	`84`
`81`	`85`	`// Check overrides make sense`