feat: add TryFrom for GethTrace for all inner variants

[Ebolon]

Fixes #929

For me it feels more rust like to use {variant}::try_from(geth_trace) instead of geth_trace.try_into_{variant}(). How do you feel about that @mattsse?

Example:

use alloy_primitives::b256;
use alloy_provider::{ext::DebugApi, ProviderBuilder};
use alloy_rpc_types_trace::geth::{
    GethDebugBuiltInTracerType, GethDebugTracerConfig, GethDebugTracerType, GethDebugTracingOptions, GethDefaultTracingOptions,
    PreStateConfig, PreStateFrame, TraceResult,
};
use std::convert::{TryFrom, TryInto};

#[tokio::main]
async fn main() {
    let url = std::env::var("ETH_MAINNET_HTTP").expect("$ETH_MAINNET_HTTP must be set.");
    let node_url = url::Url::parse(url.as_str()).unwrap();
    let provider = ProviderBuilder::new().on_http(node_url);

    let tracer_opts = GethDebugTracingOptions {
        tracer: Some(GethDebugTracerType::BuiltInTracer(GethDebugBuiltInTracerType::PreStateTracer)),
        config: GethDefaultTracingOptions::default().disable_storage().disable_stack().disable_memory().disable_return_data(),
        timeout: None,
        tracer_config: GethDebugTracerConfig::default(),
    }
    .with_prestate_config(PreStateConfig { diff_mode: Some(true) });

    let call_result =
        provider.debug_trace_block_by_hash(b256!("1b9f245b55515d17fd5b3467bfe595670c9599b3d54837cd32cfde5e2e667830"), tracer_opts).await;

    let trace_results = match call_result {
        Ok(trace_results) => trace_results,
        Err(e) => {
            println!("Error: {:?}", e);
            return;
        }
    };

    for trace_result in trace_results {
        match trace_result {
            TraceResult::Success { result, tx_hash } => {
                let pre_state_frame = PreStateFrame::try_from(result);
                assert!(pre_state_frame.is_ok());
            }
            TraceResult::Error { .. } => {}
        }
    }
}

[mattsse]

I'm okay with the tryfrom impls, but I personally would prefer gethtrace functions instead, so you can do something like:

let trace = provider.trace(..).await?.try_into_default()?;

without providing type hints, but we can do both, tryfrom and the functions

[Ebolon]

@mattsse please have a look again.

PS: If you wonder why the clippy fixes: I used the command from the CONTRIBUTING.md in differs from the CI job.

[Ebolon]

I gave the topic a fresh look and came to the conclusion that the TryFrom implementation is redundant at all. From api perspective you initial idea has a better flow and and is easier to use. I adapted the naming of the variants in the function names. My feeling is that it is much better to have the full type name in there. It is easier to read without IDE type hinting and does not maybe lead to a mixup with e.g. Default().

@mattsse feel free to have a look again