perf: only check trace in tests if has a reason to

[antazoey]

What I did

Change it to not check call_tree unnecessarily (if no --gas flag given).
This makes tests run faster when not using --gas

NOTE: This PR also fixes many race conditions and flakinesses in the tests that came up when adding a test as well as turning off geth snapshotting for now because of this issue: ethereum/go-ethereum#26154

How I did it

check if needing gas report before checking the grabbing the call tree.

How to verify it

Can work with Hardhat project with tracing issues and things still work as far as testing goes (obviously sans all tracing features)

Checklist

• All changes are completed
• New test cases have been added
• Documentation has been updated

[antazoey]

Hmm, thinking I may have a better solution, BRB.

[antazoey]

• Fix issue with snapshotting (noticed while testing)

Update: will have to ignore for now , see ethereum/go-ethereum#26154

[ghost]

Is this a configuration that should also apply to the dev-message assertion or do we only mean tracing here as in gas tracing / profiling?

[antazoey]

Is this a configuration that should also apply to the dev-message assertion or do we only mean tracing here as in gas tracing / profiling?

I removed the config because I found out it wasn't needed! So no worries there :)