-
Notifications
You must be signed in to change notification settings - Fork 15
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
move starknet settlement to zaun #125
Merged
+3,098
−210
Merged
Changes from 105 commits
Commits
Show all changes
109 commits
Select commit
Hold shift + click to select a range
b4abf82
update dependencies
byteZorvin 827fed4
update imports
byteZorvin c2e9bf1
updated changelog
byteZorvin 47b7c64
added testcase for bytes conversion
byteZorvin 35a02ca
fix clippy error
byteZorvin 1fefb12
use zaun for starknet
byteZorvin db67889
Merge branch 'dependency-bump' into zaun
byteZorvin c674a6a
add simple test for settlement
byteZorvin a5a6e18
add sepolia issue
byteZorvin 4aa76ba
to be fixed: validation error
byteZorvin 003b3ae
add madara update state test
byteZorvin d6b5e67
use relative path
byteZorvin 01b4061
add ci for settlement client
byteZorvin 59185d3
fix workflow debug
byteZorvin 1d590cf
fix workflow debug
byteZorvin 4ebfb4e
add scarb build
byteZorvin e150e0c
ignore flaky test
byteZorvin 2a074d0
fix threads issue
byteZorvin d2c5729
add scarb build for madara
byteZorvin c575e7b
check if tool version present
byteZorvin ac4cda5
fix workflow debug
byteZorvin fc7be88
fix workflwo
byteZorvin b8eab20
fix directory
byteZorvin 7daf808
should fix it
byteZorvin 8fb2450
fix bash command
byteZorvin a2b536e
fix bash command
byteZorvin 0d0fefe
make scarb installation separate step
byteZorvin 99dd1dd
use swm action
byteZorvin 56c4144
fix ci path
byteZorvin 66ebcd5
lint fixed
byteZorvin 09e82b6
fix env path for tests
byteZorvin fc44027
fix lint
byteZorvin d539edb
remove extra path
byteZorvin 16f934d
try checking structure
byteZorvin df4e955
check
byteZorvin 1a14f38
fix env path
byteZorvin 328fa0c
fix the directory
byteZorvin c49cb70
try go into
byteZorvin 2a2a1dd
fix path
byteZorvin c8c9a53
remove working directory
byteZorvin 4753c2c
final fix
byteZorvin 76c3f59
fix: gets triggered multiple times
byteZorvin 08261c4
fix path
byteZorvin 577b327
fix prettier
byteZorvin e6eaf2e
change calldata type to (#133)
apoorvsadana e537cae
update workflow
byteZorvin b10bb6a
Merge branch 'zaun' of https://github.com/madara-alliance/madara-orch…
byteZorvin dc22f6e
fix lints
byteZorvin 9a075cd
Update crates/settlement-clients/starknet/src/tests/test.rs
byteZorvin 58d007f
resolve comments
byteZorvin ec2017d
fix e2e test flow
byteZorvin d629354
fix toml lint
byteZorvin 7dbeccb
Update crates/settlement-clients/starknet/src/tests/test.rs
byteZorvin 1bbf8dc
changed envs
byteZorvin 58f323e
Merge branch 'zaun' of https://github.com/madara-alliance/madara-orch…
byteZorvin 4ba8d6f
add conversion function and test
byteZorvin 1c39968
remove redundant test
byteZorvin dd0a579
update workflow commit hash
byteZorvin 975ea22
change commit and add prettierignore
byteZorvin fde129e
Merge branch 'main' into zaun
byteZorvin 93a7aee
nonce implementaion and optimise code
byteZorvin 3fd9dc2
remove coverage bin env
byteZorvin 3788498
changed to logs
byteZorvin db2df7a
added preset file
byteZorvin 0a57792
update versioned constants path
byteZorvin f9203e5
fix current_path
byteZorvin 767d766
should do inplace replacement
byteZorvin 81ab43e
fix base path
byteZorvin 781ae30
fix final coverage workflow
byteZorvin ed64847
update preset
byteZorvin 51d0345
change runner
byteZorvin 06089b9
install dependencies
byteZorvin 8880c4e
run linter
byteZorvin 29645db
remove path
byteZorvin 5c8941e
reword
byteZorvin 5ca8d3a
change default rust
byteZorvin 31faee0
rerun
byteZorvin 144ac31
install libclang
byteZorvin 86190f9
install more dependencies
byteZorvin ee0a88f
install yq
byteZorvin b2e1562
install via snap
byteZorvin 8bc8e03
install snap first
byteZorvin f9fe926
add y flag
byteZorvin 06f23ec
no y flag
byteZorvin ed0d204
enable snapd service
byteZorvin 2fca4d1
use gh action for yq
byteZorvin 73c9221
yq action
byteZorvin bdeedbf
remove working-directory
byteZorvin 85d711e
check env variable
byteZorvin a1ec248
fix ci
byteZorvin af28cb2
remove override in e2e
byteZorvin cfd2389
dotenvy::from_filename
byteZorvin 393d417
should also build
byteZorvin 3e8f485
env override
byteZorvin 7b9acfa
ignore flaky test
byteZorvin ec9c600
restart ci
byteZorvin 8c2848f
enable cargo build
byteZorvin 029b27f
override in starknet client
byteZorvin dd96afe
only starknet
byteZorvin be9ee77
retry running
byteZorvin cb68ba0
test settle should work
byteZorvin 5dc682b
remove fail fast
byteZorvin 7a9ca1a
update blocktime
byteZorvin 3a80afa
change commit
byteZorvin 29a6ceb
run all tests
byteZorvin cf5f53e
resolve comments
byteZorvin aaad425
added test for get nonce
byteZorvin 13ee0e1
Merge branch 'main' into zaun
byteZorvin 4e68523
remove log
byteZorvin 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
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
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
|
@@ -8,7 +8,7 @@ on: | |||||||||
|
||||||||||
jobs: | ||||||||||
coverage: | ||||||||||
runs-on: ubuntu-latest | ||||||||||
runs-on: karnot-arc-runner-set | ||||||||||
|
||||||||||
services: | ||||||||||
localstack: | ||||||||||
|
@@ -30,14 +30,18 @@ jobs: | |||||||||
|
||||||||||
# selecting a toolchain either by action or manual `rustup` calls should happen | ||||||||||
# before the plugin, as the cache uses the current rustc version as its cache key | ||||||||||
- run: rustup show | ||||||||||
- name: Install necessary dependencies | ||||||||||
run: | | ||||||||||
sudo apt update | ||||||||||
sudo apt-get install -y clang llvm libudev-dev protobuf-compiler gcc g++ build-essential libssl-dev pkg-config curl wget git libclang-dev | ||||||||||
|
||||||||||
- name: Install Rust toolchain | ||||||||||
uses: actions-rs/toolchain@v1 | ||||||||||
with: | ||||||||||
profile: minimal | ||||||||||
toolchain: stable | ||||||||||
override: true | ||||||||||
default: true | ||||||||||
|
||||||||||
- name: Rust Cache | ||||||||||
uses: Swatinem/rust-cache@v2 | ||||||||||
|
@@ -62,14 +66,42 @@ jobs: | |||||||||
- name: Install nextest | ||||||||||
uses: taiki-e/install-action@nextest | ||||||||||
|
||||||||||
- name: Setup scarb | ||||||||||
uses: software-mansion/setup-scarb@v1 | ||||||||||
with: | ||||||||||
scarb-version: "2.8.2" | ||||||||||
|
||||||||||
- name: Build Mock Contracts using Scarb | ||||||||||
run: | | ||||||||||
cd crates/settlement-clients/starknet/src/tests/mock_contracts | ||||||||||
scarb fmt --check | ||||||||||
scarb build | ||||||||||
|
||||||||||
- name: Checkout Madara | ||||||||||
uses: actions/checkout@v4 | ||||||||||
with: | ||||||||||
repository: madara-alliance/madara # Replace with your first repository | ||||||||||
ref: "0c821aecd1cafc516d00548473fcaddaa4dc0ce4" # Replace with your branch name | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||
path: madara | ||||||||||
|
||||||||||
- name: Build Madara | ||||||||||
working-directory: madara | ||||||||||
run: | | ||||||||||
cargo build | ||||||||||
mv target/debug/madara ../madara-binary | ||||||||||
cd .. | ||||||||||
echo -e " | ||||||||||
MADARA_BINARY_PATH=\"$(pwd)/madara-binary\"" >> .env.test | ||||||||||
cat .env.test | ||||||||||
|
||||||||||
- name: Getting neccesary files for testing | ||||||||||
run: | | ||||||||||
wget -P ./crates/prover-services/sharp-service/tests/artifacts https://madara-orchestrator-sharp-pie.s3.amazonaws.com/238996-SN.zip | ||||||||||
|
||||||||||
- name: Run llvm-cov tests | ||||||||||
env: | ||||||||||
ETHEREUM_BLAST_RPC_URL: ${{ secrets.ETHEREUM_BLAST_RPC_URL }} | ||||||||||
run: RUST_LOG=debug RUST_BACKTRACE=1 cargo llvm-cov nextest --release --features testing --lcov --output-path lcov.info --test-threads=1 --workspace --exclude=e2e-tests | ||||||||||
run: RUST_LOG=debug RUST_BACKTRACE=1 cargo llvm-cov nextest --release --features testing --lcov --output-path lcov.info --test-threads=1 --workspace --exclude=e2e-tests --no-fail-fast | ||||||||||
|
||||||||||
- name: Coveralls | ||||||||||
uses: coverallsapp/github-action@v2 | ||||||||||
|
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 |
---|---|---|
|
@@ -6,6 +6,7 @@ | |
*.code-workspace | ||
.vscode | ||
|
||
|
||
lcov.info | ||
|
||
**/*-SN.zip | ||
**/*-SN.zip |
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 @@ | ||
target/ |
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.