Skip to content

Fix mock engine panic in bid gossip spec tests - #13

Merged
pawanjay176 merged 4 commits into
pawanjay176:gloas-gossip-handlersfrom
jimmygchen:codex/gossip-bid-mock-fcu
Sep 25, 2026
Merged

pawanjay176 merged 4 commits into
pawanjay176:gloas-gossip-handlersfrom
jimmygchen:codex/gossip-bid-mock-fcu

Conversation

@jimmygchen

@jimmygchen jimmygchen commented Sep 25, 2026 •

Copy link
Copy Markdown

The mock EL panics when running the beta.2 parent-RANDAO test, even though the gossip test can pass. all_payloads_valid() sets the response after the mock block generator runs.

This uses set_fcu_payload_status for the setup payload's hash so the mock returns VALID without running the generator.

The vector passes on minimal and mainnet with #10061 applied.

@jimmygchen
jimmygchen marked this pull request as ready for review September 25, 2026 04:10

@pawanjay176 pawanjay176 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Ohh nice! you could have just pushed to my branch lol

@pawanjay176
pawanjay176 merged commit 80044ad into pawanjay176:gloas-gossip-handlers Sep 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants