Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test: devnet block building #38

Merged
merged 7 commits into from
Oct 17, 2024
Merged

test: devnet block building #38

merged 7 commits into from
Oct 17, 2024

Conversation

0xOsiris
Copy link
Collaborator

@0xOsiris 0xOsiris commented Oct 16, 2024 •
edited
Loading

Adds a binary to assert specific behavior on the devnet:

  • Block Building -- Simply sends 30 valid pbh transactions to the builder, and asserts inclusion on all transaction.
    • This additionally validates the Builder's payloads are favored over the sequencer on the network.
  • Fallback -- This asserts that even in the case when the builder is MIA the chain progresses falling back on the sequencer.

A question I have is if we want to put this in CI? I am pretty sure we will need a larger actions runner if we do

@0xOsiris 0xOsiris changed the title Osiris/tests test: devnet block building Oct 16, 2024
0xForerunner
0xForerunner approved these changes Oct 17, 2024
View reviewed changes
Copy link
Contributor

@0xForerunner 0xForerunner left a comment

Choose a reason for hiding this comment

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

Looks good! I think it would be awesome to get this int CI. Might work as is, who knows.

@0xOsiris
Copy link
Collaborator Author

Looks good! I think it would be awesome to get this int CI. Might work as is, who knows.

Okay cool! I'm going to merge this in, and I'll isolate that into it own PR

@0xOsiris 0xOsiris merged commit a5af850 into main Oct 17, 2024
3 checks passed
@0xOsiris 0xOsiris deleted the osiris/tests branch October 17, 2024 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xForerunner 0xForerunner 0xForerunner approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@0xOsiris @0xForerunner