Skip to content

Comments

Restore randomized blocks test generator#4872

Merged
jihoonsong merged 2 commits intoethereum:masterfrom
leolara:leolara/restore-randomized-block-test-generator
Jan 29, 2026
Merged

Restore randomized blocks test generator#4872
jihoonsong merged 2 commits intoethereum:masterfrom
leolara:leolara/restore-randomized-block-test-generator

Conversation

@leolara
Copy link
Member

@leolara leolara commented Jan 28, 2026

  • This was removed in Simplify reference test generators #4317
  • Also fixed a small bug in tests/infra/yield_generator.py that was found while doing this
  • After this is merged, we can work on adding Gloas to this.

Note: you can check the generator works exactly as before by running it, should generate the same test suites that are already commited.

@leolara leolara requested a review from jihoonsong January 28, 2026 07:44
@jtraglia jtraglia added the testing CI, actions, tests, testing infra label Jan 28, 2026
Copy link
Member

@jihoonsong jihoonsong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems it has a good refactor.

zero_activation_threshold,
only_generator,
)
from eth2spec.test.context import (
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

from eth2spec.test.context import here can be merged with the above one. But it is fixed by running ruff anyway.

@jihoonsong jihoonsong merged commit 9115402 into ethereum:master Jan 29, 2026
14 checks passed
leolara pushed a commit that referenced this pull request Feb 5, 2026
# Description

Adds randomized block tests for Gloas. Covers builder, bids and payload
attestations.

## Changes

- Adds `get_random_payload_attestations` using
`prepare_signed_payload_attestation`
- Adds `random_block_gloas` based on Altair blocks appending new fields
- Adds `_add_random_builders` helper
- Adds `_build_random_signed_bid`

## Relations

Follows up:

- #4872
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing CI, actions, tests, testing infra

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants