Skip to content

Commit

Permalink
chore(blockifier_reexecution): add blockifier reexecution test to ci
Browse files Browse the repository at this point in the history
  • Loading branch information
aner-starkware committed Nov 18, 2024
1 parent 846e4ea commit 0926455
Show file tree
Hide file tree
Showing 4 changed files with 60 additions and 4 deletions.
3 changes: 0 additions & 3 deletions .github/workflows/blockifier_ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,6 @@ jobs:
# No features - build blockifier without features activated by dependencies in the workspace.
- run: cargo build -p blockifier
- run: cargo test -p blockifier
# transaction_serde is not activated by any workspace crate; test the build.
- run: cargo build -p blockifier --features transaction_serde
- run: cargo test -p blockifier --features transaction_serde
# cairo_native is not activated by any workspace crate; test the build.
- run: cargo build -p blockifier --features cairo_native
- run: cargo test -p blockifier --features cairo_native
57 changes: 57 additions & 0 deletions .github/workflows/blockifier_reexecution_ci.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
name: Blockifier-Reexecution-CI

on:
push:
branches:
- main
- main-v[0-9].**
tags:
- v[0-9].**
# TODO(Dori, 1/9/2024): Decide when exactly native-blockifier artifacts will be built. Until
# then, keep the 'paths' key empty and build on every push to a release branch / tag.

pull_request:
types:
- opened
- reopened
- synchronize
- auto_merge_enabled
- edited
paths:
# Other than code-related changes, all changes related to the blockifier should trigger the build.
- 'crates/blockifier_reexecution/**'
- '.github/workflows/blockifier_reexecution_ci.yml'
- '.github/workflows/blockifier_ci.yml'
- '.github/workflows/upload_artifacts_workflow.yml'
- 'build_native_in_docker.sh'
- 'Cargo.lock'
- 'Cargo.toml'
- 'crates/blockifier/**'
- 'crates/native_blockifier/**'
- 'scripts/build_native_blockifier.sh'
- 'scripts/dependencies.sh'
- 'scripts/install_build_tools.sh'
- 'scripts/sequencer-ci.Dockerfile'

# On PR events, cancel existing CI runs on this same PR for this workflow.
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}-${{ github.job }}
cancel-in-progress: ${{ github.event_name == 'pull_request' }}

jobs:
blockifier_reexecution:
runs-on: starkware-ubuntu-latest-medium
if: ${{ github.event_name == 'pull_request' }}
steps:
- uses: actions/checkout@v4
- uses: ./.github/actions/bootstrap
# Download the blockifier re-execution test data.
- id: auth
uses: "google-github-actions/auth@v2"
with:
credentials_json: ${{ secrets.SA_REEXECUTION_ARTIFACTS_READ_ACCESS_KEY }}
- uses: 'google-github-actions/setup-gcloud@v2'
- run: echo "REEXECUTION_INPUT_FILES_PREFIX=$(cat ./crates/blockifier_reexecution/src/resources/flow_test_files_prefix)" >> $GITHUB_ENV
- run: gcloud storage cp -r gs://blockifier-reexecution-artifacts/$REEXECUTION_INPUT_FILES_PREFIX/* ./crates/blockifier_reexecution/resources/
# Run blockifier re-execution tests.
- run: cargo test --release --ignored -p blockifier_reexecution
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
f31dc64d6
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,8 @@ fn serialize_state_maps() {
#[rstest]
// TODO(Aner): Add block for each starknet version and for declare, deploy, replace_class, etc.
#[case::v_0_13_0(600001)]
#[case::v_0_13_1(620978)]
// Commented out until fix integrated.
// #[case::v_0_13_1(620978)]
#[case::v_0_13_1_1(649367)]
#[case::v_0_13_2(685878)]
#[case::v_0_13_2_1(700000)]
Expand Down

0 comments on commit 0926455

Please sign in to comment.