Rename consensusFormatValidationFlag

[acolytec3]

• Renames consensusFormatValidation to skipConsensusFormatValidation to match similar flags in blockchain and defaults this option to false.
• Removes a couple of troublesome Kovan tests that would need to be removed with Removed Kovan (EOL) / Common Mainnet Merge Default HF Fix #2206

[codecov]

Codecov Report

Merging #2209 (2901172) into master (5dc3fd7) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Flag	Coverage Δ
block	92.98% <100.00%> (ø)
blockchain	88.88% <ø> (ø)
client	87.00% <ø> (ø)
common	98.08% <ø> (ø)
devp2p	92.22% <ø> (-0.09%)	⬇️
ethash	∅ <ø> (∅)
evm	79.07% <ø> (ø)
rlp	∅ <ø> (∅)
statemanager	88.09% <ø> (ø)
trie	89.52% <ø> (ø)
tx	97.98% <ø> (ø)
util	92.33% <ø> (ø)
vm	85.24% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

[holgerd77]

LGTM

[holgerd77]

Ah, short nit: can you please update the header description that you didn't remove the Kovan tests along this PR (just delete is sufficient)?

[acolytec3]

I did remove them though. I'm not sure why they aren't showing up in the Files Changed section of the PR. 🤔

[holgerd77]

I did remove them though. I'm not sure why they aren't showing up in the Files Changed section of the PR. 🤔

Ah, these tests were already removed in the Kovan PR #2206 🙂 .

[holgerd77]

(maybe not do something like this before the related PR is merged, might also lead to somewhat dangerous situations (so, not dramatically dangerous, but just some file/update conflicts))