onflow / ledger-app-flow

Apache License 2.0
10 stars 12 forks source link

Create account changes #50

Closed relatko closed 3 years ago

relatko commented 3 years ago

Fixes an issue with _countArgumentItems function returning parser error values as number of elements in an array. Bounds the number of Public Keys arguments in create account to 5. We discussed the max to be 3, but there were existing tests containing 4 arguments, so maybe 5 is a better bound. Tests added to verify code behavior when more than 5 public keys are present in the transaction.