Union types for generalized decodeAssetData

packages/contracts/test/asset_proxy/decoder.ts

@@ -1,6 +1,6 @@
 import { BlockchainLifecycle, devConstants, web3Factory } from '@0xproject/dev-utils';
 import { assetProxyUtils, generatePseudoRandomSalt } from '@0xproject/order-utils';
-import { AssetData, AssetProxyId, ERC20AssetData, ERC721AssetData } from '@0xproject/types';
+import { AssetProxyId, ERC20AssetData, ERC721AssetData } from '@0xproject/types';
 import { BigNumber } from '@0xproject/utils';
 import { Web3Wrapper } from '@0xproject/web3-wrapper';
 import BN = require('bn.js');
@@ -19,7 +19,7 @@ chaiSetup.configure();
 const expect = chai.expect;
 const blockchainLifecycle = new BlockchainLifecycle(web3Wrapper);
 
-describe('TestAssetDataDecoders', () => {
+describe.only('TestAssetDataDecoders', () => {
     let owner: string;
     let testAssetProxyDecoder: TestAssetDataDecodersContract;
     let testAddress: string;

packages/order-utils/src/asset_proxy_utils.ts

@@ -1,4 +1,4 @@
-import { AssetData, AssetProxyId, ERC20AssetData, ERC721AssetData } from '@0xproject/types';
+import { AssetProxyId, ERC20AssetData, ERC721AssetData } from '@0xproject/types';
 import { BigNumber } from '@0xproject/utils';
 import BN = require('bn.js');
 import ethUtil = require('ethereumjs-util');
@@ -157,24 +157,15 @@ export const assetProxyUtils = {
         const assetProxyId = assetProxyUtils.decodeAssetProxyId(encodedAssetProxyId);
         return assetProxyId;
     },
-    decodeAssetData(assetData: string): AssetData {
+    decodeAssetData(assetData: string): ERC20AssetData | ERC721AssetData {
         const assetProxyId = assetProxyUtils.decodeAssetDataId(assetData);
         switch (assetProxyId) {
             case AssetProxyId.ERC20:
                 const erc20AssetData = assetProxyUtils.decodeERC20AssetData(assetData);
-                const generalizedERC20AssetData = {
-                    assetProxyId,
-                    tokenAddress: erc20AssetData.tokenAddress,
-                };
-                return generalizedERC20AssetData;
+                return erc20AssetData;
             case AssetProxyId.ERC721:
                 const erc721AssetData = assetProxyUtils.decodeERC721AssetData(assetData);
-                const generalizedERC721AssetData = {
-                    assetProxyId,
-                    tokenAddress: erc721AssetData.tokenAddress,
-                    data: erc721AssetData.tokenId,
-                };
-                return generalizedERC721AssetData;
+                return erc721AssetData;
             default:
                 throw new Error(`Unrecognized asset proxy id: ${assetProxyId}`);
         }

packages/types/src/index.ts

@@ -167,9 +167,3 @@ export interface ERC721AssetData {
     tokenId: BigNumber;
     receiverData: string;
 }
-
-export interface AssetData {
-    assetProxyId: AssetProxyId;
-    tokenAddress?: string;
-    data?: any;
-}