Skip to content

tests: adds builder simulation test harness#219

Open
dylanlott wants to merge 3 commits intomainfrom
dylan/sim-tests
Open

tests: adds builder simulation test harness#219
dylanlott wants to merge 3 commits intomainfrom
dylan/sim-tests

Conversation

@dylanlott
Copy link
Contributor

@dylanlott dylanlott commented Feb 10, 2026 •
edited
Loading

tests: adds builder simulation test harness

  • adds a builder test harness for simulating blocks that doesn't require network
    access and doesn't use alloy for test setup
  • adds test scenario helpers for future setup and use
  • updates claude files with specifics about using the test harness

@dylanlott Graphite App
Copy link
Contributor Author

dylanlott commented Feb 10, 2026 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@dylanlott dylanlott mentioned this pull request Feb 10, 2026
Merged
@dylanlott Graphite App
Copy link
Contributor Author

dylanlott commented Feb 10, 2026

This is a branch that claude had previously hammered out before samwise was introduced. Pushing this up to review so that samwise has a more established framework to add tests with.

prestwich
prestwich reviewed Feb 10, 2026
View reviewed changes
Copy link
Member

@prestwich prestwich left a comment

Choose a reason for hiding this comment

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

i like this direction :)

@prestwich
Copy link
Member

prestwich commented Feb 10, 2026

what of these tools belong in the SDK to help direct testing of sim?

@dylanlott dylanlott force-pushed the dylan/sim-tests branch 3 times, most recently from 5c7c34e to 713e563 Compare February 13, 2026 17:58
@dylanlott dylanlott force-pushed the dylan/claude branch 2 times, most recently from 6f4dc39 to a7df3eb Compare February 13, 2026 19:25
@dylanlott Graphite App
Copy link
Contributor Author

dylanlott commented Feb 16, 2026

what of these tools belong in the SDK to help direct testing of sim?

​Most of this could go into the SDK. The parts I'd keep in the builder specifically are just around the test setup. The SDK has the test-utils crate, which would be a good place for the scenario code and generalized test harness code from this PR, taking steps towards making it more opinionated. The rest of the db and block code would go in signet-sim, I think.

@dylanlott dylanlott changed the base branch from dylan/claude to graphite-base/219 February 20, 2026 07:08
@dylanlott dylanlott marked this pull request as ready for review February 20, 2026 07:08
@dylanlott dylanlott requested a review from Evalir as a code owner February 20, 2026 07:08
@dylanlott
tests: adds builder simulation test harness
08fb862 
- adds a builder test harness for simulating blocks that doesn't require network
access and doesn't use alloy for test setup
- adds test scenario helpers for future setup and use
@dylanlott
fmt
2902953
@graphite-app graphite-app bot changed the base branch from graphite-base/219 to main February 20, 2026 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@prestwich prestwich prestwich left review comments

@Evalir Evalir Awaiting requested review from Evalir Evalir is a code owner

At least 1 approving review is required to merge this pull request.

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

@dylanlott @prestwich

Comments