ci: run good coverage tests for revm testing suite

Signed-off-by: Dmitry Savonin <[email protected]>

Author: dmitry123

.github/workflows/ci.yml

@@ -119,4 +119,4 @@ jobs:
         run: make sync_tests
       - name: Run ethereum tests
         working-directory: revm/e2e
-        run: make run_all_tests
+        run: make run_general_state_tests

crates/types/src/context/v1.rs

@@ -1,5 +1,4 @@
 use crate::{context::ContextReader, Address, B256, U256};
-use alloc::vec;
 use alloy_primitives::Bytes;
 
 #[derive(Default, Clone, Debug, PartialEq)]
@@ -14,21 +13,6 @@ pub struct BlockContextV1 {
     pub base_fee: U256,
 }
 
-impl From<&revm_primitives::Env> for BlockContextV1 {
-    fn from(value: &revm_primitives::Env) -> Self {
-        Self {
-            chain_id: value.cfg.chain_id,
-            coinbase: value.block.coinbase,
-            timestamp: value.block.timestamp.as_limbs()[0],
-            number: value.block.number.as_limbs()[0],
-            difficulty: value.block.difficulty,
-            prev_randao: value.block.prevrandao.unwrap_or_default(),
-            gas_limit: value.block.gas_limit.as_limbs()[0],
-            base_fee: value.block.basefee,
-        }
-    }
-}
-
 #[derive(Default, Clone, Debug, PartialEq)]
 pub struct ContractContextV1 {
     pub address: Address,
@@ -39,52 +23,6 @@ pub struct ContractContextV1 {
     pub gas_limit: u64,
 }
 
-pub fn env_from_context<CR: ContextReader>(cr: CR) -> revm_primitives::Env {
-    use revm_primitives::{AnalysisKind, BlockEnv, CfgEnv, Env, TransactTo, TxEnv};
-    Env {
-        cfg: {
-            let mut cfg_env = CfgEnv::default();
-            cfg_env.chain_id = cr.block_chain_id();
-            cfg_env.perf_analyse_created_bytecodes = AnalysisKind::Raw;
-            cfg_env
-        },
-        block: BlockEnv {
-            number: U256::from(cr.block_number()),
-            coinbase: cr.block_coinbase(),
-            timestamp: U256::from(cr.block_timestamp()),
-            gas_limit: U256::from(cr.block_gas_limit()),
-            basefee: cr.block_base_fee(),
-            difficulty: cr.block_difficulty(),
-            prevrandao: Some(cr.block_prev_randao()),
-            blob_excess_gas_and_price: None,
-        },
-        tx: TxEnv {
-            caller: cr.tx_origin(),
-            gas_limit: cr.tx_gas_limit(),
-            gas_price: cr.tx_gas_price(),
-            // we don't check this field, and we don't know what type of "transact"
-            // we execute right now, so can safely skip the field
-            transact_to: TransactTo::Call(Address::ZERO),
-            value: cr.tx_value(),
-            // we don't use this field, so there is no need to do redundant copy operation
-            data: Bytes::default(),
-            // we do nonce and chain id checks before executing transaction
-            nonce: None,
-            chain_id: None,
-            // we check access lists in advance before executing a smart contract, it
-            // doesn't affect gas price or something else, can skip
-            access_list: Default::default(),
-            gas_priority_fee: cr.tx_gas_priority_fee(),
-            // TODO(dmitry123): "we don't support blobs yet, so 2 tests from e2e testing suite fail"
-            blob_hashes: vec![],        // tx_context.blob_hashes.clone(),
-            max_fee_per_blob_gas: None, // tx_context.max_fee_per_blob_gas,
-            authorization_list: None,
-            #[cfg(feature = "optimism")]
-            optimism: Default::default(),
-        },
-    }
-}
-
 #[derive(Default, Clone, Debug, PartialEq)]
 pub struct TxContextV1 {
     pub gas_limit: u64,
@@ -97,22 +35,6 @@ pub struct TxContextV1 {
     pub value: U256,
 }
 
-impl From<&revm_primitives::Env> for TxContextV1 {
-    fn from(value: &revm_primitives::Env) -> Self {
-        Self {
-            gas_limit: value.tx.gas_limit,
-            nonce: value.tx.nonce.unwrap_or_default(),
-            gas_price: value.tx.gas_price,
-            gas_priority_fee: value.tx.gas_priority_fee,
-            origin: value.tx.caller,
-            // data: value.tx.data.clone(),
-            // blob_hashes: value.tx.blob_hashes.clone(),
-            // max_fee_per_blob_gas: value.tx.max_fee_per_blob_gas,
-            value: value.tx.value,
-        }
-    }
-}
-
 #[derive(Default, Clone, Debug, PartialEq)]
 pub struct SharedContextInputV1 {
     pub block: BlockContextV1,