-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
feat: l2 native token tests #2916
Merged
Merged
Changes from all commits
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
d1144b5
some changes
kelemeno 368d657
Merge branch 'sync-layer-stable' of github.com:matter-labs/zksync-era…
kelemeno 30508cd
bump contracts
kelemeno e355e5c
chain.toml
kelemeno 0357fc6
contracts.toml
kelemeno 291972e
bump contracts with fixes
kelemeno 679f8e7
config
kelemeno 01d7391
Merge branch 'sync-layer-stable' of github.com:matter-labs/zksync-era…
kelemeno db05aba
fixes
kelemeno b90e9c8
bump contracts
kelemeno e8775c7
config
kelemeno 644a076
contracts.ts config
kelemeno bd5ba3d
make CI slightly smaller
kelemeno 886460f
typo
kelemeno 037e7c6
contract env var error
kelemeno b6f0dfa
config
kelemeno 1b261d9
contracts config
kelemeno eceb869
config
kelemeno 16fbac8
Merge branch 'sync-layer-stable' of github.com:matter-labs/zksync-era…
kelemeno 6b96da9
chain config
kelemeno 8a06017
config
kelemeno 0efee21
contracts fix
kelemeno d29dbbd
bump contracts
kelemeno 6dc895f
fixes
kelemeno c4909f7
bump contracts
kelemeno 2bfc739
contracts config
kelemeno 23c7a87
bump contracts
kelemeno db612f8
fixed unit tests
kelemeno a94999d
bump contracts
kelemeno 94edd3c
last test fix
kelemeno d0bd8a9
fmt
kelemeno 170ae00
l2 native token tests
kelemeno 5983a2e
Merge branch 'sync-layer-stable' of github.com:matter-labs/zksync-era…
kelemeno 3b85806
Merge branch 'sync-layer-stable' of github.com:matter-labs/zksync-era…
kelemeno 16735cf
bump contracts
kelemeno 1ab4295
main gw->L1 migration changes
kelemeno 3eaa189
GW->L1 migration config changes
kelemeno 0ddb00f
Merge branch 'sync-layer-stable' of github.com:matter-labs/zksync-era…
kelemeno 0557dde
fixed most tests
kelemeno b362e69
Merge branch 'sync-layer-stable' of github.com:matter-labs/zksync-era…
kelemeno 327ee4d
small test fixes
kelemeno 1a0919e
zks fmt
kelemeno bf85e75
undo test fixes for CI
kelemeno c4e2e71
comment out l2 tests
kelemeno 12a903e
increase time
kelemeno affd930
make l2-erc20 test uncommenting work
kelemeno bfac316
linting
kelemeno a818916
feat: l2 native contract checks integration tests (#3010)
Raid5594 c71d11e
bump contracts
kelemeno fcde977
Merge branch 'sync-layer-stable' of github.com:matter-labs/zksync-era…
kelemeno 0a00512
undo selectors
kelemeno c2b1a4b
cleanup
kelemeno 0042c3e
genesis.yaml
kelemeno b164d6d
stas issues
kelemeno 26c3c3b
linting
kelemeno 3be0246
genesis.yaml
kelemeno 4cb7d52
linting + sdk
kelemeno 7fd6b03
rust unit test
kelemeno 2e9cba9
missing enter
kelemeno 5beb49d
bump contracts
kelemeno 7999679
config
kelemeno 0e1cdcb
bump contracts
kelemeno 3d47744
config
kelemeno File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,50 @@ | ||
sudo rm -rf ./volumes && zk_supervisor clean containers && zk_inception up -o false | ||
|
||
zk_inception ecosystem init --deploy-paymaster --deploy-erc20 \ | ||
--deploy-ecosystem --l1-rpc-url=http://localhost:8545 \ | ||
--server-db-url=postgres://postgres:notsecurepassword@localhost:5432 \ | ||
--server-db-name=zksync_server_localhost_era \ | ||
--prover-db-url=postgres://postgres:notsecurepassword@localhost:5432 \ | ||
--prover-db-name=zksync_prover_localhost_era \ | ||
--ignore-prerequisites --observability=false --skip-submodules-checkout \ | ||
--chain era \ | ||
# --skip-contract-compilation-override \ | ||
|
||
zk_inception chain create \ | ||
--chain-name gateway \ | ||
--chain-id 505 \ | ||
--prover-mode no-proofs \ | ||
--wallet-creation localhost \ | ||
--l1-batch-commit-data-generator-mode rollup \ | ||
--base-token-address 0x0000000000000000000000000000000000000001 \ | ||
--base-token-price-nominator 1 \ | ||
--base-token-price-denominator 1 \ | ||
--set-as-default false \ | ||
--ignore-prerequisites --skip-submodules-checkout --skip-contract-compilation-override | ||
|
||
zk_inception chain init \ | ||
--deploy-paymaster \ | ||
--l1-rpc-url=http://localhost:8545 \ | ||
--server-db-url=postgres://postgres:notsecurepassword@localhost:5432 \ | ||
--server-db-name=zksync_server_localhost_gateway \ | ||
--prover-db-url=postgres://postgres:notsecurepassword@localhost:5432 \ | ||
--prover-db-name=zksync_prover_localhost_gateway \ | ||
--chain gateway --skip-submodules-checkout | ||
|
||
zk_inception chain convert-to-gateway --chain gateway --ignore-prerequisites | ||
|
||
zk_inception server --ignore-prerequisites --chain gateway &> ./gateway.log & | ||
|
||
sleep 20 | ||
|
||
zk_inception chain migrate-to-gateway --chain era --gateway-chain-name gateway | ||
|
||
zk_inception chain migrate-from-gateway --chain era --gateway-chain-name gateway | ||
|
||
zk_inception chain migrate-to-gateway --chain era --gateway-chain-name gateway | ||
|
||
zk_inception server --ignore-prerequisites --chain era &> ./rollup.log & | ||
|
||
sleep 20 | ||
|
||
zk_supervisor test integration --no-deps --ignore-prerequisites --chain era |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Submodule contracts
updated
58 files
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
is this script ever used? storing it in
.github
implies that it is used in CI, but it is not the caseThere 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.
its just system contract hashes, did not bump for that.
I use it for local setup, testing, yes probably this is not the right place, I'll put it in ./bin ?
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.
Can you discuss it with @Deniallugo? I just don't feel this is something we want to leave committed forever, if it is something we will delete before merging to main, then okay let's keep it here, but it maybe it is helpful for the platform team/toolbox in general
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.
We already have things like this in sync-layer-stable, but yes will discuss:
https://github.com/matter-labs/zksync-era/blob/sync-layer-stable/bin/start_sync_layer_l3_chain.sh
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.
He is OOO today, so I messaged him, but I think we can merge before he replies
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.
why do we need such scripts?
actually
zk_supervisor clean containers
already removing volumes and adding one another binary for it seems inappropriateThere 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.
Tbh I think it is very similar to
zki e init --dev
. The difference is that it includes GW.We can either have it as a bash script, or include it into zki e init --dev .