Skip to content
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.

Sign up for GitHub

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

Jump to bottom

test: integration test CICD #137

Merged
merged 125 commits into from
Feb 5, 2024
Merged

test: integration test CICD #137

merged 125 commits into from
Feb 5, 2024

Conversation

JuArce
Copy link
Collaborator

@JuArce JuArce commented Feb 2, 2024

This integration test is a github action job that make the following steps

  • Run locally Anvil and Katana for Ethereum and Starknet blockchains respectively.
  • Deploy L1>L2 messaging contracts.
  • Deploy YABTransfer contract.
  • Deploy Escrow contract.
  • Setup MM wallet.
  • Create a set_order.
  • Make a transfer from MM wallet.
  • Make a withdraw from MM wallet.
  • Validate that balances are correct

dubzn and others added 30 commits January 15, 2024 14:18
@dubzn
Upgradeable - Escrow (#35)
71b964f 
* Starknet escrow script + make build, clean & test

* add ethereum escrow deploy + documentation

* rename commands Makefile

* update makefile + readme

* fix makefile clean commands

* OZ dependency + escrow impl

* fix import

* forge install: openzeppelin-contracts

v5.0.1

* fix feedback comments

* fix wrong env name (thx @uri-99)

* refactor deploy.sh + add upgrade.sh (wip)

* refactor deploy.sh & upgrade.sh

* add EscrowV2 for test upgradeable

* add upgradeable tests

* revert method name changes

* fix upgrade.sh + refactor

* rename SN_PROXY_ESCROW_ADDRESS to SN_ESCROW_ADDRESS

* fix deploy error + change upgrade log

* eof

* update readme

* revert solidity files changes

* remove "proxy" from echo

* rename starknet rpc env

* resolve comments

* remove openzeppelin-contracts

* rename and move EscrowV2

* fix feedback comments

* update README

* revert .gitmodules change
improve deploy script + docu
598c995
improve upgrade test
3aa4564
@uri-99
Merge remote-tracking branch 'origin/main' into upgradeable-SN
8418443
@taturosati @uri-99
refactor: create contracts without herodotus (#90)
28d46df 
* feat: escrow contract without herodotus

* fix: change ethereum withrawer in mm_bot

* feat: upgreadable Escrow_herodotus contract

* refactor: readme with herodotus

* refactor: remove comment

* refactor: add tmp solution disclaimer on readme

* refactor: rename starknet escrow contract

* refactor: update wording README.md

Co-authored-by: urimihura <[email protected]>

* refactor: remove transfer.sh

* fix: lowercase escrow module

---------

Co-authored-by: urimihura <[email protected]>
fix sed -i bug in mac
2c3f77b
remove rpc export
5818c62
@rcatalan98
refactor: deployment scripts to avoid errors
082e5dd
@rcatalan98
fix: add param for contract addr missing
ae430c8
@JuArce
fix: use . instead of source
e62b558
@JuArce
fix: run starknet-build on starknet-deploy-and-connect
ac2f8df
@JuArce
build: install dependencies in test cicd [WIP]
4e768b2
@JuArce
build: allow run test cicd manually
5a5751a
@JuArce
Merge branch 'main' into 106-chore-add-local-networks-for-starknet-an…
96fa075 
…d-ethereum

# Conflicts:
#	.github/workflows/test.yml
@JuArce
build: set python version to 3.10.12 in test cicd
150ba8c
@JuArce
build: run psql install in sudo mode
7f4862b
@JuArce
build: run katana
39e52ce
@JuArce
build: setup katana account
4f3d0f1
@JuArce
fix: call correct setup katana account script
f6ab21d
@JuArce
fix: run setup_katana_account with sudo
fb468fd
@JuArce
fix: check setup_katana_account permissions
ea5c8d8
@JuArce
fix: add x permission to setup_katana_account
1e9af33
@JuArce
build: install foundry and run anvil
a7c9a89
@JuArce
fix: run source on foundry install
4bc1214
@JuArce
fix: run source on foundry install
14298a3
@JuArce
fix: run source on foundry install
a92b2b7
@JuArce
fix: run source with sudo
b1f95de
@JuArce
fix: use . instead of source
9d3ecc1
@JuArce
fix: login before run source
1cb2947
@JuArce
build: use foundry actions
c3c58b4
JuArce and others added 21 commits January 31, 2024 17:40
@JuArce
refactor(withdraw.sh): remove sleep 60
9a13708
@JuArce
build(test.yml): add assert script to validate final values
7f19894
@JuArce
fix(assert.sh): use eval for condition check
ef12fa5
@JuArce
fix(withdraw.sh): sleep 15 seconds after withdrawal
93cff1c
@JuArce
fix(assert.sh): remove eval operator
e1ea74e
@JuArce
build(test.yml): disable python and psql setup
844a654
@JuArce
fix(assert.sh): use eval operator
e342f98
@JuArce
build(assert.sh): check escrow and mm final balances
092e7fa
@JuArce
build(assert.sh): show l2 balances as integers
5172ca0
@JuArce
build(assert.sh): add new line before asserts
2e0e4e5
@JuArce
refactor(test.yml): improve steps readability
422ff44
@JuArce
fix(test.yml): install forge for starknet messaging contract
8e0f02d
@JuArce
build(test.yml): use .env.test file for ethereum variables
7cdc1b6
@JuArce
fix(test.yml): export ETHERSCAN_API_KEY
5faa50d
@JuArce
fix(test.yml): use . instead of source
697488e
@JuArce
fix(test.yml): do not use .env.test in solidity. It does not work and…
e94ec46 
… I don't know why
@JuArce
chore: delete deploy_erc20.sh script
cbe8cf5
@JuArce
chore: add new line at eof
59132a2
@JuArce
fix: typo in error message
99a15a4
@JuArce
Merge branch 'main' into 106-chore-add-local-networks-for-starknet-an…
1642c3d 
…d-ethereum
@JuArce
refactor: import colors.sh in actions scripts
bd0857c
@JuArce JuArce merged commit 866ba14 into main Feb 5, 2024
1 check passed
@JuArce JuArce deleted the 106-chore-add-local-networks-for-starknet-and-ethereum branch February 5, 2024 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rcatalan98 rcatalan98 rcatalan98 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@JuArce @rcatalan98 @dubzn @uri-99 @taturosati