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
chore: Misc contract cleanup and fixes #10634
Conversation
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Node: E2E Test Results 1 files ± 0 1 suites ±0 23m 5s ⏱️ + 23m 5s Results for commit 693ba10. ± Comparison against base commit 0f965a4. This pull request removes 1 and adds 311 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #10634 +/- ##
==========================================
Coverage ? 62.81%
Complexity ? 30894
==========================================
Files ? 3376
Lines ? 136345
Branches ? 14207
==========================================
Hits ? 85641
Misses ? 47308
Partials ? 3396 ☔ View full report in Codecov by Sentry. |
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.
LGTM
Description:
AbortException
to eliminate duplication of computing the effectivesenderId
; now the exception thrown from an abortedprocessTransaction()
call includes thesenderId
fromcomputeInvolvedParties()
.feesOnlyUpdater
after an abort, this also lets us easily preserve the HAPI fees charged by the workflow by just committing the root updater after an abort.CustomGasCharging
to only charge any fee if all fees are payable.try
/catch (ResourceExhaustedException)
aroundrunToCompletion()
.asPriorityId()
calls inHevmTransactionFactory
).EthereumSuite
.