Validator refactor

[SWvheerden]

Description

Refactor the validators to ensure that each validation step only completes once. Before this PR some validation steps where done twice.
The stateless validator also now checks accounting balance and this is not only checked when the block is added to the main chain.

Motivation and Context

There was a lot of duplicate checking between the stateless and stateful validators. This is now ensured that each check is only ran once. It was also checked to ensure that state-full and stateless is always ran. Blocks are checked by the stateless before going into the orphan pool. After this the block is checked and might be moved to main chain. Only then is state-full validation performed.

We can now be sure that all blocks in the orphan pool was checked and it correct.

How Has This Been Tested?

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Feature refactor (No new feature or functional changes, but performance or technical debt improvements)
• New Tests
• Documentation

Checklist:

• I'm merging against the development branch.
• I ran cargo-fmt --all before pushing.
• I have squashed my commits into a single commit.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.

[SWvheerden]

This is blocked by : #1861

[neonknight64]

LGTM

[neonknight64]

LGTM