fix: return INVALID again if block contains blob transactions pre-cancun

[Rjected]

Fixes #4670 again

The test now expects an INVALID response instead of an RPC error, so we can get rid of the double nested Result, and just have a PayloadError to represent when a pre-cancun payload has blob transactions.

[mattsse]

yes! this is way nicer

[codecov]

Codecov Report

Merging #4845 (c5927d2) into main (486b289) will decrease coverage by 0.01%.
Report is 3 commits behind head on main.
The diff coverage is 83.33%.

Files	Coverage Δ
crates/rpc/rpc-types/src/eth/engine/payload.rs	80.92% <ø> (ø)
crates/consensus/beacon/src/engine/mod.rs	73.21% <83.33%> (+<0.01%)	⬆️

... and 11 files with indirect coverage changes

Flag	Coverage Δ
integration-tests	15.60% <0.00%> (-0.01%)	⬇️
unit-tests	62.70% <83.33%> (-0.02%)	⬇️

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

Components	Coverage Δ
reth binary	32.06% <ø> (ø)
blockchain tree	83.76% <ø> (ø)
pipeline	88.43% <ø> (-0.02%)	⬇️
storage (db)	73.29% <ø> (ø)
trie	94.48% <ø> (-0.04%)	⬇️
txpool	49.54% <ø> (ø)
networking	76.47% <ø> (+0.01%)	⬆️
rpc	57.80% <ø> (-0.02%)	⬇️
consensus	62.63% <83.33%> (-0.02%)	⬇️
revm	28.54% <ø> (ø)
payload builder	8.16% <ø> (ø)
primitives	85.26% <ø> (-0.02%)	⬇️