Skip to content

feat: add bootstrapped Tempo localnet instance - #105

Merged
tmm merged 4 commits into
wevm:mainfrom
brendanjryan:feat/tempo-localnet
Aug 24, 2026
Merged

tmm merged 4 commits into
wevm:mainfrom
brendanjryan:feat/tempo-localnet

Conversation

@brendanjryan

Copy link
Copy Markdown
Contributor

Motivation

Consumers need a reusable Testcontainers instance for Tempo’s fully bootstrapped localnet image without recreating node setup.

Summary

  • add Instance.tempoLocalnet with health-based readiness and dynamic host ports
  • expose bare and block-time controls with configurable image, logging, and startup timeout
  • cover configuration and real container startup/faucet behavior

Key design considerations

  • keep the existing raw Instance.tempo path for custom node and hardfork setups
  • respect the localnet image’s fixed internal RPC port and entrypoint contract
  • isolate the Docker smoke test from concurrent process tests

@pkg-pr-new

pkg-pr-new Bot commented Aug 21, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/prool@105

commit: 91dba39

@brendanjryan
brendanjryan marked this pull request as ready for review August 21, 2026 15:30
@brendanjryan

Copy link
Copy Markdown
Contributor Author

cc @jxom @tmm

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 91dba3951f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +65 to +67
pnpm test --bail=1 \
--exclude src/instances/alto.test.ts \
--testNamePattern="^(?!.*instance: alto)(?!.*instance: 'alto')"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Avoid loading forked suites when the RPC secret is absent

When VITE_FORK_URL is unavailable, this name filter skips the Alto test bodies but does not prevent their files' module-level setup from running: src/Pool.test.ts and src/Server.test.ts both start a forked Anvil before tests are filtered. Those suites therefore still contact the public fallback RPC and can fail or be rate-limited before the skipped tests are considered, so the no-secret CI path remains externally fork-dependent; make those setup hooks conditional or refactor/exclude the fork-dependent suites themselves.

Useful? React with 👍 / 👎.

uses: ./.github/actions/install-dependencies

- name: Run Tempo localnet tests
run: pnpm test --run src/testcontainers/tempoLocalnet.test.ts

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.

could be nice to setup vitest projects so we can exclude (like alto) or run specifics suite like tempo localnet more easily. (can do some other time so we can get this out.)

@tmm
tmm merged commit a55a8cf into wevm:main Aug 24, 2026
5 checks passed
@github-actions github-actions Bot mentioned this pull request Aug 24, 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