Re-enable Fulu execution_payload tests

When spec v1.6.0-alpha.1 is released we need to re-enable the tests disabled by this PR:

- https://github.com/sigp/lighthouse/pull/7540

The issue is that we are currently ahead of the spec release v1.6.0-alpha.0, but up to date with `dev` and `devnet-0`:

https://github.com/sigp/lighthouse/blob/cd83d8d95dddd7d965999b8a7fd940ed17bbfc18/consensus/types/src/chain_spec.rs#L656-L659

We can remove that comment as well when making this change.

The main thing to remove is the skipped known failure:

https://github.com/sigp/lighthouse/blob/d86233d07ba31c639f8b18ae5979cb47df4b2ec7/testing/ef_tests/src/cases/operations.rs#L577-L586

	/// Return the value of `MAX_BLOBS_PER_BLOCK` for the given `epoch`.
	/// NOTE: this function is technically not spec compliant, but
	/// I'm told this is what the other clients are doing for `devnet-0`..
	pub fn max_blobs_per_block(&self, epoch: Epoch) -> u64 {

	// 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
	&& _case_index == 32
	{
	return Err(Error::SkippedKnownFailure);
	}

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Re-enable Fulu execution_payload tests #7547

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Re-enable Fulu execution_payload tests #7547

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions