From 8bce73dc679cb05353835d011b8e1dfcff7f4670 Mon Sep 17 00:00:00 2001 From: Amir Bandeali Date: Wed, 15 Aug 2018 10:55:48 -0700 Subject: Fix typos, run prettier, and allow anyone to call on DummyERC721Token --- .../src/2.0.0/test/DummyERC20Token/DummyERC20Token.sol | 3 ++- .../src/2.0.0/test/DummyERC721Token/DummyERC721Token.sol | 3 +-- .../contracts/src/2.0.0/tokens/ERC20Token/ERC20Token.sol | 2 +- .../src/2.0.0/tokens/ERC20Token/IERC20Token.sol | 2 +- .../src/2.0.0/tokens/ERC721Token/ERC721Token.sol | 2 +- packages/contracts/test/tokens/erc721_token.ts | 16 ++++++++-------- packages/types/CHANGELOG.json | 2 +- packages/utils/CHANGELOG.json | 3 ++- 8 files changed, 17 insertions(+), 16 deletions(-) diff --git a/packages/contracts/src/2.0.0/test/DummyERC20Token/DummyERC20Token.sol b/packages/contracts/src/2.0.0/test/DummyERC20Token/DummyERC20Token.sol index c6797c0d8..412c5d1ad 100644 --- a/packages/contracts/src/2.0.0/test/DummyERC20Token/DummyERC20Token.sol +++ b/packages/contracts/src/2.0.0/test/DummyERC20Token/DummyERC20Token.sol @@ -29,6 +29,7 @@ contract DummyERC20Token is string public name; string public symbol; uint256 public decimals; + uint256 public constant MAX_MINT_AMOUNT = 10000000000000000000000; constructor ( string _name, @@ -67,7 +68,7 @@ contract DummyERC20Token is external { require( - _value <= 10000000000000000000000, + _value <= MAX_MINT_AMOUNT, "VALUE_TOO_LARGE" ); diff --git a/packages/contracts/src/2.0.0/test/DummyERC721Token/DummyERC721Token.sol b/packages/contracts/src/2.0.0/test/DummyERC721Token/DummyERC721Token.sol index e995cc967..ac9068d1d 100644 --- a/packages/contracts/src/2.0.0/test/DummyERC721Token/DummyERC721Token.sol +++ b/packages/contracts/src/2.0.0/test/DummyERC721Token/DummyERC721Token.sol @@ -46,13 +46,12 @@ contract DummyERC721Token is /// @param _tokenId ID of the token to be minted by the msg.sender function mint(address _to, uint256 _tokenId) external - onlyOwner { _mint(_to, _tokenId); } /// @dev Function to burn a token - /// Reverts if the given token ID doesn't exist + /// Reverts if the given token ID doesn't exist or not called by contract owner /// @param _owner Owner of token with given token ID /// @param _tokenId ID of the token to be burned by the msg.sender function burn(address _owner, uint256 _tokenId) diff --git a/packages/contracts/src/2.0.0/tokens/ERC20Token/ERC20Token.sol b/packages/contracts/src/2.0.0/tokens/ERC20Token/ERC20Token.sol index db2d09b9d..5ef5ee7ce 100644 --- a/packages/contracts/src/2.0.0/tokens/ERC20Token/ERC20Token.sol +++ b/packages/contracts/src/2.0.0/tokens/ERC20Token/ERC20Token.sol @@ -101,7 +101,7 @@ contract ERC20Token is /// @dev `msg.sender` approves `_spender` to spend `_value` tokens /// @param _spender The address of the account able to transfer the tokens /// @param _value The amount of wei to be approved for transfer - /// @return Always true if enough call has enough gas to complete execution + /// @return Always true if the call has enough gas to complete execution function approve(address _spender, uint256 _value) external returns (bool) diff --git a/packages/contracts/src/2.0.0/tokens/ERC20Token/IERC20Token.sol b/packages/contracts/src/2.0.0/tokens/ERC20Token/IERC20Token.sol index 462be17fd..258d47393 100644 --- a/packages/contracts/src/2.0.0/tokens/ERC20Token/IERC20Token.sol +++ b/packages/contracts/src/2.0.0/tokens/ERC20Token/IERC20Token.sol @@ -58,7 +58,7 @@ contract IERC20Token { /// @dev `msg.sender` approves `_spender` to spend `_value` tokens /// @param _spender The address of the account able to transfer the tokens /// @param _value The amount of wei to be approved for transfer - /// @return Always true if enough call has enough gas to complete execution + /// @return Always true if the call has enough gas to complete execution function approve(address _spender, uint256 _value) external returns (bool); diff --git a/packages/contracts/src/2.0.0/tokens/ERC721Token/ERC721Token.sol b/packages/contracts/src/2.0.0/tokens/ERC721Token/ERC721Token.sol index 94e084731..530f080c0 100644 --- a/packages/contracts/src/2.0.0/tokens/ERC721Token/ERC721Token.sol +++ b/packages/contracts/src/2.0.0/tokens/ERC721Token/ERC721Token.sol @@ -45,7 +45,7 @@ contract ERC721Token is /// @notice Transfers the ownership of an NFT from one address to another address /// @dev Throws unless `msg.sender` is the current owner, an authorized - /// perator, or the approved address for this NFT. Throws if `_from` is + /// operator, or the approved address for this NFT. Throws if `_from` is /// not the current owner. Throws if `_to` is the zero address. Throws if /// `_tokenId` is not a valid NFT. When transfer is complete, this function /// checks if `_to` is a smart contract (code size > 0). If so, it calls diff --git a/packages/contracts/test/tokens/erc721_token.ts b/packages/contracts/test/tokens/erc721_token.ts index bdf4a28a4..e61fd7d51 100644 --- a/packages/contracts/test/tokens/erc721_token.ts +++ b/packages/contracts/test/tokens/erc721_token.ts @@ -4,15 +4,15 @@ import { BigNumber } from '@0xproject/utils'; import * as chai from 'chai'; import { LogWithDecodedArgs } from 'ethereum-types'; -import { DummyERC721ReceiverContract } from '../../generated_contract_wrappers/dummy_erc721_receiver'; +import { + DummyERC721ReceiverContract, + DummyERC721ReceiverTokenReceivedEventArgs, +} from '../../generated_contract_wrappers/dummy_erc721_receiver'; import { DummyERC721TokenContract, DummyERC721TokenTransferEventArgs, } from '../../generated_contract_wrappers/dummy_erc721_token'; -import { - InvalidERC721ReceiverContract, - InvalidERC721ReceiverTokenReceivedEventArgs, -} from '../../generated_contract_wrappers/invalid_erc721_receiver'; +import { InvalidERC721ReceiverContract } from '../../generated_contract_wrappers/invalid_erc721_receiver'; import { artifacts } from '../utils/artifacts'; import { expectTransactionFailedAsync, expectTransactionFailedWithoutReasonAsync } from '../utils/assertions'; import { chaiSetup } from '../utils/chai_setup'; @@ -132,7 +132,7 @@ describe('ERC721Token', () => { expect(log.args._to).to.be.equal(to); expect(log.args._tokenId).to.be.bignumber.equal(tokenId); }); - it('should transfer the token is spender is individually approved', async () => { + it('should transfer the token if spender is individually approved', async () => { await web3Wrapper.awaitTransactionSuccessAsync( await token.approve.sendTransactionAsync(spender, tokenId), constants.AWAIT_TRANSACTION_MINED_MS, @@ -204,7 +204,7 @@ describe('ERC721Token', () => { const newOwner = await token.ownerOf.callAsync(tokenId); expect(newOwner).to.be.equal(to); const transferLog = txReceipt.logs[0] as LogWithDecodedArgs; - const receiverLog = txReceipt.logs[1] as LogWithDecodedArgs; + const receiverLog = txReceipt.logs[1] as LogWithDecodedArgs; expect(transferLog.args._from).to.be.equal(from); expect(transferLog.args._to).to.be.equal(to); expect(transferLog.args._tokenId).to.be.bignumber.equal(tokenId); @@ -265,7 +265,7 @@ describe('ERC721Token', () => { const newOwner = await token.ownerOf.callAsync(tokenId); expect(newOwner).to.be.equal(to); const transferLog = txReceipt.logs[0] as LogWithDecodedArgs; - const receiverLog = txReceipt.logs[1] as LogWithDecodedArgs; + const receiverLog = txReceipt.logs[1] as LogWithDecodedArgs; expect(transferLog.args._from).to.be.equal(from); expect(transferLog.args._to).to.be.equal(to); expect(transferLog.args._tokenId).to.be.bignumber.equal(tokenId); diff --git a/packages/types/CHANGELOG.json b/packages/types/CHANGELOG.json index bb39116c2..b99a8bc35 100644 --- a/packages/types/CHANGELOG.json +++ b/packages/types/CHANGELOG.json @@ -1,5 +1,5 @@ [ - { + { "version": "1.0.1-rc.5", "changes": [ { diff --git a/packages/utils/CHANGELOG.json b/packages/utils/CHANGELOG.json index 673c6b39c..796a99a5a 100644 --- a/packages/utils/CHANGELOG.json +++ b/packages/utils/CHANGELOG.json @@ -7,7 +7,8 @@ "pr": 807 }, { - "note": "Store different ABIs for events with same function signature and different amount of indexed arguments", + "note": + "Store different ABIs for events with same function signature and different amount of indexed arguments", "pr": 933 } ], -- cgit v1.2.3