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
fixed more tests and made mocks follow new vmcommon interface #2020
Conversation
github.com/cpuguy83/go-md2man v1.0.10/go.mod h1:SmD6nW6nTyfqj6ABTjUi3V3JVMnlJmwcJI5acqYI6dE= | ||
github.com/cpuguy83/go-md2man/v2 v2.0.0-20190314233015-f79a8a8ca69d h1:U+s90UTSYgptZMwQh2aRr3LuazLJIa+Pg3Kc1ylSYVY= |
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.
go mod tidy
@@ -284,7 +284,8 @@ func TestAgarioContractJoinGameReward(t *testing.T) { | |||
_, err = testContext.Accounts.Commit() |
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 folder IELE
…k/elrond-go into integrate-new-hooks
…k/elrond-go into integrate-new-hooks
@@ -162,3 +162,11 @@ func ConvertToEvenHexBigInt(value *big.Int) string { | |||
|
|||
return str | |||
} | |||
|
|||
// BooleanToInt takes a bool argument and returns 0 for false, 1 for true | |||
func BooleanToInt(b bool) int { |
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.
Not tested
require.Equal(t, scr.Value.Cmp(finalValue), 0) | ||
} | ||
|
||
func TestScProcessor_DoNotRefundGasToSenderForAsyncCall(t *testing.T) { |
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 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.
Partially reviewed ☝️
Integrated new vmcommon + removed references to IELE for now.
Some tests are still failing but unrelated to to the new vmcommon.
New PR will follow integrating arwen promise implementation