Merge branch 'main' of github.com:weiroll/weiroll.js into main

weiroll · Jun 18, 2021 · 1d3cc59 · 1d3cc59
2 parents d1a0c03 + b01216f
commit 1d3cc59
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@weiroll/weiroll.js",
-  "version": "0.1.1",
+  "version": "0.2.0",
   "description": "The weiroll planner in JS",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

diff --git a/src/planner.ts b/src/planner.ts
@@ -53,6 +53,9 @@ function abiEncodeSingle(param: ParamType, value: any): LiteralValue {
 
 function buildCall(contract: Contract, fragment: FunctionFragment): ContractFunction {
     return function(...args: Array<any>): FunctionCall {
+        if(args.length != fragment.inputs.length) {
+            throw new Error(`Function ${fragment.name} has ${fragment.inputs.length} arguments but ${args.length} provided`);
+        }
         const encodedArgs = args.map((arg, idx) => {
             const param = fragment.inputs[idx];
             if(isReturnValue(arg)) {

diff --git a/tests/test_planner.ts b/tests/test_planner.ts
@@ -148,4 +148,9 @@ describe('Planner', () => {
         expect(state[0]).to.equal(hexDataSlice(defaultAbiCoder.encode(['string'], ["Hello, "]), 32));
         expect(state[1]).to.equal(hexDataSlice(defaultAbiCoder.encode(['string'], ["world!"]), 32));
     });
+
+    it('requires argument counts to match the function definition', () => {
+        const planner = new Planner();
+        expect(() => planner.addCommand(Math.add(1))).to.throw();
+    })
 });