fix: populate transaction trace error field

[cjeva10]

Follow up for #3611

[codecov]

Codecov Report

Merging #3627 (7586357) into main (6347aac) will increase coverage by 0.01%.
The diff coverage is 0.00%.

Impacted Files	Coverage Δ
crates/revm/revm-inspectors/src/tracing/types.rs	0.00% <0.00%> (ø)

... and 8 files with indirect coverage changes

Flag	Coverage Δ
integration-tests	15.96% <0.00%> (+<0.01%)	⬆️
unit-tests	64.11% <0.00%> (+<0.01%)	⬆️

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

Components	Coverage Δ
reth binary	26.40% <ø> (ø)
blockchain tree	81.25% <ø> (ø)
pipeline	86.69% <ø> (ø)
storage (db)	73.48% <ø> (ø)
trie	94.66% <ø> (ø)
txpool	49.73% <ø> (+0.61%)	⬆️
networking	77.91% <ø> (+0.01%)	⬆️
rpc	58.04% <ø> (-0.01%)	⬇️
consensus	63.35% <ø> (-0.15%)	⬇️
revm	34.89% <0.00%> (-0.01%)	⬇️
payload builder	6.83% <ø> (ø)
primitives	88.25% <ø> (-0.02%)	⬇️

[mattsse]

lgtm!

what we need to improve is error messages here:

reth/crates/revm/revm-inspectors/src/tracing/types.rs

Lines 162 to 165 in 73bfb2c

	/// Returns the error message if it is an erroneous result.
	pub(crate) fn as_error(&self) -> Option<String> {
	self.is_error().then(|| format!("{:?}", self.status))
	}

because rn we will return Revert but we'd like to return "Reverted", so ne need a custom convert,
not sure about the other variants, but we should at least handle the revert variant