-
Notifications
You must be signed in to change notification settings - Fork 119
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
Fix ApproveAllowanceSuite tests #9649
Conversation
Signed-off-by: Miroslav Gatsanoga <miroslav.gatsanoga@limechain.tech>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Nikita Lebedev <nikita.lebedev@limechain.tech>
Signed-off-by: Nikita Lebedev <nikita.lebedev@limechain.tech>
# Conflicts: # hedera-node/hedera-smart-contract-service-impl/src/main/java/com/hedera/node/app/service/contract/impl/exec/systemcontracts/hts/AbstractNonRevertibleTokenViewCall.java
Signed-off-by: Nikita Lebedev <nikita.lebedev@limechain.tech>
Node: E2E Test Results 1 files ± 0 1 suites ±0 21m 18s ⏱️ + 21m 18s Results for commit 894c7bb. ± Comparison against base commit 2f890ea. This pull request removes 1 and adds 310 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Signed-off-by: Nikita Lebedev <nikita.lebedev@limechain.tech>
...p/service/contract/impl/exec/systemcontracts/hts/grantapproval/ClassicGrantApprovalCall.java
Outdated
Show resolved
Hide resolved
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.
Overall great work! 🙌
Two items:
- (Blocking) We need to re-create
EncodingFacade
utilities incontract-service-impl
so we don't have lingering dependencies on mono-service. - (Nice-to-have) Can we add to the
HtsCall
interface a default method,
default PricedResult execute(MessageFrame frame) {
return execute();
}
so this PR doesn't have to change all the files for HtsCall
implementations that don't actually need the MessageFrame
?
# Conflicts: # hedera-node/hedera-smart-contract-service-impl/src/main/java/com/hedera/node/app/service/contract/impl/exec/systemcontracts/hts/AbstractRevertibleTokenViewCall.java
…d a default method to the HtsCall interface; reverted changes in unit tests Signed-off-by: Nikita Lebedev <nikita.lebedev@limechain.tech>
Hey @tinker-michaelj, I've addressed your points 🙂 |
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.
Awesome, tyvm @thenswan 🙌
Signed-off-by: Miroslav Gatsanoga <miroslav.gatsanoga@limechain.tech> Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com> Signed-off-by: Nikita Lebedev <nikita.lebedev@limechain.tech> Co-authored-by: Miroslav Gatsanoga <miroslav.gatsanoga@limechain.tech> Co-authored-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Miroslav Gatsanoga <miroslav.gatsanoga@limechain.tech> Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com> Signed-off-by: Nikita Lebedev <nikita.lebedev@limechain.tech> Co-authored-by: Miroslav Gatsanoga <miroslav.gatsanoga@limechain.tech> Co-authored-by: Michael Tinker <michael.tinker@swirldslabs.com> Signed-off-by: Nick Poorman <nick@swirldslabs.com>
Description:
Fix ApproveAllowanceSuite tests
Related issue(s):
Fixes #9357