New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EN-8010/ESDT-ticker-and-name #2455
EN-8010/ESDT-ticker-and-name #2455
Conversation
vm/errors.go
Outdated
@@ -170,6 +167,12 @@ var ErrInvalidMaxNumberOfNodes = errors.New("invalid number of max number of nod | |||
// ErrTokenNameNotHumanReadable signals that token name is not human readable | |||
var ErrTokenNameNotHumanReadable = errors.New("token name is not human readable") | |||
|
|||
// ErrTickerNameNotValid signals that ticker name is not valid | |||
var ErrTickerNameNotValid = errors.New("token name is not valid") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ticker name is not valid
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
vm/errors.go
Outdated
var ErrTickerNameNotValid = errors.New("token name is not valid") | ||
|
||
// ErrCouldNotCreateNewIdentifier signals that token identifier could not be created | ||
var ErrCouldNotCreateNewIdentifier = errors.New("token identifier could not be created") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ErrCouldNotCreateNewTokenIdentifier or ErrCouldNotCreateTokenIdentifier
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
@@ -174,7 +174,7 @@ func TestEsdt_ExecuteIssue(t *testing.T) { | |||
assert.Equal(t, vmcommon.Ok, output) | |||
|
|||
assert.Equal(t, 1, len(eei.output)) | |||
assert.Equal(t, []byte("01234567891"), eei.output[0]) | |||
assert.Equal(t, tokenID, eei.output[0]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the check for initial supply would also help here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there are other unit tests for this.
vm/errors.go
Outdated
@@ -176,7 +176,7 @@ var ErrCouldNotCreateNewTokenIdentifier = errors.New("token identifier could not | |||
// ErrBLSPublicKeyAlreadyJailed signals that bls public key was already jailed | |||
var ErrBLSPublicKeyAlreadyJailed = errors.New("bls public key already jailed") | |||
|
|||
// ErrInvalidEndOfEpochAccessAddress signals that end of epoch access address is invalid | |||
// ErrInvalidEndOfEpochAccessAddress signals that end of epoch access address is invalidgetExistingToken |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete getExistingToken from the end of the comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
vm/errors.go
Outdated
@@ -185,8 +185,8 @@ var ErrNilChanceComputer = errors.New("nil chance computer") | |||
// ErrNilEpochNotifier signals that the provided EpochNotifier is nil | |||
var ErrNilEpochNotifier = errors.New("nil EpochNotifier") | |||
|
|||
// ErrNoTokenWithGivenName signals that token does not exists with given name | |||
var ErrNoTokenWithGivenName = errors.New("no token with given name") | |||
// ErrNoTickerWithGivenName signals that token does not exists with given name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"ticker does not exist" instead "token does not exists"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
vm/errors.go
Outdated
// ErrNoTokenWithGivenName signals that token does not exists with given name | ||
var ErrNoTokenWithGivenName = errors.New("no token with given name") | ||
// ErrNoTickerWithGivenName signals that token does not exists with given name | ||
var ErrNoTickerWithGivenName = errors.New("no token with given name") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"no ticker" instead "no token"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
3964fab
Implemented ESDT ticker and creation of new identifier