From d72a7318658e3f6f860db7e9aa6dc9059451d7b0 Mon Sep 17 00:00:00 2001 From: Greg Hysen Date: Tue, 10 Apr 2018 22:24:21 -0700 Subject: setAssetProxy to addAssetProxy --- .../AssetProxyDispatcher/AssetProxyDispatcher.sol | 2 +- .../AssetProxyDispatcher/IAssetProxyDispatcher.sol | 2 +- .../test/asset_proxy_dispatcher/dispatcher.ts | 28 +++++++++++----------- packages/contracts/test/exchange/core.ts | 6 ++--- packages/contracts/test/exchange/helpers.ts | 4 ++-- packages/contracts/test/exchange/wrapper.ts | 4 ++-- 6 files changed, 23 insertions(+), 23 deletions(-) diff --git a/packages/contracts/src/contracts/current/protocol/AssetProxyDispatcher/AssetProxyDispatcher.sol b/packages/contracts/src/contracts/current/protocol/AssetProxyDispatcher/AssetProxyDispatcher.sol index cce330818..e22b88b34 100644 --- a/packages/contracts/src/contracts/current/protocol/AssetProxyDispatcher/AssetProxyDispatcher.sol +++ b/packages/contracts/src/contracts/current/protocol/AssetProxyDispatcher/AssetProxyDispatcher.sol @@ -58,7 +58,7 @@ contract AssetProxyDispatcher is /// @param assetProxyId Id of the asset proxy. /// @param newAssetProxyAddress Address of the asset proxy contract to register. /// @param currentAssetProxyAddress Address of existing asset proxy to overwrite. - function setAssetProxy( + function addAssetProxy( uint8 assetProxyId, address newAssetProxyAddress, address currentAssetProxyAddress) diff --git a/packages/contracts/src/contracts/current/protocol/AssetProxyDispatcher/IAssetProxyDispatcher.sol b/packages/contracts/src/contracts/current/protocol/AssetProxyDispatcher/IAssetProxyDispatcher.sol index 9fe7b49a3..e86e99ab9 100644 --- a/packages/contracts/src/contracts/current/protocol/AssetProxyDispatcher/IAssetProxyDispatcher.sol +++ b/packages/contracts/src/contracts/current/protocol/AssetProxyDispatcher/IAssetProxyDispatcher.sol @@ -36,7 +36,7 @@ contract IAssetProxyDispatcher is /// @param assetProxyId Id of the asset proxy. /// @param newAssetProxyAddress Address of the asset proxy contract to register. /// @param currentAssetProxyAddress Address of existing asset proxy to overwrite. - function setAssetProxy( + function addAssetProxy( uint8 assetProxyId, address newAssetProxyAddress, address currentAssetProxyAddress) diff --git a/packages/contracts/test/asset_proxy_dispatcher/dispatcher.ts b/packages/contracts/test/asset_proxy_dispatcher/dispatcher.ts index f7235ae47..6df4e0eab 100644 --- a/packages/contracts/test/asset_proxy_dispatcher/dispatcher.ts +++ b/packages/contracts/test/asset_proxy_dispatcher/dispatcher.ts @@ -124,9 +124,9 @@ describe('AssetProxyDispatcher', () => { await blockchainLifecycle.revertAsync(); }); - describe('setAssetProxy', () => { + describe('addAssetProxy', () => { it('should record proxy upon registration', async () => { - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, @@ -137,7 +137,7 @@ describe('AssetProxyDispatcher', () => { }); it('should be able to record multiple proxies', async () => { - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, @@ -146,7 +146,7 @@ describe('AssetProxyDispatcher', () => { let proxyAddress = await assetProxyDispatcher.getAssetProxy.callAsync(AssetProxyId.ERC20); expect(proxyAddress).to.be.equal(erc20TransferProxy.address); - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC721, erc721TransferProxy.address, nilAddress, @@ -157,7 +157,7 @@ describe('AssetProxyDispatcher', () => { }); it('should replace proxy address upon re-registration', async () => { - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, @@ -176,7 +176,7 @@ describe('AssetProxyDispatcher', () => { const newAddress = newErc20TransferProxy.address; const currentAddress = erc20TransferProxy.address; - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, newAddress, currentAddress, @@ -187,7 +187,7 @@ describe('AssetProxyDispatcher', () => { }); it('should throw if registering with incorrect "old_address" field', async () => { - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, @@ -198,7 +198,7 @@ describe('AssetProxyDispatcher', () => { // The following transaction will throw because the currentAddress is no longer nilAddress return expect( - assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, @@ -208,7 +208,7 @@ describe('AssetProxyDispatcher', () => { }); it('should be able to reset proxy address to NULL', async () => { - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, @@ -218,7 +218,7 @@ describe('AssetProxyDispatcher', () => { expect(proxyAddress).to.be.equal(erc20TransferProxy.address); // The following transaction will reset the proxy address - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, nilAddress, erc20TransferProxy.address, @@ -230,7 +230,7 @@ describe('AssetProxyDispatcher', () => { it('should throw if requesting address is not authorized', async () => { return expect( - assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, @@ -242,7 +242,7 @@ describe('AssetProxyDispatcher', () => { describe('getAssetProxy', () => { it('should return correct address of registered proxy', async () => { - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, @@ -261,7 +261,7 @@ describe('AssetProxyDispatcher', () => { describe('transferFrom', () => { it('should dispatch transfer to registered proxy', async () => { // Register ERC20 proxy - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, @@ -312,7 +312,7 @@ describe('AssetProxyDispatcher', () => { it('should throw if requesting address is not authorized', async () => { // Register ERC20 proxy - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, diff --git a/packages/contracts/test/exchange/core.ts b/packages/contracts/test/exchange/core.ts index f0856eac3..49a05fd5b 100644 --- a/packages/contracts/test/exchange/core.ts +++ b/packages/contracts/test/exchange/core.ts @@ -154,19 +154,19 @@ describe('Exchange', () => { from: accounts[0], }); const nilAddress = '0x0000000000000000000000000000000000000000'; - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20_V1, erc20TransferProxyV1.address, nilAddress, { from: accounts[0] }, ); - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, { from: accounts[0] }, ); - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC721, erc721TransferProxy.address, nilAddress, diff --git a/packages/contracts/test/exchange/helpers.ts b/packages/contracts/test/exchange/helpers.ts index 32e06e988..97421e644 100644 --- a/packages/contracts/test/exchange/helpers.ts +++ b/packages/contracts/test/exchange/helpers.ts @@ -69,13 +69,13 @@ describe('Exchange', () => { from: accounts[0], }); const nilAddress = '0x0000000000000000000000000000000000000000'; - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20_V1, erc20TransferProxyV1.address, nilAddress, { from: accounts[0] }, ); - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, diff --git a/packages/contracts/test/exchange/wrapper.ts b/packages/contracts/test/exchange/wrapper.ts index 28547034e..f993eb2d3 100644 --- a/packages/contracts/test/exchange/wrapper.ts +++ b/packages/contracts/test/exchange/wrapper.ts @@ -119,13 +119,13 @@ describe('Exchange', () => { from: accounts[0], }); const nilAddress = '0x0000000000000000000000000000000000000000'; - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20_V1, erc20TransferProxyV1.address, nilAddress, { from: accounts[0] }, ); - await assetProxyDispatcher.setAssetProxy.sendTransactionAsync( + await assetProxyDispatcher.addAssetProxy.sendTransactionAsync( AssetProxyId.ERC20, erc20TransferProxy.address, nilAddress, -- cgit v1.2.3