Skip to content

Switch integration tests to OffchainLabs/arbitrum-testnode action#629

Merged
douglance merged 16 commits into
mainfrom
dl/use-arbitrum-testnode
Jun 30, 2026
Merged

Switch integration tests to OffchainLabs/arbitrum-testnode action#629
douglance merged 16 commits into
mainfrom
dl/use-arbitrum-testnode

Conversation

@douglance

@douglance douglance commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replace OffchainLabs/actions/run-nitro-test-node with OffchainLabs/arbitrum-testnode@v0.1.0
  • Uses pre-built snapshot Docker images instead of building nitro-testnode from source
  • fee-token-decimals input replaces --l3-fee-token args
  • Network config copied from exported env var instead of docker exec into container

Test plan

  • All 10 integration test matrix entries pass (l2-only, l3-eth, l3-custom-16/18/20, each on Node 18+20)

Replace OffchainLabs/actions/run-nitro-test-node with the new
snapshot-backed OffchainLabs/arbitrum-testnode action. Uses pre-built
Docker images instead of building from source, cutting testnode
startup from ~20min to ~30sec.

- fee-token-decimals input replaces --l3-fee-token args
- Network config exported via env var instead of docker exec
@cla-bot cla-bot Bot added the cla-signed label Mar 18, 2026
douglance and others added 14 commits March 18, 2026 17:04
Replace polling-based waitUntilReadyToExecute (500ms interval, 5-7 RPC
calls per poll) with a test utility that listens for NodeConfirmed /
AssertionConfirmed events and resolves immediately. Also reduce
mineUntilStop interval from 15s to 1s.
Use the migrated action (l3-node -> l3-enabled input rename) and the
nc-format fast snapshot image dl-ci-fast-2 from the arbitrum-testnode-ci
package.
# Conflicts:
#	packages/sdk/tests/integration/custom-fee-token/customFeeTokenEthBridger.test.ts
#	packages/sdk/tests/integration/eth.test.ts
#	packages/sdk/tests/integration/testHelpers.ts
Adds GHSA-vxpw-j846-p89q, GHSA-p88m-4jfj-68fv, GHSA-35p6-xmwp-9g52,
GHSA-g8m3-5g58-fq7m. All are dev-only via hardhat>undici (pinned 5.x);
fixes require undici >=6.27.0 which hardhat does not yet allow.
@douglance douglance changed the base branch from main to dl/audit-allowlist-undici-6.27 June 23, 2026 13:31
Base automatically changed from dl/audit-allowlist-undici-6.27 to main June 24, 2026 15:29
@douglance douglance merged commit 7948889 into main Jun 30, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants