core: restrict the number of allowed SC notifications
Close #3490.
@roman-khimov, what do you think? Another option is to cache size for every emitted notification and then check the overall notifications size every time we're going to emit one more notification, it also may be implemented.
I'll add more tests once we agree on solution.
Ref. https://github.com/neo-project/neo/pull/3548.
Codecov Report
Attention: Patch coverage is 38.09524% with 39 lines in your changes missing coverage. Please review.
Project coverage is 82.74%. Comparing base (
e63cbe7) to head (2c8bd05). Report is 5 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #3640 +/- ##
==========================================
- Coverage 82.88% 82.74% -0.14%
==========================================
Files 338 338
Lines 47448 47484 +36
==========================================
- Hits 39326 39290 -36
- Misses 6502 6563 +61
- Partials 1620 1631 +11
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
🚀 New features to boost your workflow:
- ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.