-
Notifications
You must be signed in to change notification settings - Fork 2
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.
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
Add smoke test for altlayer config #59
base: integration
Are you sure you want to change the base?
Conversation
895f653
to
e473d68
Compare
altlayer-test-node.bash
Outdated
@@ -0,0 +1,193 @@ | |||
#!/usr/bin/env bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How hard would it be to not duplicate this bash script and the docker compose file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldnt be hard, I can do that!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4a31b13
to
61b5ddc
Compare
61b5ddc
to
b41364d
Compare
|
||
# Sending L2 transaction through the full-node's api | ||
user=user_l2user | ||
./test-node.bash script send-l2 --ethamount 100 --to $user --wait |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think by default the L2 url is set to the sequencer, not the full node
nitro-testnode/scripts/index.ts
Line 29 in b33fc8d
l2url: { string: true, default: "ws://sequencer:8548" }, |
Closes #56
#56
This PR:
Adds a smoke test with altlayer configuration
This PR does not:
Key places to review: