-
Notifications
You must be signed in to change notification settings - Fork 76
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: add CI for integration test suite for contract upgrades #381
Merged
Merged
Changes from all commits
Commits
Show all changes
81 commits
Select commit
Hold shift + click to select a range
d4c266d
chore: add workspace in package.json
viraj124 61e3417
feat: add docker setup in contracts package
viraj124 a51dc48
chore: add test placeholder contract
viraj124 a32de3f
feat: add upgrade scripts
viraj124 fab154b
chore: enable forking in hardhat config
viraj124 8647dbc
chore: add shell scripts
viraj124 e684961
chore: add additional committer signer in test setup
viraj124 c21c645
feat: add fork integration tests
viraj124 f36aef9
feat: add github action for fork integration tests
viraj124 7b932fb
chore: add build command in workflow
viraj124 2ebc8b7
chore: add docker and rust setup in ci
viraj124 22dafb8
chore: debug ci failure
viraj124 092e1db
chore: fast forward finalization time before relay
viraj124 8768fbb
chore: debug ci failure
viraj124 4a8ac6e
chore: debug ci failure
viraj124 3f56a0d
chore: update portal upgrade script
viraj124 b0e7f05
chore: uncomment code
viraj124 98afdf2
chore: add changeset
viraj124 666ccc6
chore: formatting
viraj124 51f7a9f
chore: update forc version
viraj124 1d53792
chore: debug format check in ci
viraj124 a134435
chore: ts lint fixes
viraj124 417733c
chore: update check shell script
viraj124 2ada6f5
chore: update order of commands in check script
viraj124 430bce2
chore: debug forc formatting
viraj124 da63952
chore: remove --check
viraj124 e933a2e
chore: update forc command
viraj124 08917b8
chore: add build command
viraj124 6348e38
chore: remove forc lock files in ci
viraj124 c4d176b
chore: update installation for forc-fmt
viraj124 8ff85d2
chore: test ci
viraj124 e31d697
chore: test ci
viraj124 396622d
chore: minor update
viraj124 537f38a
chore: small touchup in action.yml
viraj124 861d437
chore: revert workflow changes
viraj124 38ba169
chore: update fmt command
viraj124 37ad5bd
chore: add package.json replacement logic
viraj124 76aa970
chore: revert changes in build script
viraj124 cf1e1e4
chore: revert peer dependencies
viraj124 e3c3e33
chore: add setup for fork integration tests
viraj124 70da686
chore: revert workflow changes
viraj124 f2a469f
chore: remove unused workspace package
viraj124 9c9382d
chore: add integration test in tsconfig import
viraj124 2510749
refactor: use custom integration test helpers
viraj124 3925774
chore: remove unused test helper
viraj124 2d66759
chore: add block committer container env vars
viraj124 6572fac
chore: add comments
viraj124 5208aa6
refactor: add a utils section for integration tests
viraj124 d7ce105
fix
viraj124 2916314
fix: update import
viraj124 0c97237
chore: small touchup
viraj124 a666ed9
chore: simplify commiter role grant script
viraj124 4d954d5
chore: revert erc721 deployment script updates
viraj124 79575c9
chore: remove erc721 gateway from artifacts in docker container
viraj124 5d038a7
refactor: handle original docker setup to handle both fork and local env
viraj124 ab87875
chore: L1 container setup updates
viraj124 88a98b9
chore: remove docker startup script inside solidity-contracts
viraj124 81b7cf0
chore: set consensus secret in docker file
viraj124 f3c2438
refactor: remove docker setup in solidity-contracts package
viraj124 9075b59
chore: add comment
viraj124 f030adf
chore: add docker setup initiation in upgrade test suite workflow
viraj124 9296322
chore: remove workspace dependencies in solidity-contracts
viraj124 52bdd4c
chore: remove package.json in the L1 docker setup
viraj124 01f21a8
refactor: reloacte fork tests under integration-tests package
viraj124 55204c0
chore: remove placeholder contract
viraj124 a32508b
chore: remove integration tests in solidity-contracts package
viraj124 0625167
chore: add fork test run script in root
viraj124 537ca5a
chore: handle erc721 gateway setup for fork tests
viraj124 517b818
chore: remove package manager in package.json
viraj124 95194b8
chore: update test command in upgrade ci workflow
viraj124 b384ca5
chore: relocated upgrade scripts to a different deploy folder
viraj124 7990f2e
chore: revert updates in the original hardhat scripts
viraj124 58e22cf
chore: add forking related logic in hardhat config
viraj124 f4cea4d
chore: formatting
viraj124 92d1023
chore: ts linting
viraj124 bfd2896
chore: remove mocha in solidity-contracts
viraj124 3f0e2bd
chore: remove unused signer
viraj124 37df68c
test: remove zero hash commit logic
viraj124 0ceff15
fix: ts linting
viraj124 aff2ddc
test: remove duplicate code
viraj124 f9b9acb
chore: use fuel provider to fetch block using height
viraj124 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,6 @@ | ||
--- | ||
'@fuel-bridge/solidity-contracts': minor | ||
'@fuel-bridge/test-utils': minor | ||
--- | ||
|
||
ci for contract upgrade test suite |
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,46 @@ | ||
name: Upgrade Test Suite | ||
|
||
on: | ||
push: | ||
branches: | ||
- main | ||
pull_request: | ||
branches: | ||
- main # Target branch for the PR | ||
release: | ||
types: [published] | ||
|
||
concurrency: | ||
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
cancel-in-progress: true | ||
|
||
jobs: | ||
upgrade-test-suite: | ||
runs-on: ubuntu-latest | ||
if: github.event_name == 'pull_request' | ||
steps: | ||
- uses: actions/checkout@v3 | ||
- uses: FuelLabs/github-actions/setups/node@master | ||
with: | ||
node-version: 20.16.0 | ||
pnpm-version: 9.0.6 | ||
- uses: FuelLabs/github-actions/setups/docker@master | ||
with: | ||
username: ${{ github.repository_owner }} | ||
password: ${{ secrets.GITHUB_TOKEN }} | ||
- uses: ./.github/actions/setup-rust | ||
- name: Build project | ||
run: pnpm build | ||
- name: Sets the tenderly rpc endpoint in the L1 docker container env and sets forking variable for fuel core setup | ||
run: | | ||
cat << EOF > l1_chain.env | ||
TENDERLY_RPC_URL=${{ secrets.TENDERLY_RPC_URL }} | ||
EOF | ||
|
||
cat << EOF > fuel_core.env | ||
FORKING=true | ||
EOF | ||
working-directory: docker/envs | ||
- name: Run integration tests on a L1 fork after upgrading contracts | ||
run: | | ||
pnpm run test:fork |
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 |
---|---|---|
@@ -1,5 +1,6 @@ | ||
RUST_BACKTRACE=1 | ||
CONSENSUS_KEY_SECRET="0xa449b1ffee0e2205fa924c6740cc48b3b473aa28587df6dab12abc245d1f5298" | ||
# Uncommend to use the variables by removing # | ||
# FUEL_IP= | ||
# FUEL_PORT= | ||
# set true when running integration tests over forked environment | ||
FORKING=false |
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 safe to expose this key here?
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.
yeah it is a hardhat test account key we already expose it here