Un-ignore EF test for v1.6.0-alpha.1 (#7632)

michaelsproul · web-flow · commit fd643c310c4e · 2025-06-23T13:11:46.000Z
Closes: - #7547 Run the test that was previously ignored when we were between spec versions.
diff --git a/testing/ef_tests/src/cases/operations.rs b/testing/ef_tests/src/cases/operations.rs
@@ -22,7 +22,6 @@ use state_processing::{
     ConsensusContext,
 };
 use std::fmt::Debug;
-use std::path::PathBuf;
 use types::{
     Attestation, AttesterSlashing, BeaconBlock, BeaconBlockBody, BeaconBlockBodyBellatrix,
     BeaconBlockBodyCapella, BeaconBlockBodyDeneb, BeaconBlockBodyElectra, BeaconBlockBodyFulu,
@@ -50,7 +49,6 @@ pub struct WithdrawalsPayload<E: EthSpec> {
 
 #[derive(Debug, Clone)]
 pub struct Operations<E: EthSpec, O: Operation<E>> {
-    path: PathBuf,
     metadata: Metadata,
     execution_metadata: Option<ExecutionMetadata>,
     pub pre: BeaconState<E>,
@@ -557,7 +555,6 @@ impl<E: EthSpec, O: Operation<E>> LoadCase for Operations<E, O> {
         };
 
         Ok(Self {
-            path: path.into(),
             metadata,
             execution_metadata,
             pre,
@@ -577,17 +574,6 @@ impl<E: EthSpec, O: Operation<E>> Case for Operations<E, O> {
     }
 
     fn result(&self, _case_index: usize, fork_name: ForkName) -> Result<(), Error> {
-        // FIXME(das): remove this once v1.6.0-alpha.1 is released
-        // We are ahead of the v1.6.0-alpha.0 spec in our implementation of
-        // `get_max_blobs_per_block`, so we fail the execution payload test which expects the
-        // empty blob schedule to generate an error.
-        if O::handler_name() == "execution_payload"
-            && fork_name == ForkName::Fulu
-            && self.path.ends_with("invalid_exceed_max_blobs_per_block")
-        {
-            return Err(Error::SkippedKnownFailure);
-        }
-
         let spec = &testing_spec::<E>(fork_name);
 
         let mut pre_state = self.pre.clone();
