hicommonwealth / edgeware-lockdrop

Lockdrop contracts
GNU General Public License v3.0
23 stars 15 forks source link

index is Undefined in lockdropHelper #30

Open ltfschoen opened 4 years ago

ltfschoen commented 4 years ago

The lockdropHelper iterates through index of the lockdropContracts that is passed into the function here https://github.com/hicommonwealth/edgeware-lockdrop/blob/master/helpers/lockdropHelper.js#L71, but in most of the tests we're only passing in a single contract https://github.com/hicommonwealth/edgeware-lockdrop/blob/master/test/1-lockdrop.spec.js#L144 that's generated here https://github.com/hicommonwealth/edgeware-lockdrop/blob/master/test/1-lockdrop.spec.js#L21, instead of passing in an array of them.

ltfschoen commented 4 years ago

It should be for (let index in lockdropContracts) { instead of just for (index in lockdropContracts) { otherwise it gives error index is not defined.