Sitelet https://github.com/init4tech/builder/pull/219
Skip to content

tests: adds builder simulation test harness - #219

Merged
dylanlott merged 11 commits into
mainfrom
dylan/sim-tests
Apr 8, 2026
Merged

tests: adds builder simulation test harness#219
dylanlott merged 11 commits into
mainfrom
dylan/sim-tests

Conversation

@dylanlott

@dylanlott dylanlott commented Feb 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

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

dylanlott commented Feb 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

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

dylanlott commented Feb 10, 2026

Copy link
Copy Markdown
Contributor Author

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

@prestwich prestwich left a comment

Copy link
Copy Markdown
Member

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

prestwich commented Feb 10, 2026

Copy link
Copy Markdown
Member

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

dylanlott commented Feb 16, 2026

Copy link
Copy Markdown
Contributor Author

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
@graphite-app
graphite-app Bot changed the base branch from graphite-base/219 to main February 20, 2026 07:09
@dylanlott
dylanlott force-pushed the dylan/sim-tests branch 2 times, most recently from c601de3 to f8e58c5 Compare February 25, 2026 21:39
@dylanlott dylanlott mentioned this pull request Mar 4, 2026
Merged
@dylanlott
dylanlott requested a review from Fraser999 March 11, 2026 06:15
Fraser999
Fraser999 reviewed Mar 11, 2026
View reviewed changes

@Fraser999 Fraser999 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should probably gate the whole test_utils mod behind #[cfg(test)].

Comment thread src/test_utils/block.rs
Comment thread src/test_utils/block.rs Outdated
Evalir
Evalir reviewed Mar 11, 2026
View reviewed changes
Comment thread src/test_utils/block.rs Outdated
Comment thread src/test_utils/env.rs
@Fraser999 Fraser999 mentioned this pull request Mar 12, 2026
Merged
4 tasks done
@prestwich

prestwich commented Mar 17, 2026

Copy link
Copy Markdown
Member

[Claude Code]

Linking to Linear: ENG-1386 — Create a test harness for the builder

@dylanlott Graphite App

dylanlott commented Mar 25, 2026

Copy link
Copy Markdown
Contributor Author

@Fraser999 I gated the test_utils mod behind #[cfg(test)] and have updated the other tests to match the convention instead of using the previous ignored convention.

@dylanlott
dylanlott force-pushed the dylan/sim-tests branch 2 times, most recently from 17e1a5b to f9b1c97 Compare March 27, 2026 19:56
@dylanlott
dylanlott requested a review from Evalir March 30, 2026 22:50
@dylanlott
dylanlott merged commit 50f8444 into main Apr 8, 2026
6 checks passed
@dylanlott
dylanlott deleted the dylan/sim-tests branch April 8, 2026 04:06
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
@Fraser999 Fraser999 Fraser999 approved these changes
@Evalir Evalir Awaiting requested review from Evalir Evalir is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dylanlott @prestwich @Fraser999 @Evalir