tari-project / tari / 6066120729

61%

push

github

feat: add consistent ban reason for sync (#5729)

Description
---
Added consistent ban reason for sync/block_sync sync/header_sync and
sync/horizon_state_sync w.r.t. validation errors.

Motivation and Context
---
`ValidationError::get_ban_reason` will determine if a particular
validation error is ban-able.

How Has This Been Tested?
---
Pass existing tests

What process can a PR reviewer use to test or verify this change?
---
Code walk-through

<!-- Checklist -->
<!-- 1. Is the title of your PR in the form that would make nice release
notes? The title, excluding the conventional commit
tag, will be included exactly as is in the CHANGELOG, so please think
about it carefully. -->


Breaking Changes
---

- [x] None
- [ ] Requires data directory on base node to be deleted
- [ ] Requires hard fork
- [ ] Other - Please specify

<!-- Does this include a breaking change? If so, include this line as a
footer -->
<!-- BREAKING CHANGE: Description what the user should do, e.g. delete a
database, resync the chain -->

Co-authored-by: SW van Heerden <swvheerden@gmail.com>

5 of 5 new or added lines in 3 files covered. (100.0%)

71202 of 94744 relevant lines covered (75.15%)

190741.53 hits per line