nspcc-dev / neo-go

Go Node and SDK for the NEO blockchain
MIT License
122 stars 78 forks source link

Improve neotest to serve coverage needs #3551

Closed AnnaShaleva closed 1 month ago

AnnaShaleva commented 1 month ago
  1. Bind neotest.AddSystemFee and neotest.TestInvoke to Executor state.
  2. Calculate system fee for deployment transactions.

Reasons are explained in the commit messages. @roman-khimov, these commits made in a breaking way, but if we care about our users a lot, then I can mark related methods as "deprecated" and introduce new Executor methods in a soft way. Do we?

A ground for #3462.

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 86.05%. Comparing base (ffcbe6a) to head (72d1427).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #3551 +/- ## ========================================== + Coverage 86.04% 86.05% +0.01% ========================================== Files 330 330 Lines 38674 38675 +1 ========================================== + Hits 33276 33282 +6 + Misses 3853 3848 -5 Partials 1545 1545 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.