From 7b9000222601adaec22658b1709c072e45242c65 Mon Sep 17 00:00:00 2001
From: Aner Ben Efraim <aner@starkware.co>
Date: Wed, 6 Nov 2024 14:14:37 +0200
Subject: [PATCH] chore(blockifier_reexecution): add blockifier reexecution
 test to ci

---
 .github/workflows/blockifier_ci.yml           |  3 -
 .../workflows/blockifier_reexecution_ci.yml   | 57 +++++++++++++++++++
 .../offline_reexecution_files_prefix          |  1 +
 .../src/state_reader/raw_rpc_json_test.rs     |  3 +-
 4 files changed, 60 insertions(+), 4 deletions(-)
 create mode 100644 .github/workflows/blockifier_reexecution_ci.yml
 create mode 100644 crates/blockifier_reexecution/resources/offline_reexecution_files_prefix

diff --git a/.github/workflows/blockifier_ci.yml b/.github/workflows/blockifier_ci.yml
index ec3655e2bf9..058cd3abe7d 100644
--- a/.github/workflows/blockifier_ci.yml
+++ b/.github/workflows/blockifier_ci.yml
@@ -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
diff --git a/.github/workflows/blockifier_reexecution_ci.yml b/.github/workflows/blockifier_reexecution_ci.yml
new file mode 100644
index 00000000000..7394a559c56
--- /dev/null
+++ b/.github/workflows/blockifier_reexecution_ci.yml
@@ -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/resources/offline_reexecution_files_prefix)" >> $GITHUB_ENV
+      - run: gcloud storage cp -r gs://reexecution_artifacts/$REEXECUTION_INPUT_FILES_PREFIX/offline_reexecution_files_prefix/* ./crates/blockifier_reexecution/resources/
+      # Run blockifier re-execution tests.
+      - run: cargo test --release --ignored -p blockifier_reexecution
diff --git a/crates/blockifier_reexecution/resources/offline_reexecution_files_prefix b/crates/blockifier_reexecution/resources/offline_reexecution_files_prefix
new file mode 100644
index 00000000000..57c9cac268f
--- /dev/null
+++ b/crates/blockifier_reexecution/resources/offline_reexecution_files_prefix
@@ -0,0 +1 @@
+f31dc64d6
diff --git a/crates/blockifier_reexecution/src/state_reader/raw_rpc_json_test.rs b/crates/blockifier_reexecution/src/state_reader/raw_rpc_json_test.rs
index f79d1046856..1757fc49ca6 100644
--- a/crates/blockifier_reexecution/src/state_reader/raw_rpc_json_test.rs
+++ b/crates/blockifier_reexecution/src/state_reader/raw_rpc_json_test.rs
@@ -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)]