crowetic/protocol
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Transfer ERC721 token instead of burning, so we can still set an allowance to it in tests

Browse Source
This commit is contained in:
Fabio Berger
2018-07-02 10:50:48 +02:00
parent cb6cace910
commit 32d75b6483
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
packages/contracts/src/utils/asset_wrapper.ts
View File

@@ -86,7 +86,15 @@ export class AssetWrapper {
);
} else if (tokenOwner === userAddress && desiredBalance.eq(0)) {
// Burn token
await erc721Wrapper.burnAsync(assetProxyData.tokenAddress, assetProxyData.tokenId, userAddress);
// await erc721Wrapper.burnAsync(assetProxyData.tokenAddress, assetProxyData.tokenId, userAddress);
const userAddresses = await (erc721Wrapper as any)._web3Wrapper.getAvailableAddressesAsync();
const nonOwner = _.find(userAddresses, a => a !== userAddress);
await erc721Wrapper.transferFromAsync(
assetProxyData.tokenAddress,
assetProxyData.tokenId,
tokenOwner,
nonOwner,
);
return;
} else if (
(userAddress !== tokenOwner && desiredBalance.eq(0)) ||