chore: set trace result to null if non revert or selfdestruct

[mattsse]

closes #3834

ref https://github.com/ledgerwatch/erigon/blob/ef4ccd47736019132c1967d6921f6c4b014eb515/turbo/jsonrpc/trace_adhoc.go#L392-L408

https://github.com/ledgerwatch/erigon/blob/ef4ccd47736019132c1967d6921f6c4b014eb515/turbo/jsonrpc/trace_adhoc.go#L325-L327

[codecov]

Codecov Report

Merging #3840 (a8380c7) into main (0ef68b7) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

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

... and 13 files with indirect coverage changes

Flag	Coverage Δ
integration-tests	15.60% <0.00%> (-0.03%)	⬇️
unit-tests	64.02% <0.00%> (-0.05%)	⬇️

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

Components	Coverage Δ
reth binary	26.49% <ø> (ø)
blockchain tree	83.02% <ø> (ø)
pipeline	87.04% <ø> (ø)
storage (db)	74.32% <ø> (ø)
trie	94.65% <ø> (ø)
txpool	46.55% <ø> (-0.27%)	⬇️
networking	77.51% <ø> (-0.09%)	⬇️
rpc	58.34% <ø> (ø)
consensus	65.00% <ø> (ø)
revm	33.66% <0.00%> (ø)
payload builder	6.61% <ø> (ø)
primitives	88.06% <ø> (+<0.01%)	⬆️

[Rjected]

lgtm, the

        let result = if action.is_selfdestruct() ||
            (self.trace.is_error() && !self.trace.is_revert())

condition is much easier to understand