refactor: remove trait bound on TxType

[ftupas]

This pr does the following:

• removes trait bound From<alloy_consensus::TxType>
  

  reth/crates/primitives-traits/src/tx_type.rs

  Lines 6 to 28 in d8edf9c

  	/// Trait representing the behavior of a transaction type.
  	pub trait TxType:
  	Into<u8>
  	+ Into<U8>
  	+ PartialEq
  	+ Eq
  	+ PartialEq<u8>
  	+ TryFrom<u8, Error = Eip2718Error>
  	+ TryFrom<u64>
  	+ TryFrom<U64>
  	+ From<alloy_consensus::TxType>
  	+ Debug
  	+ Display
  	+ Clone
  	+ Copy
  	+ Default
  	+ Encodable
  	+ Decodable
  	+ Send
  	+ Sync
  	+ 'static
  	{
  	}

• refactor tests and use rstest
• resolves Remove trait bound on TxType #12346

[emhane]

would have been preferred if the tests refactor came in a separate pr, makes more sense to review like that

[ftupas]

would have been preferred if the tests refactor came in a separate pr, makes more sense to review like that

gotcha I could create an issue to do a proper refactor of the tests