search

OneDaijo / daijo-contracts

Apache License 2.0
6 stars 2 forks source link

Added back working purchase call to js test #31

Closed thiefinparis closed 7 years ago

thiefinparis commented 7 years ago

This adds some of the previously failing logic in the js test back. The purchase call takes around 95k gas (the default is 21k), so that needed to be bumped for the payment call to make the test work. For once, the OOG error was telling the truth :)

thiefinparis commented 7 years ago

@zlgrube @nickdeng4 Please review when you get a chance

thiefinparis commented 7 years ago

Will finish this tomm. Sorry about the delay.

thiefinparis commented 7 years ago

Okay, this test should be fixed now. Please review @nickdeng4 @zlgrube. I refactored the crowdsale contract a bit. Found a few unused variables along the way, and there were multiple bugs that were rather subtle related to initialization and state.

thiefinparis commented 7 years ago

@nickdeng4 any more comments?