fix: add blob_gas_used after succesful tx execution

[Rjected]

Previously we were adding blob_gas_used, even if the tx failed execution. This caused incorrect blob gas used returned in built payloads, since the payload would contain fewer blobs. This means a subsequent engine_newPayload would fail.

This makes sure we only add the blob_gas_used if the tx succeeded.

Fixes the hive engine cancun/Blob Transaction Ordering, Single Account tests.

[codecov]

Codecov Report

Merging #4913 (7578c76) into main (066998d) will decrease coverage by 0.04%.
Report is 3 commits behind head on main.
The diff coverage is 0.00%.

Files	Coverage Δ
crates/payload/basic/src/lib.rs	0.00% <0.00%> (ø)
crates/rpc/rpc/src/eth/api/pending_block.rs	0.00% <0.00%> (ø)

... and 8 files with indirect coverage changes

Flag	Coverage Δ
integration-tests	15.48% <0.00%> (-0.02%)	⬇️
unit-tests	62.61% <0.00%> (-0.03%)	⬇️

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

Components	Coverage Δ
reth binary	32.63% <ø> (ø)
blockchain tree	80.64% <ø> (ø)
pipeline	88.45% <ø> (ø)
storage (db)	73.30% <ø> (ø)
trie	94.52% <ø> (+0.03%)	⬆️
txpool	48.90% <ø> (-0.20%)	⬇️
networking	76.06% <ø> (-0.08%)	⬇️
rpc	57.71% <0.00%> (-0.01%)	⬇️
consensus	61.05% <ø> (ø)
revm	28.47% <ø> (ø)
payload builder	8.14% <0.00%> (-0.03%)	⬇️
primitives	85.41% <ø> (-0.02%)	⬇️

[mattsse]

nice find