maptuhec/vesting

vesting contract

vesting's Issues

What is address.balance returning you and is this related to this task?

Tokens or ETH? Can you then guess why your tests are failing with not having ether?

Will I loose my tokens forever?

vesting/contracts/Vesting.sol

Line 58 in 608288d

owner.transfer(fourthPeriodTokens);

There is this part of the requirement:
'Once the owner calls claim, he will claim all unclaimed tokens from this and any previous periods.'

So lets look at your code. Two scenarios - what will happen:

1. I have 'missed' the first period (decided not to withdraw my money in the first period) - will I be able to get them at a later period?
2. I've claimed my first period tokens, but missed the second period - will I be able to get them at a later period?

Name this better like - tokenOwnerAddress

vesting/test/vesting.js

Line 13 in beed2c6

var tokenAddress = accounts[0];

How many tokens does this contract own?

When you name the upper variables better you will see that you have a problem here

vesting/test/vesting.js

Line 25 in beed2c6

tokenAddress,

This will never happen

vesting/contracts/Vesting.sol

Line 31 in 608288d

if (tokenBalance < 0) {

Name this better

vesting/test/vesting.js

Line 14 in beed2c6

var vestingAddress = accounts[1];

Name this better - is this the withdrawn token amount?

vesting/contracts/Vesting.sol

Line 7 in beed2c6

uint256 public tokenAmount;

If this is the withdrawn token amount name it better. If this is the total token balance you should not be storing it but instead of reading it from the vesting contract on the fly