2 files changed, 13 insertions, 9 deletions
diff --git a/packages/order-utils/test/abi/data_type.ts b/packages/order-utils/test/abi/data_type.ts
index 201eb89f9..c1e352508 100644
--- a/packages/order-utils/test/abi/data_type.ts
+++ b/packages/order-utils/test/abi/data_type.ts
@@ -95,15 +95,17 @@ export abstract class MemberDataType extends DataType {
     private members: DataType[];
     private isArray: boolean;
     protected arrayLength: number | undefined;
+    protected arrayElementType: string | undefined;
 
 
-    public constructor(dataItem: DataItem, isArray: boolean = false, arrayLength?: number) {
+    public constructor(dataItem: DataItem, isArray: boolean = false, arrayLength?: number, arrayElementType?: string) {
         super(dataItem);
 
         this.memberMap = {};
         this.members = [];
         this.isArray = isArray;
         this.arrayLength = arrayLength;
+        this.arrayElementType = arrayElementType;
         if (isArray && arrayLength !== undefined) {
             [this.members, this.memberMap] = this.createMembersWithLength(dataItem, arrayLength);
         } else if (!isArray) {
@@ -119,10 +121,10 @@ export abstract class MemberDataType extends DataType {
 
         let members: DataType[] = [];
         let memberMap: MemberMap = {};
-        _.each(dataItem.components, (dataItem: DataItem) => {
+        _.each(dataItem.components, (memberItem: DataItem) => {
             const childDataItem = {
-                type: dataItem.type,
-                name: `${dataItem.name}.${dataItem.name}`,
+                type: memberItem.type,
+                name: `${dataItem.name}.${memberItem.name}`,
             } as DataItem;
             const child = DataTypeFactory.create(childDataItem, this);
             members.push(child);
@@ -133,12 +135,13 @@ export abstract class MemberDataType extends DataType {
     }
 
     private createMembersWithLength(dataItem: DataItem, length: number): [DataType[], MemberMap] {
+        console.log(dataItem);
         let members: DataType[] = [];
         let memberMap: MemberMap = {};
         const range = _.range(length);
         _.each(range, (idx: number) => {
             const childDataItem = {
-                type: dataItem.type,
+                type: this.arrayElementType,
                 name: `${dataItem.name}[${idx.toString(10)}]`,
             } as DataItem;
             const components = dataItem.components;
@@ -164,7 +167,7 @@ export abstract class MemberDataType extends DataType {
         }
 
         let members = this.members;
-        if (this.arrayLength === undefined) {
+        if (this.isArray && this.arrayLength === undefined) {
             [members,] = this.createMembersWithLength(this.getDataItem(), value.length);
         }
 
diff --git a/packages/order-utils/test/abi/evm_data_types.ts b/packages/order-utils/test/abi/evm_data_types.ts
index 4dffedb8d..1bd4f0d51 100644
--- a/packages/order-utils/test/abi/evm_data_types.ts
+++ b/packages/order-utils/test/abi/evm_data_types.ts
@@ -347,9 +347,10 @@ export class SolArray extends MemberDataType {
         }
 
         const isArray = true;
+        const arrayElementType = matches[1];
         const arrayLength = (matches[2] === '') ? undefined : parseInt(matches[2], 10);
-        super(dataItem, isArray, arrayLength);
-        this.elementType = matches[1];
+        super(dataItem, isArray, arrayLength, arrayElementType);
+        this.elementType = arrayElementType;
         this.arraySignature = this.computeSignature();
     }
 
@@ -388,7 +389,7 @@ export class Method extends MemberDataType {
     public selector: string;
 
     constructor(abi: MethodAbi) {
-        super({ type: 'method', name: abi.name });
+        super({ type: 'method', name: abi.name, components: abi.inputs });
         this.methodSignature = this.computeSignature();
         this.selector = this.methodSelector = this.computeSelector();