-
Notifications
You must be signed in to change notification settings - Fork 198
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
process, integrationTests, testscommon: fix total gas provided with scheduled SC calls #3743
process, integrationTests, testscommon: fix total gas provided with scheduled SC calls #3743
Conversation
Codecov Report
@@ Coverage Diff @@
## fix-edge-case-of-set-scheduled-scrs-at-bootstrap #3743 +/- ##
=================================================================================
Coverage 73.66% 73.66%
=================================================================================
Files 599 599
Lines 79167 79162 -5
=================================================================================
Hits 58315 58315
+ Misses 16266 16260 -6
- Partials 4586 4587 +1
Continue to review full report at Codecov.
|
…fix-total-gas-provided-with-scheduled
@@ -906,11 +906,21 @@ func (txs *transactions) getAllTxsFromMiniBlock( | |||
} | |||
|
|||
func (txs *transactions) getRemainingGasPerBlock() uint64 { | |||
gasConsumed := txs.getTotalGasConsumed() | |||
gasProvided := txs.getTotalGasConsumed() |
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.
Actually this is really consumed as in getTotalGasConsumed method, from total gas provided is subtracted gas penalized and gas refunded if they exist
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
01da2f9
to
0c82a8e
Compare
No description provided.