mirror of
https://github.com/pezkuwichain/revive-differential-tests.git
synced 2026-04-23 00:17:57 +00:00
Compare commits
21 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 4ad3084fe1 | |||
| 66feb36b4e | |||
| cc753a1a2c | |||
| 31dfd67569 | |||
| a6e4932a08 | |||
| 06c2e023a9 | |||
| 347dcb4488 | |||
| f9a63a5641 | |||
| fb009f65c1 | |||
| dff4c25e24 | |||
| e433d93cbf | |||
| 408754e8fb | |||
| 59bfffe5fe | |||
| 380ea693be | |||
| d02152b565 | |||
| 75159229df | |||
| 2af1a62319 | |||
| e09be4f3fa | |||
| 33b5faca45 | |||
| 172fb4700f | |||
| fefea17c8e |
+139
-164
@@ -18,134 +18,95 @@ env:
|
|||||||
POLKADOT_VERSION: polkadot-stable2506-2
|
POLKADOT_VERSION: polkadot-stable2506-2
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
cache-polkadot:
|
machete:
|
||||||
name: Build and cache Polkadot binaries on ${{ matrix.os }}
|
name: Check for Unneeded Dependencies
|
||||||
runs-on: ${{ matrix.os }}
|
runs-on: ubuntu-24.04
|
||||||
strategy:
|
env:
|
||||||
matrix:
|
SCCACHE_GHA_ENABLED: "true"
|
||||||
os: [ubuntu-24.04, macos-14]
|
RUSTC_WRAPPER: "sccache"
|
||||||
|
|
||||||
steps:
|
steps:
|
||||||
- name: Checkout repo and submodules
|
- name: Checkout This Repository
|
||||||
uses: actions/checkout@v4
|
uses: actions/checkout@v4
|
||||||
with:
|
with:
|
||||||
submodules: recursive
|
submodules: recursive
|
||||||
|
- name: Run Sccache
|
||||||
- name: Install dependencies (Linux)
|
uses: mozilla-actions/sccache-action@v0.0.9
|
||||||
if: matrix.os == 'ubuntu-24.04'
|
- name: Install the Rust Toolchain
|
||||||
run: |
|
uses: actions-rust-lang/setup-rust-toolchain@v1
|
||||||
sudo apt-get update
|
- name: Install the Cargo Make Binary
|
||||||
sudo apt-get install -y protobuf-compiler clang libclang-dev
|
uses: davidB/rust-cargo-make@v1
|
||||||
rustup target add wasm32-unknown-unknown
|
- name: Run Cargo Machete
|
||||||
rustup component add rust-src
|
run: cargo make machete
|
||||||
|
check-fmt:
|
||||||
- name: Install dependencies (macOS)
|
name: Check Formatting
|
||||||
if: matrix.os == 'macos-14'
|
runs-on: ubuntu-24.04
|
||||||
run: |
|
env:
|
||||||
brew install protobuf
|
SCCACHE_GHA_ENABLED: "true"
|
||||||
rustup target add wasm32-unknown-unknown
|
RUSTC_WRAPPER: "sccache"
|
||||||
rustup component add rust-src
|
steps:
|
||||||
|
- name: Checkout This Repository
|
||||||
- name: Cache binaries
|
uses: actions/checkout@v4
|
||||||
id: cache
|
|
||||||
uses: actions/cache@v3
|
|
||||||
with:
|
with:
|
||||||
path: |
|
submodules: recursive
|
||||||
~/.cargo/bin/substrate-node
|
- name: Run Sccache
|
||||||
~/.cargo/bin/eth-rpc
|
uses: mozilla-actions/sccache-action@v0.0.9
|
||||||
key: polkadot-binaries-${{ matrix.os }}-${{ hashFiles('polkadot-sdk/.git') }}
|
- name: Install the Rust Toolchain
|
||||||
|
uses: actions-rust-lang/setup-rust-toolchain@v1
|
||||||
- name: Build substrate-node
|
- name: Install the Cargo Make Binary
|
||||||
if: steps.cache.outputs.cache-hit != 'true'
|
uses: davidB/rust-cargo-make@v1
|
||||||
run: |
|
- name: Run Cargo Formatter
|
||||||
cd polkadot-sdk
|
run: cargo make fmt-check
|
||||||
cargo install --locked --force --profile=production --path substrate/bin/node/cli --bin substrate-node --features cli
|
check-clippy:
|
||||||
|
name: Check Clippy Lints
|
||||||
- name: Build eth-rpc
|
runs-on: ubuntu-24.04
|
||||||
if: steps.cache.outputs.cache-hit != 'true'
|
env:
|
||||||
run: |
|
SCCACHE_GHA_ENABLED: "true"
|
||||||
cd polkadot-sdk
|
RUSTC_WRAPPER: "sccache"
|
||||||
cargo install --path substrate/frame/revive/rpc --bin eth-rpc
|
steps:
|
||||||
|
- name: Checkout This Repository
|
||||||
- name: Cache downloaded Polkadot binaries
|
uses: actions/checkout@v4
|
||||||
id: cache-polkadot
|
|
||||||
uses: actions/cache@v3
|
|
||||||
with:
|
with:
|
||||||
path: |
|
submodules: recursive
|
||||||
~/polkadot-cache/polkadot
|
- name: Run Sccache
|
||||||
~/polkadot-cache/polkadot-execute-worker
|
uses: mozilla-actions/sccache-action@v0.0.9
|
||||||
~/polkadot-cache/polkadot-prepare-worker
|
- name: Install the Rust Toolchain
|
||||||
~/polkadot-cache/polkadot-parachain
|
uses: actions-rust-lang/setup-rust-toolchain@v1
|
||||||
key: polkadot-downloaded-${{ matrix.os }}-${{ env.POLKADOT_VERSION }}
|
- name: Install the Cargo Make Binary
|
||||||
|
uses: davidB/rust-cargo-make@v1
|
||||||
- name: Download Polkadot binaries on macOS
|
- name: Run Cargo Clippy
|
||||||
if: matrix.os == 'macos-14' && steps.cache-polkadot.outputs.cache-hit != 'true'
|
run: cargo make clippy
|
||||||
run: |
|
test:
|
||||||
mkdir -p ~/polkadot-cache
|
name: Unit Tests
|
||||||
curl -sL https://github.com/paritytech/polkadot-sdk/releases/download/${{ env.POLKADOT_VERSION }}/polkadot-aarch64-apple-darwin -o ~/polkadot-cache/polkadot
|
|
||||||
curl -sL https://github.com/paritytech/polkadot-sdk/releases/download/${{ env.POLKADOT_VERSION }}/polkadot-execute-worker-aarch64-apple-darwin -o ~/polkadot-cache/polkadot-execute-worker
|
|
||||||
curl -sL https://github.com/paritytech/polkadot-sdk/releases/download/${{ env.POLKADOT_VERSION }}/polkadot-prepare-worker-aarch64-apple-darwin -o ~/polkadot-cache/polkadot-prepare-worker
|
|
||||||
curl -sL https://github.com/paritytech/polkadot-sdk/releases/download/${{ env.POLKADOT_VERSION }}/polkadot-parachain-aarch64-apple-darwin -o ~/polkadot-cache/polkadot-parachain
|
|
||||||
chmod +x ~/polkadot-cache/*
|
|
||||||
|
|
||||||
- name: Download Polkadot binaries on Ubuntu
|
|
||||||
if: matrix.os == 'ubuntu-24.04' && steps.cache-polkadot.outputs.cache-hit != 'true'
|
|
||||||
run: |
|
|
||||||
mkdir -p ~/polkadot-cache
|
|
||||||
curl -sL https://github.com/paritytech/polkadot-sdk/releases/download/${{ env.POLKADOT_VERSION }}/polkadot -o ~/polkadot-cache/polkadot
|
|
||||||
curl -sL https://github.com/paritytech/polkadot-sdk/releases/download/${{ env.POLKADOT_VERSION }}/polkadot-execute-worker -o ~/polkadot-cache/polkadot-execute-worker
|
|
||||||
curl -sL https://github.com/paritytech/polkadot-sdk/releases/download/${{ env.POLKADOT_VERSION }}/polkadot-prepare-worker -o ~/polkadot-cache/polkadot-prepare-worker
|
|
||||||
curl -sL https://github.com/paritytech/polkadot-sdk/releases/download/${{ env.POLKADOT_VERSION }}/polkadot-parachain -o ~/polkadot-cache/polkadot-parachain
|
|
||||||
chmod +x ~/polkadot-cache/*
|
|
||||||
|
|
||||||
ci:
|
|
||||||
name: CI on ${{ matrix.os }}
|
|
||||||
needs: cache-polkadot
|
|
||||||
runs-on: ${{ matrix.os }}
|
runs-on: ${{ matrix.os }}
|
||||||
|
needs: cache-polkadot
|
||||||
strategy:
|
strategy:
|
||||||
matrix:
|
matrix:
|
||||||
os: [ubuntu-24.04, macos-14]
|
os: [ubuntu-24.04, macos-14]
|
||||||
|
env:
|
||||||
|
SCCACHE_GHA_ENABLED: "true"
|
||||||
|
RUSTC_WRAPPER: "sccache"
|
||||||
|
POLKADOT_SDK_COMMIT_HASH: "30cda2aad8612a10ff729d494acd9d5353294d63"
|
||||||
steps:
|
steps:
|
||||||
- name: Checkout repo
|
- name: Checkout This Repository
|
||||||
uses: actions/checkout@v4
|
uses: actions/checkout@v4
|
||||||
|
|
||||||
- name: Restore binaries from cache
|
|
||||||
uses: actions/cache@v3
|
|
||||||
with:
|
with:
|
||||||
path: |
|
submodules: recursive
|
||||||
~/.cargo/bin/substrate-node
|
- name: Run Sccache
|
||||||
~/.cargo/bin/eth-rpc
|
uses: mozilla-actions/sccache-action@v0.0.9
|
||||||
key: polkadot-binaries-${{ matrix.os }}-${{ hashFiles('polkadot-sdk/.git') }}
|
- name: Install the Rust Toolchain
|
||||||
|
|
||||||
- name: Restore downloaded Polkadot binaries from cache
|
|
||||||
uses: actions/cache@v3
|
|
||||||
with:
|
|
||||||
path: |
|
|
||||||
~/polkadot-cache/polkadot
|
|
||||||
~/polkadot-cache/polkadot-execute-worker
|
|
||||||
~/polkadot-cache/polkadot-prepare-worker
|
|
||||||
~/polkadot-cache/polkadot-parachain
|
|
||||||
key: polkadot-downloaded-${{ matrix.os }}-${{ env.POLKADOT_VERSION }}
|
|
||||||
|
|
||||||
- name: Install Polkadot binaries
|
|
||||||
run: |
|
|
||||||
sudo cp ~/polkadot-cache/polkadot /usr/local/bin/
|
|
||||||
sudo cp ~/polkadot-cache/polkadot-execute-worker /usr/local/bin/
|
|
||||||
sudo cp ~/polkadot-cache/polkadot-prepare-worker /usr/local/bin/
|
|
||||||
sudo cp ~/polkadot-cache/polkadot-parachain /usr/local/bin/
|
|
||||||
sudo chmod +x /usr/local/bin/polkadot*
|
|
||||||
|
|
||||||
- name: Setup Rust toolchain
|
|
||||||
uses: actions-rust-lang/setup-rust-toolchain@v1
|
uses: actions-rust-lang/setup-rust-toolchain@v1
|
||||||
with:
|
with:
|
||||||
rustflags: ""
|
target: "wasm32-unknown-unknown"
|
||||||
|
components: "rust-src,rust-std"
|
||||||
- name: Add wasm32 target and formatting
|
- name: Install the Cargo Make Binary
|
||||||
run: |
|
uses: davidB/rust-cargo-make@v1
|
||||||
rustup target add wasm32-unknown-unknown
|
- name: Caching Step
|
||||||
rustup component add rust-src rustfmt clippy
|
uses: actions/cache@v4
|
||||||
|
with:
|
||||||
|
path: |
|
||||||
|
~/.cargo/bin/eth-rpc
|
||||||
|
~/.cargo/bin/revive-dev-node
|
||||||
|
key: polkadot-binaries-${{ env.POLKADOT_SDK_COMMIT_HASH }}-${{ matrix.os }}
|
||||||
- name: Install Geth on Ubuntu
|
- name: Install Geth on Ubuntu
|
||||||
if: matrix.os == 'ubuntu-24.04'
|
if: matrix.os == 'ubuntu-24.04'
|
||||||
run: |
|
run: |
|
||||||
@@ -178,7 +139,6 @@ jobs:
|
|||||||
curl -sL https://github.com/paritytech/revive/releases/download/v0.3.0/resolc-x86_64-unknown-linux-musl -o resolc
|
curl -sL https://github.com/paritytech/revive/releases/download/v0.3.0/resolc-x86_64-unknown-linux-musl -o resolc
|
||||||
chmod +x resolc
|
chmod +x resolc
|
||||||
sudo mv resolc /usr/local/bin
|
sudo mv resolc /usr/local/bin
|
||||||
|
|
||||||
- name: Install Geth on macOS
|
- name: Install Geth on macOS
|
||||||
if: matrix.os == 'macos-14'
|
if: matrix.os == 'macos-14'
|
||||||
run: |
|
run: |
|
||||||
@@ -190,64 +150,79 @@ jobs:
|
|||||||
curl -sL https://github.com/paritytech/revive/releases/download/v0.3.0/resolc-universal-apple-darwin -o resolc
|
curl -sL https://github.com/paritytech/revive/releases/download/v0.3.0/resolc-universal-apple-darwin -o resolc
|
||||||
chmod +x resolc
|
chmod +x resolc
|
||||||
sudo mv resolc /usr/local/bin
|
sudo mv resolc /usr/local/bin
|
||||||
|
|
||||||
- name: Install Kurtosis on macOS
|
- name: Install Kurtosis on macOS
|
||||||
if: matrix.os == 'macos-14'
|
if: matrix.os == 'macos-14'
|
||||||
run: brew install kurtosis-tech/tap/kurtosis-cli
|
run: brew install kurtosis-tech/tap/kurtosis-cli
|
||||||
|
|
||||||
- name: Install Kurtosis on Ubuntu
|
- name: Install Kurtosis on Ubuntu
|
||||||
if: matrix.os == 'ubuntu-24.04'
|
if: matrix.os == 'ubuntu-24.04'
|
||||||
run: |
|
run: |
|
||||||
echo "deb [trusted=yes] https://apt.fury.io/kurtosis-tech/ /" | sudo tee /etc/apt/sources.list.d/kurtosis.list
|
echo "deb [trusted=yes] https://apt.fury.io/kurtosis-tech/ /" | sudo tee /etc/apt/sources.list.d/kurtosis.list
|
||||||
sudo apt update
|
sudo apt update
|
||||||
sudo apt install kurtosis-cli
|
sudo apt install kurtosis-cli
|
||||||
|
- name: Run Tests
|
||||||
|
run: cargo make test
|
||||||
|
cache-polkadot:
|
||||||
|
name: Build and Cache Polkadot Binaries on ${{ matrix.os }}
|
||||||
|
runs-on: ${{ matrix.os }}
|
||||||
|
strategy:
|
||||||
|
matrix:
|
||||||
|
os: [ubuntu-24.04, macos-14]
|
||||||
|
env:
|
||||||
|
SCCACHE_GHA_ENABLED: "true"
|
||||||
|
RUSTC_WRAPPER: "sccache"
|
||||||
|
RUSTFLAGS: "-Awarnings"
|
||||||
|
POLKADOT_SDK_COMMIT_HASH: "30cda2aad8612a10ff729d494acd9d5353294d63"
|
||||||
|
steps:
|
||||||
|
- name: Caching Step
|
||||||
|
id: cache-step
|
||||||
|
uses: actions/cache@v4
|
||||||
|
with:
|
||||||
|
path: |
|
||||||
|
~/.cargo/bin/eth-rpc
|
||||||
|
~/.cargo/bin/revive-dev-node
|
||||||
|
key: polkadot-binaries-${{ env.POLKADOT_SDK_COMMIT_HASH }}-${{ matrix.os }}
|
||||||
|
- name: Checkout the Polkadot SDK Repository
|
||||||
|
uses: actions/checkout@v4
|
||||||
|
if: steps.cache-step.outputs.cache-hit != 'true'
|
||||||
|
with:
|
||||||
|
repository: paritytech/polkadot-sdk
|
||||||
|
ref: ${{ env.POLKADOT_SDK_COMMIT_HASH }}
|
||||||
|
submodules: recursive
|
||||||
|
- name: Run Sccache
|
||||||
|
uses: mozilla-actions/sccache-action@v0.0.9
|
||||||
|
if: steps.cache-step.outputs.cache-hit != 'true'
|
||||||
|
- name: Install the Rust Toolchain
|
||||||
|
uses: actions-rust-lang/setup-rust-toolchain@v1
|
||||||
|
if: steps.cache-step.outputs.cache-hit != 'true'
|
||||||
|
with:
|
||||||
|
target: "wasm32-unknown-unknown"
|
||||||
|
components: "rust-src"
|
||||||
|
toolchain: "1.90.0"
|
||||||
|
|
||||||
- name: Machete
|
- name: Install dependencies (Linux)
|
||||||
uses: bnjbvr/cargo-machete@v0.7.1
|
if: matrix.os == 'ubuntu-24.04' && steps.cache-step.outputs.cache-hit != 'true'
|
||||||
|
|
||||||
- name: Format
|
|
||||||
run: make format
|
|
||||||
|
|
||||||
- name: Clippy
|
|
||||||
run: make clippy
|
|
||||||
|
|
||||||
- name: Check substrate-node version
|
|
||||||
run: substrate-node --version
|
|
||||||
|
|
||||||
- name: Check eth-rpc version
|
|
||||||
run: eth-rpc --version
|
|
||||||
|
|
||||||
- name: Check resolc version
|
|
||||||
run: resolc --version
|
|
||||||
|
|
||||||
- name: Check polkadot version
|
|
||||||
run: polkadot --version
|
|
||||||
|
|
||||||
- name: Check polkadot-parachain version
|
|
||||||
run: polkadot-parachain --version
|
|
||||||
|
|
||||||
- name: Check polkadot-execute-worker version
|
|
||||||
run: polkadot-execute-worker --version
|
|
||||||
|
|
||||||
- name: Check polkadot-prepare-worker version
|
|
||||||
run: polkadot-prepare-worker --version
|
|
||||||
|
|
||||||
- name: Test Formatting
|
|
||||||
run: make format
|
|
||||||
|
|
||||||
- name: Test Clippy
|
|
||||||
run: make clippy
|
|
||||||
|
|
||||||
- name: Test Machete
|
|
||||||
run: make machete
|
|
||||||
|
|
||||||
- name: Unit Tests
|
|
||||||
if: matrix.os == 'ubuntu-24.04'
|
|
||||||
run: cargo test --workspace -- --nocapture
|
|
||||||
|
|
||||||
# We can't install docker in the MacOS image used in CI and therefore we need to skip the
|
|
||||||
# Kurtosis and lighthouse related tests when running the CI on MacOS.
|
|
||||||
- name: Unit Tests
|
|
||||||
if: matrix.os == 'macos-14'
|
|
||||||
run: |
|
run: |
|
||||||
cargo test --workspace -- --nocapture --skip lighthouse_geth::tests::
|
sudo apt-get update
|
||||||
|
sudo apt-get install -y protobuf-compiler clang libclang-dev
|
||||||
|
- name: Install dependencies (macOS)
|
||||||
|
if: matrix.os == 'macos-14' && steps.cache-step.outputs.cache-hit != 'true'
|
||||||
|
run: |
|
||||||
|
brew install protobuf llvm
|
||||||
|
LLVM_PREFIX="$(brew --prefix llvm)"
|
||||||
|
echo "LDFLAGS=-L${LLVM_PREFIX}/lib" >> "$GITHUB_ENV"
|
||||||
|
echo "CPPFLAGS=-I${LLVM_PREFIX}/include" >> "$GITHUB_ENV"
|
||||||
|
echo "CMAKE_PREFIX_PATH=${LLVM_PREFIX}" >> "$GITHUB_ENV"
|
||||||
|
echo "LIBCLANG_PATH=${LLVM_PREFIX}/lib" >> "$GITHUB_ENV"
|
||||||
|
echo "DYLD_FALLBACK_LIBRARY_PATH=${LLVM_PREFIX}/lib" >> "$GITHUB_ENV"
|
||||||
|
echo "${LLVM_PREFIX}/bin" >> "$GITHUB_PATH"
|
||||||
|
- name: Build Polkadot Dependencies
|
||||||
|
if: steps.cache-step.outputs.cache-hit != 'true'
|
||||||
|
run: |
|
||||||
|
cargo build \
|
||||||
|
--locked \
|
||||||
|
--profile production \
|
||||||
|
--package revive-dev-node \
|
||||||
|
--package pallet-revive-eth-rpc;
|
||||||
|
mv ./target/production/revive-dev-node ~/.cargo/bin
|
||||||
|
mv ./target/production/eth-rpc ~/.cargo/bin
|
||||||
|
chmod +x ~/.cargo/bin/*
|
||||||
|
|||||||
@@ -3,6 +3,7 @@
|
|||||||
.DS_Store
|
.DS_Store
|
||||||
node_modules
|
node_modules
|
||||||
/*.json
|
/*.json
|
||||||
|
*.sh
|
||||||
|
|
||||||
# We do not want to commit any log files that we produce from running the code locally so this is
|
# We do not want to commit any log files that we produce from running the code locally so this is
|
||||||
# added to the .gitignore file.
|
# added to the .gitignore file.
|
||||||
@@ -13,3 +14,4 @@ workdir
|
|||||||
|
|
||||||
!/schema.json
|
!/schema.json
|
||||||
!/dev-genesis.json
|
!/dev-genesis.json
|
||||||
|
!/scripts/*
|
||||||
|
|||||||
@@ -1,6 +1,3 @@
|
|||||||
[submodule "polkadot-sdk"]
|
|
||||||
path = polkadot-sdk
|
|
||||||
url = https://github.com/paritytech/polkadot-sdk.git
|
|
||||||
[submodule "resolc-compiler-tests"]
|
[submodule "resolc-compiler-tests"]
|
||||||
path = resolc-compiler-tests
|
path = resolc-compiler-tests
|
||||||
url = https://github.com/paritytech/resolc-compiler-tests
|
url = https://github.com/paritytech/resolc-compiler-tests
|
||||||
|
|||||||
Generated
+12
@@ -1920,6 +1920,7 @@ dependencies = [
|
|||||||
"anstyle",
|
"anstyle",
|
||||||
"clap_lex",
|
"clap_lex",
|
||||||
"strsim",
|
"strsim",
|
||||||
|
"terminal_size",
|
||||||
]
|
]
|
||||||
|
|
||||||
[[package]]
|
[[package]]
|
||||||
@@ -5656,6 +5657,7 @@ dependencies = [
|
|||||||
"semver 1.0.26",
|
"semver 1.0.26",
|
||||||
"serde",
|
"serde",
|
||||||
"serde_json",
|
"serde_json",
|
||||||
|
"subxt 0.44.0",
|
||||||
"tokio",
|
"tokio",
|
||||||
"tracing",
|
"tracing",
|
||||||
"tracing-appender",
|
"tracing-appender",
|
||||||
@@ -7838,6 +7840,16 @@ dependencies = [
|
|||||||
"winapi-util",
|
"winapi-util",
|
||||||
]
|
]
|
||||||
|
|
||||||
|
[[package]]
|
||||||
|
name = "terminal_size"
|
||||||
|
version = "0.4.2"
|
||||||
|
source = "registry+https://github.com/rust-lang/crates.io-index"
|
||||||
|
checksum = "45c6481c4829e4cc63825e62c49186a34538b7b2750b73b266581ffb612fb5ed"
|
||||||
|
dependencies = [
|
||||||
|
"rustix",
|
||||||
|
"windows-sys 0.59.0",
|
||||||
|
]
|
||||||
|
|
||||||
[[package]]
|
[[package]]
|
||||||
name = "thiserror"
|
name = "thiserror"
|
||||||
version = "1.0.69"
|
version = "1.0.69"
|
||||||
|
|||||||
+1
-1
@@ -26,7 +26,7 @@ ansi_term = "0.12.1"
|
|||||||
anyhow = "1.0"
|
anyhow = "1.0"
|
||||||
bson = { version = "2.15.0" }
|
bson = { version = "2.15.0" }
|
||||||
cacache = { version = "13.1.0" }
|
cacache = { version = "13.1.0" }
|
||||||
clap = { version = "4", features = ["derive"] }
|
clap = { version = "4", features = ["derive", "wrap_help"] }
|
||||||
dashmap = { version = "6.1.0" }
|
dashmap = { version = "6.1.0" }
|
||||||
foundry-compilers-artifacts = { version = "0.18.0" }
|
foundry-compilers-artifacts = { version = "0.18.0" }
|
||||||
futures = { version = "0.3.31" }
|
futures = { version = "0.3.31" }
|
||||||
|
|||||||
@@ -1,15 +0,0 @@
|
|||||||
.PHONY: format clippy test machete
|
|
||||||
|
|
||||||
format:
|
|
||||||
cargo fmt --all -- --check
|
|
||||||
|
|
||||||
clippy:
|
|
||||||
cargo clippy --all-features --workspace -- --deny warnings
|
|
||||||
|
|
||||||
machete:
|
|
||||||
cargo install cargo-machete
|
|
||||||
cargo machete crates
|
|
||||||
|
|
||||||
test: format clippy machete
|
|
||||||
cargo test --workspace -- --nocapture
|
|
||||||
|
|
||||||
@@ -0,0 +1,21 @@
|
|||||||
|
[config]
|
||||||
|
default_to_workspace = false
|
||||||
|
|
||||||
|
[tasks.machete]
|
||||||
|
command = "cargo"
|
||||||
|
args = ["machete", "crates"]
|
||||||
|
install_crate = "cargo-machete"
|
||||||
|
|
||||||
|
[tasks.fmt-check]
|
||||||
|
command = "cargo"
|
||||||
|
args = ["fmt", "--all", "--", "--check"]
|
||||||
|
install_crate = "rustfmt"
|
||||||
|
|
||||||
|
[tasks.clippy]
|
||||||
|
command = "cargo"
|
||||||
|
args = ["clippy", "--all-features", "--workspace", "--", "--deny", "warnings"]
|
||||||
|
install_crate = "clippy"
|
||||||
|
|
||||||
|
[tasks.test]
|
||||||
|
command = "cargo"
|
||||||
|
args = ["test", "--workspace", "--", "--nocapture"]
|
||||||
@@ -9,7 +9,7 @@
|
|||||||
This project compiles and executes declarative smart-contract tests against multiple platforms, then compares behavior (status, return data, events, and state diffs). Today it supports:
|
This project compiles and executes declarative smart-contract tests against multiple platforms, then compares behavior (status, return data, events, and state diffs). Today it supports:
|
||||||
|
|
||||||
- Geth (EVM reference implementation)
|
- Geth (EVM reference implementation)
|
||||||
- Revive Kitchensink (Substrate-based PolkaVM + `eth-rpc` proxy)
|
- Revive Dev Node (Substrate-based PolkaVM + `eth-rpc` proxy)
|
||||||
|
|
||||||
Use it to:
|
Use it to:
|
||||||
|
|
||||||
@@ -39,9 +39,9 @@ This repository contains none of the tests and only contains the testing framewo
|
|||||||
This section describes the required dependencies that this framework requires to run. Compiling this framework is pretty straightforward and no additional dependencies beyond what's specified in the `Cargo.toml` file should be required.
|
This section describes the required dependencies that this framework requires to run. Compiling this framework is pretty straightforward and no additional dependencies beyond what's specified in the `Cargo.toml` file should be required.
|
||||||
|
|
||||||
- Stable Rust
|
- Stable Rust
|
||||||
- Geth - When doing differential testing against the PVM we submit transactions to a Geth node and to Kitchensink to compare them.
|
- Geth - When doing differential testing against the PVM we submit transactions to a Geth node and to Revive Dev Node to compare them.
|
||||||
- Kitchensink - When doing differential testing against the PVM we submit transactions to a Geth node and to Kitchensink to compare them.
|
- Revive Dev Node - When doing differential testing against the PVM we submit transactions to a Geth node and to Revive Dev Node to compare them.
|
||||||
- ETH-RPC - All communication with Kitchensink is done through the ETH RPC.
|
- ETH-RPC - All communication with Revive Dev Node is done through the ETH RPC.
|
||||||
- Solc - This is actually a transitive dependency, while this tool doesn't require solc as it downloads the versions that it requires, resolc requires that Solc is installed and available in the path.
|
- Solc - This is actually a transitive dependency, while this tool doesn't require solc as it downloads the versions that it requires, resolc requires that Solc is installed and available in the path.
|
||||||
- Resolc - This is required to compile the contracts to PolkaVM bytecode.
|
- Resolc - This is required to compile the contracts to PolkaVM bytecode.
|
||||||
- Kurtosis - The Kurtosis CLI tool is required for the production Ethereum mainnet-like node configuration with Geth as the execution layer and lighthouse as the consensus layer. Kurtosis also requires docker to be installed since it runs everything inside of docker containers.
|
- Kurtosis - The Kurtosis CLI tool is required for the production Ethereum mainnet-like node configuration with Geth as the execution layer and lighthouse as the consensus layer. Kurtosis also requires docker to be installed since it runs everything inside of docker containers.
|
||||||
|
|||||||
Binary file not shown.
@@ -31,10 +31,6 @@ pub enum PlatformIdentifier {
|
|||||||
GethEvmSolc,
|
GethEvmSolc,
|
||||||
/// The Lighthouse Go-ethereum reference full node EVM implementation with the solc compiler.
|
/// The Lighthouse Go-ethereum reference full node EVM implementation with the solc compiler.
|
||||||
LighthouseGethEvmSolc,
|
LighthouseGethEvmSolc,
|
||||||
/// The kitchensink node with the PolkaVM backend with the resolc compiler.
|
|
||||||
KitchensinkPolkavmResolc,
|
|
||||||
/// The kitchensink node with the REVM backend with the solc compiler.
|
|
||||||
KitchensinkRevmSolc,
|
|
||||||
/// The revive dev node with the PolkaVM backend with the resolc compiler.
|
/// The revive dev node with the PolkaVM backend with the resolc compiler.
|
||||||
ReviveDevNodePolkavmResolc,
|
ReviveDevNodePolkavmResolc,
|
||||||
/// The revive dev node with the REVM backend with the solc compiler.
|
/// The revive dev node with the REVM backend with the solc compiler.
|
||||||
@@ -95,8 +91,6 @@ pub enum NodeIdentifier {
|
|||||||
Geth,
|
Geth,
|
||||||
/// The go-ethereum node implementation.
|
/// The go-ethereum node implementation.
|
||||||
LighthouseGeth,
|
LighthouseGeth,
|
||||||
/// The Kitchensink node implementation.
|
|
||||||
Kitchensink,
|
|
||||||
/// The revive dev node implementation.
|
/// The revive dev node implementation.
|
||||||
ReviveDevNode,
|
ReviveDevNode,
|
||||||
/// A zombienet spawned nodes
|
/// A zombienet spawned nodes
|
||||||
|
|||||||
@@ -16,6 +16,7 @@ use alloy::{
|
|||||||
primitives::{B256, FixedBytes, U256},
|
primitives::{B256, FixedBytes, U256},
|
||||||
signers::local::PrivateKeySigner,
|
signers::local::PrivateKeySigner,
|
||||||
};
|
};
|
||||||
|
use anyhow::Context as _;
|
||||||
use clap::{Parser, ValueEnum, ValueHint};
|
use clap::{Parser, ValueEnum, ValueHint};
|
||||||
use revive_dt_common::types::{ParsedTestSpecifier, PlatformIdentifier};
|
use revive_dt_common::types::{ParsedTestSpecifier, PlatformIdentifier};
|
||||||
use semver::Version;
|
use semver::Version;
|
||||||
@@ -24,7 +25,7 @@ use strum::{AsRefStr, Display, EnumString, IntoStaticStr};
|
|||||||
use temp_dir::TempDir;
|
use temp_dir::TempDir;
|
||||||
|
|
||||||
#[derive(Clone, Debug, Parser, Serialize, Deserialize)]
|
#[derive(Clone, Debug, Parser, Serialize, Deserialize)]
|
||||||
#[command(name = "retester")]
|
#[command(name = "retester", term_width = 100)]
|
||||||
pub enum Context {
|
pub enum Context {
|
||||||
/// Executes tests in the MatterLabs format differentially on multiple targets concurrently.
|
/// Executes tests in the MatterLabs format differentially on multiple targets concurrently.
|
||||||
Test(Box<TestExecutionContext>),
|
Test(Box<TestExecutionContext>),
|
||||||
@@ -131,17 +132,6 @@ impl AsRef<PolkadotParachainConfiguration> for Context {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
impl AsRef<KitchensinkConfiguration> for Context {
|
|
||||||
fn as_ref(&self) -> &KitchensinkConfiguration {
|
|
||||||
match self {
|
|
||||||
Self::Test(context) => context.as_ref().as_ref(),
|
|
||||||
Self::Benchmark(context) => context.as_ref().as_ref(),
|
|
||||||
Self::ExportGenesis(context) => context.as_ref().as_ref(),
|
|
||||||
Self::ExportJsonSchema => unreachable!(),
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
impl AsRef<ReviveDevNodeConfiguration> for Context {
|
impl AsRef<ReviveDevNodeConfiguration> for Context {
|
||||||
fn as_ref(&self) -> &ReviveDevNodeConfiguration {
|
fn as_ref(&self) -> &ReviveDevNodeConfiguration {
|
||||||
match self {
|
match self {
|
||||||
@@ -283,10 +273,6 @@ pub struct TestExecutionContext {
|
|||||||
#[clap(flatten, next_help_heading = "Lighthouse Configuration")]
|
#[clap(flatten, next_help_heading = "Lighthouse Configuration")]
|
||||||
pub lighthouse_configuration: KurtosisConfiguration,
|
pub lighthouse_configuration: KurtosisConfiguration,
|
||||||
|
|
||||||
/// Configuration parameters for the Kitchensink.
|
|
||||||
#[clap(flatten, next_help_heading = "Kitchensink Configuration")]
|
|
||||||
pub kitchensink_configuration: KitchensinkConfiguration,
|
|
||||||
|
|
||||||
/// Configuration parameters for the Revive Dev Node.
|
/// Configuration parameters for the Revive Dev Node.
|
||||||
#[clap(flatten, next_help_heading = "Revive Dev Node Configuration")]
|
#[clap(flatten, next_help_heading = "Revive Dev Node Configuration")]
|
||||||
pub revive_dev_node_configuration: ReviveDevNodeConfiguration,
|
pub revive_dev_node_configuration: ReviveDevNodeConfiguration,
|
||||||
@@ -409,10 +395,6 @@ pub struct BenchmarkingContext {
|
|||||||
#[clap(flatten, next_help_heading = "Lighthouse Configuration")]
|
#[clap(flatten, next_help_heading = "Lighthouse Configuration")]
|
||||||
pub lighthouse_configuration: KurtosisConfiguration,
|
pub lighthouse_configuration: KurtosisConfiguration,
|
||||||
|
|
||||||
/// Configuration parameters for the Kitchensink.
|
|
||||||
#[clap(flatten, next_help_heading = "Kitchensink Configuration")]
|
|
||||||
pub kitchensink_configuration: KitchensinkConfiguration,
|
|
||||||
|
|
||||||
/// Configuration parameters for the Polkadot Parachain.
|
/// Configuration parameters for the Polkadot Parachain.
|
||||||
#[clap(flatten, next_help_heading = "Polkadot Parachain Configuration")]
|
#[clap(flatten, next_help_heading = "Polkadot Parachain Configuration")]
|
||||||
pub polkadot_parachain_configuration: PolkadotParachainConfiguration,
|
pub polkadot_parachain_configuration: PolkadotParachainConfiguration,
|
||||||
@@ -491,10 +473,6 @@ pub struct ExportGenesisContext {
|
|||||||
#[clap(flatten, next_help_heading = "Lighthouse Configuration")]
|
#[clap(flatten, next_help_heading = "Lighthouse Configuration")]
|
||||||
pub lighthouse_configuration: KurtosisConfiguration,
|
pub lighthouse_configuration: KurtosisConfiguration,
|
||||||
|
|
||||||
/// Configuration parameters for the Kitchensink.
|
|
||||||
#[clap(flatten, next_help_heading = "Kitchensink Configuration")]
|
|
||||||
pub kitchensink_configuration: KitchensinkConfiguration,
|
|
||||||
|
|
||||||
/// Configuration parameters for the Polkadot Parachain.
|
/// Configuration parameters for the Polkadot Parachain.
|
||||||
#[clap(flatten, next_help_heading = "Polkadot Parachain Configuration")]
|
#[clap(flatten, next_help_heading = "Polkadot Parachain Configuration")]
|
||||||
pub polkadot_parachain_configuration: PolkadotParachainConfiguration,
|
pub polkadot_parachain_configuration: PolkadotParachainConfiguration,
|
||||||
@@ -510,7 +488,7 @@ pub struct ExportGenesisContext {
|
|||||||
|
|
||||||
impl Default for TestExecutionContext {
|
impl Default for TestExecutionContext {
|
||||||
fn default() -> Self {
|
fn default() -> Self {
|
||||||
Self::parse_from(["execution-context"])
|
Self::parse_from(["execution-context", "--test", "."])
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -556,12 +534,6 @@ impl AsRef<KurtosisConfiguration> for TestExecutionContext {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
impl AsRef<KitchensinkConfiguration> for TestExecutionContext {
|
|
||||||
fn as_ref(&self) -> &KitchensinkConfiguration {
|
|
||||||
&self.kitchensink_configuration
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
impl AsRef<ReviveDevNodeConfiguration> for TestExecutionContext {
|
impl AsRef<ReviveDevNodeConfiguration> for TestExecutionContext {
|
||||||
fn as_ref(&self) -> &ReviveDevNodeConfiguration {
|
fn as_ref(&self) -> &ReviveDevNodeConfiguration {
|
||||||
&self.revive_dev_node_configuration
|
&self.revive_dev_node_configuration
|
||||||
@@ -612,7 +584,7 @@ impl AsRef<IgnoreSuccessConfiguration> for TestExecutionContext {
|
|||||||
|
|
||||||
impl Default for BenchmarkingContext {
|
impl Default for BenchmarkingContext {
|
||||||
fn default() -> Self {
|
fn default() -> Self {
|
||||||
Self::parse_from(["benchmarking-context"])
|
Self::parse_from(["benchmarking-context", "--test", "."])
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -658,12 +630,6 @@ impl AsRef<PolkadotParachainConfiguration> for BenchmarkingContext {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
impl AsRef<KitchensinkConfiguration> for BenchmarkingContext {
|
|
||||||
fn as_ref(&self) -> &KitchensinkConfiguration {
|
|
||||||
&self.kitchensink_configuration
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
impl AsRef<ReviveDevNodeConfiguration> for BenchmarkingContext {
|
impl AsRef<ReviveDevNodeConfiguration> for BenchmarkingContext {
|
||||||
fn as_ref(&self) -> &ReviveDevNodeConfiguration {
|
fn as_ref(&self) -> &ReviveDevNodeConfiguration {
|
||||||
&self.revive_dev_node_configuration
|
&self.revive_dev_node_configuration
|
||||||
@@ -718,12 +684,6 @@ impl AsRef<KurtosisConfiguration> for ExportGenesisContext {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
impl AsRef<KitchensinkConfiguration> for ExportGenesisContext {
|
|
||||||
fn as_ref(&self) -> &KitchensinkConfiguration {
|
|
||||||
&self.kitchensink_configuration
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
impl AsRef<PolkadotParachainConfiguration> for ExportGenesisContext {
|
impl AsRef<PolkadotParachainConfiguration> for ExportGenesisContext {
|
||||||
fn as_ref(&self) -> &PolkadotParachainConfiguration {
|
fn as_ref(&self) -> &PolkadotParachainConfiguration {
|
||||||
&self.polkadot_parachain_configuration
|
&self.polkadot_parachain_configuration
|
||||||
@@ -759,7 +719,7 @@ pub struct CorpusConfiguration {
|
|||||||
/// - `{metadata_file_path}::{case_idx}::{mode}`: This is very similar to the above specifier
|
/// - `{metadata_file_path}::{case_idx}::{mode}`: This is very similar to the above specifier
|
||||||
/// with the exception that in this case the mode is specified and will be used in the test.
|
/// with the exception that in this case the mode is specified and will be used in the test.
|
||||||
#[serde_as(as = "Vec<serde_with::DisplayFromStr>")]
|
#[serde_as(as = "Vec<serde_with::DisplayFromStr>")]
|
||||||
#[arg(short = 't', long = "test")]
|
#[arg(short = 't', long = "test", required = true)]
|
||||||
pub test_specifiers: Vec<ParsedTestSpecifier>,
|
pub test_specifiers: Vec<ParsedTestSpecifier>,
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -842,30 +802,6 @@ pub struct KurtosisConfiguration {
|
|||||||
pub path: PathBuf,
|
pub path: PathBuf,
|
||||||
}
|
}
|
||||||
|
|
||||||
/// A set of configuration parameters for Kitchensink.
|
|
||||||
#[derive(Clone, Debug, Parser, Serialize, Deserialize)]
|
|
||||||
pub struct KitchensinkConfiguration {
|
|
||||||
/// Specifies the path of the kitchensink node to be used by the tool.
|
|
||||||
///
|
|
||||||
/// If this is not specified, then the tool assumes that it should use the kitchensink binary
|
|
||||||
/// that's provided in the user's $PATH.
|
|
||||||
#[clap(
|
|
||||||
id = "kitchensink.path",
|
|
||||||
long = "kitchensink.path",
|
|
||||||
default_value = "substrate-node"
|
|
||||||
)]
|
|
||||||
pub path: PathBuf,
|
|
||||||
|
|
||||||
/// The amount of time to wait upon startup before considering that the node timed out.
|
|
||||||
#[clap(
|
|
||||||
id = "kitchensink.start-timeout-ms",
|
|
||||||
long = "kitchensink.start-timeout-ms",
|
|
||||||
default_value = "30000",
|
|
||||||
value_parser = parse_duration
|
|
||||||
)]
|
|
||||||
pub start_timeout_ms: Duration,
|
|
||||||
}
|
|
||||||
|
|
||||||
/// A set of configuration parameters for the revive dev node.
|
/// A set of configuration parameters for the revive dev node.
|
||||||
#[derive(Clone, Debug, Parser, Serialize, Deserialize)]
|
#[derive(Clone, Debug, Parser, Serialize, Deserialize)]
|
||||||
pub struct ReviveDevNodeConfiguration {
|
pub struct ReviveDevNodeConfiguration {
|
||||||
@@ -1144,7 +1080,10 @@ impl FromStr for WorkingDirectoryConfiguration {
|
|||||||
fn from_str(s: &str) -> Result<Self, Self::Err> {
|
fn from_str(s: &str) -> Result<Self, Self::Err> {
|
||||||
match s {
|
match s {
|
||||||
"" => Ok(Default::default()),
|
"" => Ok(Default::default()),
|
||||||
_ => Ok(Self::Path(PathBuf::from(s))),
|
_ => PathBuf::from(s)
|
||||||
|
.canonicalize()
|
||||||
|
.context("Failed to canonicalize the working directory path")
|
||||||
|
.map(Self::Path),
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -1161,35 +1100,6 @@ fn parse_duration(s: &str) -> anyhow::Result<Duration> {
|
|||||||
.map_err(Into::into)
|
.map_err(Into::into)
|
||||||
}
|
}
|
||||||
|
|
||||||
/// The Solidity compatible node implementation.
|
|
||||||
///
|
|
||||||
/// This describes the solutions to be tested against on a high level.
|
|
||||||
#[derive(
|
|
||||||
Clone,
|
|
||||||
Copy,
|
|
||||||
Debug,
|
|
||||||
PartialEq,
|
|
||||||
Eq,
|
|
||||||
PartialOrd,
|
|
||||||
Ord,
|
|
||||||
Hash,
|
|
||||||
Serialize,
|
|
||||||
ValueEnum,
|
|
||||||
EnumString,
|
|
||||||
Display,
|
|
||||||
AsRefStr,
|
|
||||||
IntoStaticStr,
|
|
||||||
)]
|
|
||||||
#[strum(serialize_all = "kebab-case")]
|
|
||||||
pub enum TestingPlatform {
|
|
||||||
/// The go-ethereum reference full node EVM implementation.
|
|
||||||
Geth,
|
|
||||||
/// The kitchensink runtime provides the PolkaVM (PVM) based node implementation.
|
|
||||||
Kitchensink,
|
|
||||||
/// A polkadot/Substrate based network
|
|
||||||
Zombienet,
|
|
||||||
}
|
|
||||||
|
|
||||||
/// The output format to use for the test execution output.
|
/// The output format to use for the test execution output.
|
||||||
#[derive(
|
#[derive(
|
||||||
Clone,
|
Clone,
|
||||||
|
|||||||
@@ -37,6 +37,7 @@ schemars = { workspace = true }
|
|||||||
semver = { workspace = true }
|
semver = { workspace = true }
|
||||||
serde = { workspace = true }
|
serde = { workspace = true }
|
||||||
serde_json = { workspace = true }
|
serde_json = { workspace = true }
|
||||||
|
subxt = { workspace = true }
|
||||||
|
|
||||||
[lints]
|
[lints]
|
||||||
workspace = true
|
workspace = true
|
||||||
|
|||||||
@@ -127,6 +127,8 @@ where
|
|||||||
.inspect_err(|err| error!(?err, "Pre-linking compilation failed"))
|
.inspect_err(|err| error!(?err, "Pre-linking compilation failed"))
|
||||||
.context("Failed to produce the pre-linking compiled contracts")?;
|
.context("Failed to produce the pre-linking compiled contracts")?;
|
||||||
|
|
||||||
|
let deployer_address = self.test_definition.case.deployer_address();
|
||||||
|
|
||||||
let mut deployed_libraries = None::<HashMap<_, _>>;
|
let mut deployed_libraries = None::<HashMap<_, _>>;
|
||||||
let mut contract_sources = self
|
let mut contract_sources = self
|
||||||
.test_definition
|
.test_definition
|
||||||
@@ -159,23 +161,6 @@ where
|
|||||||
|
|
||||||
let code = alloy::hex::decode(code)?;
|
let code = alloy::hex::decode(code)?;
|
||||||
|
|
||||||
// Getting the deployer address from the cases themselves. This is to ensure
|
|
||||||
// that we're doing the deployments from different accounts and therefore we're
|
|
||||||
// not slowed down by the nonce.
|
|
||||||
let deployer_address = self
|
|
||||||
.test_definition
|
|
||||||
.case
|
|
||||||
.steps
|
|
||||||
.iter()
|
|
||||||
.filter_map(|step| match step {
|
|
||||||
Step::FunctionCall(input) => input.caller.as_address().copied(),
|
|
||||||
Step::BalanceAssertion(..) => None,
|
|
||||||
Step::StorageEmptyAssertion(..) => None,
|
|
||||||
Step::Repeat(..) => None,
|
|
||||||
Step::AllocateAccount(..) => None,
|
|
||||||
})
|
|
||||||
.next()
|
|
||||||
.unwrap_or(FunctionCallStep::default_caller_address());
|
|
||||||
let tx = TransactionBuilder::<Ethereum>::with_deploy_code(
|
let tx = TransactionBuilder::<Ethereum>::with_deploy_code(
|
||||||
TransactionRequest::default().from(deployer_address),
|
TransactionRequest::default().from(deployer_address),
|
||||||
code,
|
code,
|
||||||
|
|||||||
@@ -112,12 +112,23 @@ impl Watcher {
|
|||||||
let all_transactions_submitted = all_transactions_submitted.clone();
|
let all_transactions_submitted = all_transactions_submitted.clone();
|
||||||
let mut blocks_information_stream = self.blocks_stream;
|
let mut blocks_information_stream = self.blocks_stream;
|
||||||
async move {
|
async move {
|
||||||
while let Some(block) = blocks_information_stream.next().await {
|
while let Some(mut block) = blocks_information_stream.next().await {
|
||||||
// If the block number is equal to or less than the last block before the
|
// If the block number is equal to or less than the last block before the
|
||||||
// repetition then we ignore it and continue on to the next block.
|
// repetition then we ignore it and continue on to the next block.
|
||||||
if block.ethereum_block_information.block_number <= ignore_block_before {
|
if block.ethereum_block_information.block_number <= ignore_block_before {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
{
|
||||||
|
let watch_for_transaction_hashes =
|
||||||
|
watch_for_transaction_hashes.read().await;
|
||||||
|
for tx_hash in block.ethereum_block_information.transaction_hashes.iter() {
|
||||||
|
let Some((step_path, _)) = watch_for_transaction_hashes.get(tx_hash)
|
||||||
|
else {
|
||||||
|
continue;
|
||||||
|
};
|
||||||
|
*block.tx_counts.entry(step_path.clone()).or_default() += 1
|
||||||
|
}
|
||||||
|
}
|
||||||
reporter
|
reporter
|
||||||
.report_block_mined_event(block.clone())
|
.report_block_mined_event(block.clone())
|
||||||
.expect("Can't fail");
|
.expect("Can't fail");
|
||||||
@@ -189,7 +200,6 @@ pub enum WatcherEvent {
|
|||||||
/// streaming the blocks.
|
/// streaming the blocks.
|
||||||
ignore_block_before: BlockNumber,
|
ignore_block_before: BlockNumber,
|
||||||
},
|
},
|
||||||
|
|
||||||
/// Informs the watcher that a transaction was submitted and that the watcher should watch for a
|
/// Informs the watcher that a transaction was submitted and that the watcher should watch for a
|
||||||
/// transaction with this hash in the blocks that it watches.
|
/// transaction with this hash in the blocks that it watches.
|
||||||
SubmittedTransaction {
|
SubmittedTransaction {
|
||||||
@@ -198,7 +208,6 @@ pub enum WatcherEvent {
|
|||||||
/// The step path of the step that the transaction belongs to.
|
/// The step path of the step that the transaction belongs to.
|
||||||
step_path: StepPath,
|
step_path: StepPath,
|
||||||
},
|
},
|
||||||
|
|
||||||
/// Informs the watcher that all of the transactions of this benchmark have been submitted and
|
/// Informs the watcher that all of the transactions of this benchmark have been submitted and
|
||||||
/// that it can expect to receive no further transaction hashes and not even watch the channel
|
/// that it can expect to receive no further transaction hashes and not even watch the channel
|
||||||
/// any longer.
|
/// any longer.
|
||||||
|
|||||||
@@ -8,7 +8,7 @@ use alloy::{
|
|||||||
hex,
|
hex,
|
||||||
json_abi::JsonAbi,
|
json_abi::JsonAbi,
|
||||||
network::{Ethereum, TransactionBuilder},
|
network::{Ethereum, TransactionBuilder},
|
||||||
primitives::{Address, TxHash, U256},
|
primitives::{Address, TxHash, U256, address},
|
||||||
rpc::types::{
|
rpc::types::{
|
||||||
TransactionReceipt, TransactionRequest,
|
TransactionReceipt, TransactionRequest,
|
||||||
trace::geth::{
|
trace::geth::{
|
||||||
@@ -18,9 +18,9 @@ use alloy::{
|
|||||||
},
|
},
|
||||||
};
|
};
|
||||||
use anyhow::{Context as _, Result, bail};
|
use anyhow::{Context as _, Result, bail};
|
||||||
use futures::TryStreamExt;
|
use futures::{TryStreamExt, future::try_join_all};
|
||||||
use indexmap::IndexMap;
|
use indexmap::IndexMap;
|
||||||
use revive_dt_common::types::{PlatformIdentifier, PrivateKeyAllocator};
|
use revive_dt_common::types::{PlatformIdentifier, PrivateKeyAllocator, VmIdentifier};
|
||||||
use revive_dt_format::{
|
use revive_dt_format::{
|
||||||
metadata::{ContractInstance, ContractPathAndIdent},
|
metadata::{ContractInstance, ContractPathAndIdent},
|
||||||
steps::{
|
steps::{
|
||||||
@@ -30,6 +30,7 @@ use revive_dt_format::{
|
|||||||
},
|
},
|
||||||
traits::ResolutionContext,
|
traits::ResolutionContext,
|
||||||
};
|
};
|
||||||
|
use subxt::{ext::codec::Decode, metadata::Metadata, tx::Payload};
|
||||||
use tokio::sync::Mutex;
|
use tokio::sync::Mutex;
|
||||||
use tracing::{error, info, instrument};
|
use tracing::{error, info, instrument};
|
||||||
|
|
||||||
@@ -198,6 +199,8 @@ where
|
|||||||
})
|
})
|
||||||
.context("Failed to produce the pre-linking compiled contracts")?;
|
.context("Failed to produce the pre-linking compiled contracts")?;
|
||||||
|
|
||||||
|
let deployer_address = test_definition.case.deployer_address();
|
||||||
|
|
||||||
let mut deployed_libraries = None::<HashMap<_, _>>;
|
let mut deployed_libraries = None::<HashMap<_, _>>;
|
||||||
let mut contract_sources = test_definition
|
let mut contract_sources = test_definition
|
||||||
.metadata
|
.metadata
|
||||||
@@ -232,22 +235,6 @@ where
|
|||||||
|
|
||||||
let code = alloy::hex::decode(code)?;
|
let code = alloy::hex::decode(code)?;
|
||||||
|
|
||||||
// Getting the deployer address from the cases themselves. This is to ensure
|
|
||||||
// that we're doing the deployments from different accounts and therefore we're
|
|
||||||
// not slowed down by the nonce.
|
|
||||||
let deployer_address = test_definition
|
|
||||||
.case
|
|
||||||
.steps
|
|
||||||
.iter()
|
|
||||||
.filter_map(|step| match step {
|
|
||||||
Step::FunctionCall(input) => input.caller.as_address().copied(),
|
|
||||||
Step::BalanceAssertion(..) => None,
|
|
||||||
Step::StorageEmptyAssertion(..) => None,
|
|
||||||
Step::Repeat(..) => None,
|
|
||||||
Step::AllocateAccount(..) => None,
|
|
||||||
})
|
|
||||||
.next()
|
|
||||||
.unwrap_or(FunctionCallStep::default_caller_address());
|
|
||||||
let tx = TransactionBuilder::<Ethereum>::with_deploy_code(
|
let tx = TransactionBuilder::<Ethereum>::with_deploy_code(
|
||||||
TransactionRequest::default().from(deployer_address),
|
TransactionRequest::default().from(deployer_address),
|
||||||
code,
|
code,
|
||||||
@@ -295,6 +282,51 @@ where
|
|||||||
})
|
})
|
||||||
.context("Failed to compile the post-link contracts")?;
|
.context("Failed to compile the post-link contracts")?;
|
||||||
|
|
||||||
|
// Factory contracts on the PVM refer to the code that they're instantiating by hash rather
|
||||||
|
// than including the actual bytecode. This creates a problem where a factory contract could
|
||||||
|
// be deployed but the code it's supposed to create is not on chain. Therefore, we upload
|
||||||
|
// all the code to the chain prior to running any transactions on the driver.
|
||||||
|
if platform_information.platform.vm_identifier() == VmIdentifier::PolkaVM {
|
||||||
|
#[subxt::subxt(runtime_metadata_path = "../../assets/revive_metadata.scale")]
|
||||||
|
pub mod revive {}
|
||||||
|
|
||||||
|
let metadata_bytes = include_bytes!("../../../../assets/revive_metadata.scale");
|
||||||
|
let metadata = Metadata::decode(&mut &metadata_bytes[..])
|
||||||
|
.context("Failed to decode the revive metadata")?;
|
||||||
|
|
||||||
|
const RUNTIME_PALLET_ADDRESS: Address =
|
||||||
|
address!("0x6d6f646c70792f70616464720000000000000000");
|
||||||
|
|
||||||
|
let code_upload_tasks = compiler_output
|
||||||
|
.contracts
|
||||||
|
.values()
|
||||||
|
.flat_map(|item| item.values())
|
||||||
|
.map(|(code_string, _)| {
|
||||||
|
let metadata = metadata.clone();
|
||||||
|
async move {
|
||||||
|
let code = alloy::hex::decode(code_string)
|
||||||
|
.context("Failed to hex-decode the post-link code. This is a bug")?;
|
||||||
|
let payload = revive::tx().revive().upload_code(code, u128::MAX);
|
||||||
|
let encoded_payload = payload
|
||||||
|
.encode_call_data(&metadata)
|
||||||
|
.context("Failed to encode the upload code payload")?;
|
||||||
|
|
||||||
|
let tx_request = TransactionRequest::default()
|
||||||
|
.from(deployer_address)
|
||||||
|
.to(RUNTIME_PALLET_ADDRESS)
|
||||||
|
.input(encoded_payload.into());
|
||||||
|
platform_information
|
||||||
|
.node
|
||||||
|
.execute_transaction(tx_request)
|
||||||
|
.await
|
||||||
|
.context("Failed to execute transaction")
|
||||||
|
}
|
||||||
|
});
|
||||||
|
try_join_all(code_upload_tasks)
|
||||||
|
.await
|
||||||
|
.context("Code upload failed")?;
|
||||||
|
}
|
||||||
|
|
||||||
Ok(ExecutionState::new(
|
Ok(ExecutionState::new(
|
||||||
compiler_output.contracts,
|
compiler_output.contracts,
|
||||||
deployed_libraries.unwrap_or_default(),
|
deployed_libraries.unwrap_or_default(),
|
||||||
@@ -353,12 +385,17 @@ where
|
|||||||
.execute_account_allocation(step_path, step.as_ref())
|
.execute_account_allocation(step_path, step.as_ref())
|
||||||
.await
|
.await
|
||||||
.context("Account Allocation Step Failed"),
|
.context("Account Allocation Step Failed"),
|
||||||
}?;
|
}
|
||||||
|
.context(format!("Failure on step {step_path}"))?;
|
||||||
self.steps_executed += steps_executed;
|
self.steps_executed += steps_executed;
|
||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|
||||||
#[instrument(level = "info", skip_all)]
|
#[instrument(
|
||||||
|
level = "info",
|
||||||
|
skip_all,
|
||||||
|
fields(block_number = tracing::field::Empty)
|
||||||
|
)]
|
||||||
pub async fn execute_function_call(
|
pub async fn execute_function_call(
|
||||||
&mut self,
|
&mut self,
|
||||||
_: &StepPath,
|
_: &StepPath,
|
||||||
@@ -372,6 +409,7 @@ where
|
|||||||
.handle_function_call_execution(step, deployment_receipts)
|
.handle_function_call_execution(step, deployment_receipts)
|
||||||
.await
|
.await
|
||||||
.context("Failed to handle the function call execution")?;
|
.context("Failed to handle the function call execution")?;
|
||||||
|
tracing::Span::current().record("block_number", execution_receipt.block_number);
|
||||||
let tracing_result = self
|
let tracing_result = self
|
||||||
.handle_function_call_call_frame_tracing(execution_receipt.transaction_hash)
|
.handle_function_call_call_frame_tracing(execution_receipt.transaction_hash)
|
||||||
.await
|
.await
|
||||||
@@ -597,21 +635,26 @@ where
|
|||||||
let expected = !assertion.exception;
|
let expected = !assertion.exception;
|
||||||
let actual = receipt.status();
|
let actual = receipt.status();
|
||||||
if actual != expected {
|
if actual != expected {
|
||||||
|
let revert_reason = tracing_result
|
||||||
|
.revert_reason
|
||||||
|
.as_ref()
|
||||||
|
.or(tracing_result.error.as_ref());
|
||||||
tracing::error!(
|
tracing::error!(
|
||||||
expected,
|
expected,
|
||||||
actual,
|
actual,
|
||||||
?receipt,
|
?receipt,
|
||||||
?tracing_result,
|
?tracing_result,
|
||||||
|
?revert_reason,
|
||||||
"Transaction status assertion failed"
|
"Transaction status assertion failed"
|
||||||
);
|
);
|
||||||
anyhow::bail!(
|
anyhow::bail!(
|
||||||
"Transaction status assertion failed - Expected {expected} but got {actual}",
|
"Transaction status assertion failed - Expected {expected} but got {actual}. Revert reason: {revert_reason:?}",
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Handling the calldata assertion
|
// Handling the calldata assertion
|
||||||
if let Some(ref expected_calldata) = assertion.return_data {
|
if let Some(ref expected_output) = assertion.return_data {
|
||||||
let expected = expected_calldata;
|
let expected = expected_output;
|
||||||
let actual = &tracing_result.output.as_ref().unwrap_or_default();
|
let actual = &tracing_result.output.as_ref().unwrap_or_default();
|
||||||
if !expected
|
if !expected
|
||||||
.is_equivalent(actual, resolver.as_ref(), resolution_context)
|
.is_equivalent(actual, resolver.as_ref(), resolution_context)
|
||||||
@@ -622,9 +665,9 @@ where
|
|||||||
?receipt,
|
?receipt,
|
||||||
?expected,
|
?expected,
|
||||||
%actual,
|
%actual,
|
||||||
"Calldata assertion failed"
|
"Output assertion failed"
|
||||||
);
|
);
|
||||||
anyhow::bail!("Calldata assertion failed - Expected {expected:?} but got {actual}",);
|
anyhow::bail!("Output assertion failed - Expected {expected:?} but got {actual}",);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -330,6 +330,7 @@ async fn start_cli_reporting_task(output_format: OutputFormat, reporter: Reporte
|
|||||||
.unwrap();
|
.unwrap();
|
||||||
writeln!(buf).unwrap();
|
writeln!(buf).unwrap();
|
||||||
|
|
||||||
|
if aggregator_events_rx.is_empty() {
|
||||||
buf = tokio::task::spawn_blocking(move || {
|
buf = tokio::task::spawn_blocking(move || {
|
||||||
buf.flush().unwrap();
|
buf.flush().unwrap();
|
||||||
buf
|
buf
|
||||||
@@ -339,6 +340,8 @@ async fn start_cli_reporting_task(output_format: OutputFormat, reporter: Reporte
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
info!("Aggregator Broadcast Channel Closed");
|
||||||
|
|
||||||
// Summary at the end.
|
// Summary at the end.
|
||||||
match output_format {
|
match output_format {
|
||||||
|
|||||||
@@ -172,134 +172,6 @@ impl Platform for LighthouseGethEvmSolcPlatform {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
#[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord, Default, Hash)]
|
|
||||||
pub struct KitchensinkPolkavmResolcPlatform;
|
|
||||||
|
|
||||||
impl Platform for KitchensinkPolkavmResolcPlatform {
|
|
||||||
fn platform_identifier(&self) -> PlatformIdentifier {
|
|
||||||
PlatformIdentifier::KitchensinkPolkavmResolc
|
|
||||||
}
|
|
||||||
|
|
||||||
fn node_identifier(&self) -> NodeIdentifier {
|
|
||||||
NodeIdentifier::Kitchensink
|
|
||||||
}
|
|
||||||
|
|
||||||
fn vm_identifier(&self) -> VmIdentifier {
|
|
||||||
VmIdentifier::PolkaVM
|
|
||||||
}
|
|
||||||
|
|
||||||
fn compiler_identifier(&self) -> CompilerIdentifier {
|
|
||||||
CompilerIdentifier::Resolc
|
|
||||||
}
|
|
||||||
|
|
||||||
fn new_node(
|
|
||||||
&self,
|
|
||||||
context: Context,
|
|
||||||
) -> anyhow::Result<JoinHandle<anyhow::Result<Box<dyn EthereumNode + Send + Sync>>>> {
|
|
||||||
let genesis_configuration = AsRef::<GenesisConfiguration>::as_ref(&context);
|
|
||||||
let kitchensink_path = AsRef::<KitchensinkConfiguration>::as_ref(&context)
|
|
||||||
.path
|
|
||||||
.clone();
|
|
||||||
let genesis = genesis_configuration.genesis()?.clone();
|
|
||||||
Ok(thread::spawn(move || {
|
|
||||||
let node = SubstrateNode::new(
|
|
||||||
kitchensink_path,
|
|
||||||
SubstrateNode::KITCHENSINK_EXPORT_CHAINSPEC_COMMAND,
|
|
||||||
None,
|
|
||||||
context,
|
|
||||||
&[],
|
|
||||||
);
|
|
||||||
let node = spawn_node(node, genesis)?;
|
|
||||||
Ok(Box::new(node) as Box<_>)
|
|
||||||
}))
|
|
||||||
}
|
|
||||||
|
|
||||||
fn new_compiler(
|
|
||||||
&self,
|
|
||||||
context: Context,
|
|
||||||
version: Option<VersionOrRequirement>,
|
|
||||||
) -> Pin<Box<dyn Future<Output = anyhow::Result<Box<dyn SolidityCompiler>>>>> {
|
|
||||||
Box::pin(async move {
|
|
||||||
let compiler = Resolc::new(context, version).await;
|
|
||||||
compiler.map(|compiler| Box::new(compiler) as Box<dyn SolidityCompiler>)
|
|
||||||
})
|
|
||||||
}
|
|
||||||
|
|
||||||
fn export_genesis(&self, context: Context) -> anyhow::Result<serde_json::Value> {
|
|
||||||
let kitchensink_path = AsRef::<KitchensinkConfiguration>::as_ref(&context)
|
|
||||||
.path
|
|
||||||
.as_path();
|
|
||||||
let wallet = AsRef::<WalletConfiguration>::as_ref(&context).wallet();
|
|
||||||
let export_chainspec_command = SubstrateNode::KITCHENSINK_EXPORT_CHAINSPEC_COMMAND;
|
|
||||||
|
|
||||||
SubstrateNode::node_genesis(kitchensink_path, export_chainspec_command, &wallet)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
#[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord, Default, Hash)]
|
|
||||||
pub struct KitchensinkRevmSolcPlatform;
|
|
||||||
|
|
||||||
impl Platform for KitchensinkRevmSolcPlatform {
|
|
||||||
fn platform_identifier(&self) -> PlatformIdentifier {
|
|
||||||
PlatformIdentifier::KitchensinkRevmSolc
|
|
||||||
}
|
|
||||||
|
|
||||||
fn node_identifier(&self) -> NodeIdentifier {
|
|
||||||
NodeIdentifier::Kitchensink
|
|
||||||
}
|
|
||||||
|
|
||||||
fn vm_identifier(&self) -> VmIdentifier {
|
|
||||||
VmIdentifier::Evm
|
|
||||||
}
|
|
||||||
|
|
||||||
fn compiler_identifier(&self) -> CompilerIdentifier {
|
|
||||||
CompilerIdentifier::Solc
|
|
||||||
}
|
|
||||||
|
|
||||||
fn new_node(
|
|
||||||
&self,
|
|
||||||
context: Context,
|
|
||||||
) -> anyhow::Result<JoinHandle<anyhow::Result<Box<dyn EthereumNode + Send + Sync>>>> {
|
|
||||||
let genesis_configuration = AsRef::<GenesisConfiguration>::as_ref(&context);
|
|
||||||
let kitchensink_path = AsRef::<KitchensinkConfiguration>::as_ref(&context)
|
|
||||||
.path
|
|
||||||
.clone();
|
|
||||||
let genesis = genesis_configuration.genesis()?.clone();
|
|
||||||
Ok(thread::spawn(move || {
|
|
||||||
let node = SubstrateNode::new(
|
|
||||||
kitchensink_path,
|
|
||||||
SubstrateNode::KITCHENSINK_EXPORT_CHAINSPEC_COMMAND,
|
|
||||||
None,
|
|
||||||
context,
|
|
||||||
&[],
|
|
||||||
);
|
|
||||||
let node = spawn_node(node, genesis)?;
|
|
||||||
Ok(Box::new(node) as Box<_>)
|
|
||||||
}))
|
|
||||||
}
|
|
||||||
|
|
||||||
fn new_compiler(
|
|
||||||
&self,
|
|
||||||
context: Context,
|
|
||||||
version: Option<VersionOrRequirement>,
|
|
||||||
) -> Pin<Box<dyn Future<Output = anyhow::Result<Box<dyn SolidityCompiler>>>>> {
|
|
||||||
Box::pin(async move {
|
|
||||||
let compiler = Solc::new(context, version).await;
|
|
||||||
compiler.map(|compiler| Box::new(compiler) as Box<dyn SolidityCompiler>)
|
|
||||||
})
|
|
||||||
}
|
|
||||||
|
|
||||||
fn export_genesis(&self, context: Context) -> anyhow::Result<serde_json::Value> {
|
|
||||||
let kitchensink_path = AsRef::<KitchensinkConfiguration>::as_ref(&context)
|
|
||||||
.path
|
|
||||||
.as_path();
|
|
||||||
let wallet = AsRef::<WalletConfiguration>::as_ref(&context).wallet();
|
|
||||||
let export_chainspec_command = SubstrateNode::KITCHENSINK_EXPORT_CHAINSPEC_COMMAND;
|
|
||||||
|
|
||||||
SubstrateNode::node_genesis(kitchensink_path, export_chainspec_command, &wallet)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
#[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord, Default, Hash)]
|
#[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord, Default, Hash)]
|
||||||
pub struct ReviveDevNodePolkavmResolcPlatform;
|
pub struct ReviveDevNodePolkavmResolcPlatform;
|
||||||
|
|
||||||
@@ -557,12 +429,6 @@ impl From<PlatformIdentifier> for Box<dyn Platform> {
|
|||||||
PlatformIdentifier::LighthouseGethEvmSolc => {
|
PlatformIdentifier::LighthouseGethEvmSolc => {
|
||||||
Box::new(LighthouseGethEvmSolcPlatform) as Box<_>
|
Box::new(LighthouseGethEvmSolcPlatform) as Box<_>
|
||||||
}
|
}
|
||||||
PlatformIdentifier::KitchensinkPolkavmResolc => {
|
|
||||||
Box::new(KitchensinkPolkavmResolcPlatform) as Box<_>
|
|
||||||
}
|
|
||||||
PlatformIdentifier::KitchensinkRevmSolc => {
|
|
||||||
Box::new(KitchensinkRevmSolcPlatform) as Box<_>
|
|
||||||
}
|
|
||||||
PlatformIdentifier::ReviveDevNodePolkavmResolc => {
|
PlatformIdentifier::ReviveDevNodePolkavmResolc => {
|
||||||
Box::new(ReviveDevNodePolkavmResolcPlatform) as Box<_>
|
Box::new(ReviveDevNodePolkavmResolcPlatform) as Box<_>
|
||||||
}
|
}
|
||||||
@@ -584,12 +450,6 @@ impl From<PlatformIdentifier> for &dyn Platform {
|
|||||||
PlatformIdentifier::LighthouseGethEvmSolc => {
|
PlatformIdentifier::LighthouseGethEvmSolc => {
|
||||||
&LighthouseGethEvmSolcPlatform as &dyn Platform
|
&LighthouseGethEvmSolcPlatform as &dyn Platform
|
||||||
}
|
}
|
||||||
PlatformIdentifier::KitchensinkPolkavmResolc => {
|
|
||||||
&KitchensinkPolkavmResolcPlatform as &dyn Platform
|
|
||||||
}
|
|
||||||
PlatformIdentifier::KitchensinkRevmSolc => {
|
|
||||||
&KitchensinkRevmSolcPlatform as &dyn Platform
|
|
||||||
}
|
|
||||||
PlatformIdentifier::ReviveDevNodePolkavmResolc => {
|
PlatformIdentifier::ReviveDevNodePolkavmResolc => {
|
||||||
&ReviveDevNodePolkavmResolcPlatform as &dyn Platform
|
&ReviveDevNodePolkavmResolcPlatform as &dyn Platform
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,3 +1,4 @@
|
|||||||
|
use alloy::primitives::Address;
|
||||||
use schemars::JsonSchema;
|
use schemars::JsonSchema;
|
||||||
use serde::{Deserialize, Serialize};
|
use serde::{Deserialize, Serialize};
|
||||||
|
|
||||||
@@ -107,6 +108,20 @@ impl Case {
|
|||||||
None => Mode::all().cloned().collect(),
|
None => Mode::all().cloned().collect(),
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
pub fn deployer_address(&self) -> Address {
|
||||||
|
self.steps
|
||||||
|
.iter()
|
||||||
|
.filter_map(|step| match step {
|
||||||
|
Step::FunctionCall(input) => input.caller.as_address().copied(),
|
||||||
|
Step::BalanceAssertion(..) => None,
|
||||||
|
Step::StorageEmptyAssertion(..) => None,
|
||||||
|
Step::Repeat(..) => None,
|
||||||
|
Step::AllocateAccount(..) => None,
|
||||||
|
})
|
||||||
|
.next()
|
||||||
|
.unwrap_or(FunctionCallStep::default_caller_address())
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
define_wrapper_type!(
|
define_wrapper_type!(
|
||||||
|
|||||||
@@ -1,5 +1,6 @@
|
|||||||
use std::{collections::HashMap, fmt::Display, str::FromStr};
|
use std::{collections::HashMap, fmt::Display, str::FromStr};
|
||||||
|
|
||||||
|
use alloy::hex::ToHexExt;
|
||||||
use alloy::primitives::{FixedBytes, utils::parse_units};
|
use alloy::primitives::{FixedBytes, utils::parse_units};
|
||||||
use alloy::{
|
use alloy::{
|
||||||
eips::BlockNumberOrTag,
|
eips::BlockNumberOrTag,
|
||||||
@@ -686,8 +687,8 @@ impl Calldata {
|
|||||||
Calldata::Compound(items) => {
|
Calldata::Compound(items) => {
|
||||||
stream::iter(items.iter().zip(other.chunks(32)))
|
stream::iter(items.iter().zip(other.chunks(32)))
|
||||||
.map(|(this, other)| async move {
|
.map(|(this, other)| async move {
|
||||||
// The matterlabs format supports wildcards and therefore we
|
// The MatterLabs format supports wildcards and therefore we also need to
|
||||||
// also need to support them.
|
// support them.
|
||||||
if this.as_ref() == "*" {
|
if this.as_ref() == "*" {
|
||||||
return Ok::<_, anyhow::Error>(true);
|
return Ok::<_, anyhow::Error>(true);
|
||||||
}
|
}
|
||||||
@@ -705,6 +706,7 @@ impl Calldata {
|
|||||||
.await
|
.await
|
||||||
.context("Failed to resolve calldata item during equivalence check")?;
|
.context("Failed to resolve calldata item during equivalence check")?;
|
||||||
let other = U256::from_be_slice(&other);
|
let other = U256::from_be_slice(&other);
|
||||||
|
|
||||||
Ok(this == other)
|
Ok(this == other)
|
||||||
})
|
})
|
||||||
.buffered(0xFF)
|
.buffered(0xFF)
|
||||||
@@ -717,7 +719,7 @@ impl Calldata {
|
|||||||
}
|
}
|
||||||
|
|
||||||
impl CalldataItem {
|
impl CalldataItem {
|
||||||
#[instrument(level = "info", skip_all, err)]
|
#[instrument(level = "info", skip_all, err(Debug))]
|
||||||
async fn resolve(
|
async fn resolve(
|
||||||
&self,
|
&self,
|
||||||
resolver: &(impl ResolverApi + ?Sized),
|
resolver: &(impl ResolverApi + ?Sized),
|
||||||
@@ -768,7 +770,14 @@ impl CalldataItem {
|
|||||||
match stack.as_slice() {
|
match stack.as_slice() {
|
||||||
// Empty stack means that we got an empty compound calldata which we resolve to zero.
|
// Empty stack means that we got an empty compound calldata which we resolve to zero.
|
||||||
[] => Ok(U256::ZERO),
|
[] => Ok(U256::ZERO),
|
||||||
[CalldataToken::Item(item)] => Ok(*item),
|
[CalldataToken::Item(item)] => {
|
||||||
|
tracing::debug!(
|
||||||
|
original_item = ?self,
|
||||||
|
resolved_item = item.to_be_bytes::<32>().encode_hex(),
|
||||||
|
"Resolution Done"
|
||||||
|
);
|
||||||
|
Ok(*item)
|
||||||
|
}
|
||||||
_ => Err(anyhow::anyhow!(
|
_ => Err(anyhow::anyhow!(
|
||||||
"Invalid calldata arithmetic operation - Invalid stack"
|
"Invalid calldata arithmetic operation - Invalid stack"
|
||||||
)),
|
)),
|
||||||
@@ -898,7 +907,7 @@ impl<T: AsRef<str>> CalldataToken<T> {
|
|||||||
let block_hash = resolver
|
let block_hash = resolver
|
||||||
.block_hash(desired_block_number.into())
|
.block_hash(desired_block_number.into())
|
||||||
.await
|
.await
|
||||||
.context("Failed to resolve block hash for desired block number")?;
|
.context(format!("Failed to resolve the block hash of block number {desired_block_number}"))?;
|
||||||
|
|
||||||
Ok(U256::from_be_bytes(block_hash.0))
|
Ok(U256::from_be_bytes(block_hash.0))
|
||||||
} else if item == Self::BLOCK_NUMBER_VARIABLE {
|
} else if item == Self::BLOCK_NUMBER_VARIABLE {
|
||||||
|
|||||||
@@ -540,6 +540,7 @@ impl EthereumNode for GethNode {
|
|||||||
.to_vec(),
|
.to_vec(),
|
||||||
},
|
},
|
||||||
substrate_block_information: None,
|
substrate_block_information: None,
|
||||||
|
tx_counts: Default::default(),
|
||||||
})
|
})
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|||||||
@@ -771,6 +771,7 @@ impl EthereumNode for LighthouseGethNode {
|
|||||||
.to_vec(),
|
.to_vec(),
|
||||||
},
|
},
|
||||||
substrate_block_information: None,
|
substrate_block_information: None,
|
||||||
|
tx_counts: Default::default(),
|
||||||
})
|
})
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|||||||
@@ -4,7 +4,7 @@ use std::{
|
|||||||
pin::Pin,
|
pin::Pin,
|
||||||
process::{Command, Stdio},
|
process::{Command, Stdio},
|
||||||
sync::{
|
sync::{
|
||||||
Arc,
|
Arc, Mutex,
|
||||||
atomic::{AtomicU32, Ordering},
|
atomic::{AtomicU32, Ordering},
|
||||||
},
|
},
|
||||||
time::Duration,
|
time::Duration,
|
||||||
@@ -32,7 +32,7 @@ use futures::{FutureExt, Stream, StreamExt};
|
|||||||
use revive_common::EVMVersion;
|
use revive_common::EVMVersion;
|
||||||
use revive_dt_common::fs::clear_directory;
|
use revive_dt_common::fs::clear_directory;
|
||||||
use revive_dt_format::traits::ResolverApi;
|
use revive_dt_format::traits::ResolverApi;
|
||||||
use serde_json::json;
|
use serde_json::{Value, json};
|
||||||
use sp_core::crypto::Ss58Codec;
|
use sp_core::crypto::Ss58Codec;
|
||||||
use sp_runtime::AccountId32;
|
use sp_runtime::AccountId32;
|
||||||
|
|
||||||
@@ -57,6 +57,9 @@ use crate::{
|
|||||||
|
|
||||||
static NODE_COUNT: AtomicU32 = AtomicU32::new(0);
|
static NODE_COUNT: AtomicU32 = AtomicU32::new(0);
|
||||||
|
|
||||||
|
/// The number of blocks that should be cached by the revive-dev-node and the eth-rpc.
|
||||||
|
const NUMBER_OF_CACHED_BLOCKS: u32 = 100_000;
|
||||||
|
|
||||||
/// A node implementation for Substrate based chains. Currently, this supports either substrate
|
/// A node implementation for Substrate based chains. Currently, this supports either substrate
|
||||||
/// or the revive-dev-node which is done by changing the path and some of the other arguments passed
|
/// or the revive-dev-node which is done by changing the path and some of the other arguments passed
|
||||||
/// to the command.
|
/// to the command.
|
||||||
@@ -92,7 +95,6 @@ impl SubstrateNode {
|
|||||||
const SUBSTRATE_LOG_ENV: &str = "error,evm=debug,sc_rpc_server=info,runtime::revive=debug";
|
const SUBSTRATE_LOG_ENV: &str = "error,evm=debug,sc_rpc_server=info,runtime::revive=debug";
|
||||||
const PROXY_LOG_ENV: &str = "info,eth-rpc=debug";
|
const PROXY_LOG_ENV: &str = "info,eth-rpc=debug";
|
||||||
|
|
||||||
pub const KITCHENSINK_EXPORT_CHAINSPEC_COMMAND: &str = "export-chain-spec";
|
|
||||||
pub const REVIVE_DEV_NODE_EXPORT_CHAINSPEC_COMMAND: &str = "build-spec";
|
pub const REVIVE_DEV_NODE_EXPORT_CHAINSPEC_COMMAND: &str = "build-spec";
|
||||||
|
|
||||||
pub fn new(
|
pub fn new(
|
||||||
@@ -139,6 +141,8 @@ impl SubstrateNode {
|
|||||||
}
|
}
|
||||||
|
|
||||||
fn init(&mut self, _: Genesis) -> anyhow::Result<&mut Self> {
|
fn init(&mut self, _: Genesis) -> anyhow::Result<&mut Self> {
|
||||||
|
static CHAINSPEC_MUTEX: Mutex<Option<Value>> = Mutex::new(None);
|
||||||
|
|
||||||
if !self.rpc_url.is_empty() {
|
if !self.rpc_url.is_empty() {
|
||||||
return Ok(self);
|
return Ok(self);
|
||||||
}
|
}
|
||||||
@@ -157,12 +161,22 @@ impl SubstrateNode {
|
|||||||
let template_chainspec_path = self.base_directory.join(Self::CHAIN_SPEC_JSON_FILE);
|
let template_chainspec_path = self.base_directory.join(Self::CHAIN_SPEC_JSON_FILE);
|
||||||
|
|
||||||
trace!("Creating the node genesis");
|
trace!("Creating the node genesis");
|
||||||
|
let chainspec_json = {
|
||||||
|
let mut chainspec_mutex = CHAINSPEC_MUTEX.lock().expect("Poisoned");
|
||||||
|
match chainspec_mutex.as_ref() {
|
||||||
|
Some(chainspec_json) => chainspec_json.clone(),
|
||||||
|
None => {
|
||||||
let chainspec_json = Self::node_genesis(
|
let chainspec_json = Self::node_genesis(
|
||||||
&self.node_binary,
|
&self.node_binary,
|
||||||
&self.export_chainspec_command,
|
&self.export_chainspec_command,
|
||||||
&self.wallet,
|
&self.wallet,
|
||||||
)
|
)
|
||||||
.context("Failed to prepare the chainspec command")?;
|
.context("Failed to prepare the chainspec command")?;
|
||||||
|
*chainspec_mutex = Some(chainspec_json.clone());
|
||||||
|
chainspec_json
|
||||||
|
}
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
trace!("Writing the node genesis");
|
trace!("Writing the node genesis");
|
||||||
serde_json::to_writer_pretty(
|
serde_json::to_writer_pretty(
|
||||||
@@ -213,6 +227,8 @@ impl SubstrateNode {
|
|||||||
.arg(u32::MAX.to_string())
|
.arg(u32::MAX.to_string())
|
||||||
.arg("--pool-kbytes")
|
.arg("--pool-kbytes")
|
||||||
.arg(u32::MAX.to_string())
|
.arg(u32::MAX.to_string())
|
||||||
|
.arg("--state-pruning")
|
||||||
|
.arg(NUMBER_OF_CACHED_BLOCKS.to_string())
|
||||||
.env("RUST_LOG", Self::SUBSTRATE_LOG_ENV)
|
.env("RUST_LOG", Self::SUBSTRATE_LOG_ENV)
|
||||||
.stdout(stdout_file)
|
.stdout(stdout_file)
|
||||||
.stderr(stderr_file);
|
.stderr(stderr_file);
|
||||||
@@ -252,6 +268,10 @@ impl SubstrateNode {
|
|||||||
.arg(format!("ws://127.0.0.1:{substrate_rpc_port}"))
|
.arg(format!("ws://127.0.0.1:{substrate_rpc_port}"))
|
||||||
.arg("--rpc-max-connections")
|
.arg("--rpc-max-connections")
|
||||||
.arg(u32::MAX.to_string())
|
.arg(u32::MAX.to_string())
|
||||||
|
.arg("--index-last-n-blocks")
|
||||||
|
.arg(NUMBER_OF_CACHED_BLOCKS.to_string())
|
||||||
|
.arg("--cache-size")
|
||||||
|
.arg(NUMBER_OF_CACHED_BLOCKS.to_string())
|
||||||
.env("RUST_LOG", Self::PROXY_LOG_ENV)
|
.env("RUST_LOG", Self::PROXY_LOG_ENV)
|
||||||
.stdout(stdout_file)
|
.stdout(stdout_file)
|
||||||
.stderr(stderr_file);
|
.stderr(stderr_file);
|
||||||
@@ -304,7 +324,7 @@ impl SubstrateNode {
|
|||||||
.get_or_try_init(|| async move {
|
.get_or_try_init(|| async move {
|
||||||
construct_concurrency_limited_provider::<Ethereum, _>(
|
construct_concurrency_limited_provider::<Ethereum, _>(
|
||||||
self.rpc_url.as_str(),
|
self.rpc_url.as_str(),
|
||||||
FallbackGasFiller::new(u64::MAX, 5_000_000_000, 1_000_000_000),
|
FallbackGasFiller::new(u64::MAX, 50_000_000_000, 1_000_000_000),
|
||||||
ChainIdFiller::new(Some(CHAIN_ID)),
|
ChainIdFiller::new(Some(CHAIN_ID)),
|
||||||
NonceFiller::new(self.nonce_manager.clone()),
|
NonceFiller::new(self.nonce_manager.clone()),
|
||||||
self.wallet.clone(),
|
self.wallet.clone(),
|
||||||
@@ -333,7 +353,7 @@ impl SubstrateNode {
|
|||||||
trace!("Waiting for chainspec export");
|
trace!("Waiting for chainspec export");
|
||||||
if !output.status.success() {
|
if !output.status.success() {
|
||||||
anyhow::bail!(
|
anyhow::bail!(
|
||||||
"Substrate-node export-chain-spec failed: {}",
|
"substrate-node export-chain-spec failed: {}",
|
||||||
String::from_utf8_lossy(&output.stderr)
|
String::from_utf8_lossy(&output.stderr)
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
@@ -578,6 +598,7 @@ impl EthereumNode for SubstrateNode {
|
|||||||
proof_size: block_proof_size,
|
proof_size: block_proof_size,
|
||||||
max_proof_size,
|
max_proof_size,
|
||||||
}),
|
}),
|
||||||
|
tx_counts: Default::default(),
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
@@ -799,8 +820,8 @@ mod tests {
|
|||||||
|
|
||||||
let context = test_config();
|
let context = test_config();
|
||||||
let mut node = SubstrateNode::new(
|
let mut node = SubstrateNode::new(
|
||||||
context.kitchensink_configuration.path.clone(),
|
context.revive_dev_node_configuration.path.clone(),
|
||||||
SubstrateNode::KITCHENSINK_EXPORT_CHAINSPEC_COMMAND,
|
SubstrateNode::REVIVE_DEV_NODE_EXPORT_CHAINSPEC_COMMAND,
|
||||||
None,
|
None,
|
||||||
&context,
|
&context,
|
||||||
&[],
|
&[],
|
||||||
@@ -822,6 +843,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
|
#[ignore = "Ignored since it takes a long time to run"]
|
||||||
async fn node_mines_simple_transfer_transaction_and_returns_receipt() {
|
async fn node_mines_simple_transfer_transaction_and_returns_receipt() {
|
||||||
// Arrange
|
// Arrange
|
||||||
let (context, node) = shared_state();
|
let (context, node) = shared_state();
|
||||||
@@ -838,11 +860,14 @@ mod tests {
|
|||||||
.value(U256::from(100_000_000_000_000u128));
|
.value(U256::from(100_000_000_000_000u128));
|
||||||
|
|
||||||
// Act
|
// Act
|
||||||
let receipt = provider.send_transaction(transaction).await;
|
let mut pending_transaction = provider
|
||||||
|
.send_transaction(transaction)
|
||||||
|
.await
|
||||||
|
.expect("Submission failed");
|
||||||
|
pending_transaction.set_timeout(Some(Duration::from_secs(60)));
|
||||||
|
|
||||||
// Assert
|
// Assert
|
||||||
let _ = receipt
|
let _ = pending_transaction
|
||||||
.expect("Failed to send the transfer transaction")
|
|
||||||
.get_receipt()
|
.get_receipt()
|
||||||
.await
|
.await
|
||||||
.expect("Failed to get the receipt for the transfer");
|
.expect("Failed to get the receipt for the transfer");
|
||||||
@@ -866,8 +891,8 @@ mod tests {
|
|||||||
|
|
||||||
let context = test_config();
|
let context = test_config();
|
||||||
let mut dummy_node = SubstrateNode::new(
|
let mut dummy_node = SubstrateNode::new(
|
||||||
context.kitchensink_configuration.path.clone(),
|
context.revive_dev_node_configuration.path.clone(),
|
||||||
SubstrateNode::KITCHENSINK_EXPORT_CHAINSPEC_COMMAND,
|
SubstrateNode::REVIVE_DEV_NODE_EXPORT_CHAINSPEC_COMMAND,
|
||||||
None,
|
None,
|
||||||
&context,
|
&context,
|
||||||
&[],
|
&[],
|
||||||
@@ -960,7 +985,7 @@ mod tests {
|
|||||||
|
|
||||||
assert!(
|
assert!(
|
||||||
version.starts_with("substrate-node"),
|
version.starts_with("substrate-node"),
|
||||||
"Expected Substrate-node version string, got: {version}"
|
"Expected substrate-node version string, got: {version}"
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -210,6 +210,7 @@ impl ZombienetNode {
|
|||||||
.with_args(vec![
|
.with_args(vec![
|
||||||
("--pool-limit", u32::MAX.to_string().as_str()).into(),
|
("--pool-limit", u32::MAX.to_string().as_str()).into(),
|
||||||
("--pool-kbytes", u32::MAX.to_string().as_str()).into(),
|
("--pool-kbytes", u32::MAX.to_string().as_str()).into(),
|
||||||
|
("--dev-block-time", 12000u16.to_string().as_str()).into(),
|
||||||
])
|
])
|
||||||
})
|
})
|
||||||
})
|
})
|
||||||
@@ -355,7 +356,7 @@ impl ZombienetNode {
|
|||||||
|
|
||||||
if !output.status.success() {
|
if !output.status.success() {
|
||||||
anyhow::bail!(
|
anyhow::bail!(
|
||||||
"Substrate-node export-chain-spec failed: {}",
|
"substrate-node export-chain-spec failed: {}",
|
||||||
String::from_utf8_lossy(&output.stderr)
|
String::from_utf8_lossy(&output.stderr)
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
@@ -599,6 +600,7 @@ impl EthereumNode for ZombienetNode {
|
|||||||
proof_size: block_proof_size,
|
proof_size: block_proof_size,
|
||||||
max_proof_size,
|
max_proof_size,
|
||||||
}),
|
}),
|
||||||
|
tx_counts: Default::default(),
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
@@ -854,8 +856,9 @@ mod tests {
|
|||||||
use utils::{new_node, test_config};
|
use utils::{new_node, test_config};
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
#[ignore = "Ignored for the time being"]
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
async fn test_transfer_transaction_should_return_receipt() {
|
async fn test_transfer_transaction_should_return_receipt() {
|
||||||
|
// Arrange
|
||||||
let (ctx, node) = new_node().await;
|
let (ctx, node) = new_node().await;
|
||||||
|
|
||||||
let provider = node.provider().await.expect("Failed to create provider");
|
let provider = node.provider().await.expect("Failed to create provider");
|
||||||
@@ -864,15 +867,22 @@ mod tests {
|
|||||||
.to(account_address)
|
.to(account_address)
|
||||||
.value(U256::from(100_000_000_000_000u128));
|
.value(U256::from(100_000_000_000_000u128));
|
||||||
|
|
||||||
let receipt = provider.send_transaction(transaction).await;
|
// Act
|
||||||
let _ = receipt
|
let mut pending_transaction = provider
|
||||||
.expect("Failed to send the transfer transaction")
|
.send_transaction(transaction)
|
||||||
|
.await
|
||||||
|
.expect("Submission failed");
|
||||||
|
pending_transaction.set_timeout(Some(Duration::from_secs(60)));
|
||||||
|
|
||||||
|
// Assert
|
||||||
|
let _ = pending_transaction
|
||||||
.get_receipt()
|
.get_receipt()
|
||||||
.await
|
.await
|
||||||
.expect("Failed to get the receipt for the transfer");
|
.expect("Failed to get the receipt for the transfer");
|
||||||
}
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
fn print_eth_to_polkadot_mappings() {
|
fn print_eth_to_polkadot_mappings() {
|
||||||
let eth_addresses = vec![
|
let eth_addresses = vec![
|
||||||
"0x90F8bf6A479f320ead074411a4B0e7944Ea8c9C1",
|
"0x90F8bf6A479f320ead074411a4B0e7944Ea8c9C1",
|
||||||
@@ -888,6 +898,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
fn test_eth_to_polkadot_address() {
|
fn test_eth_to_polkadot_address() {
|
||||||
let cases = vec![
|
let cases = vec![
|
||||||
(
|
(
|
||||||
@@ -918,6 +929,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
fn eth_rpc_version_works() {
|
fn eth_rpc_version_works() {
|
||||||
// Arrange
|
// Arrange
|
||||||
let context = test_config();
|
let context = test_config();
|
||||||
@@ -937,6 +949,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
fn version_works() {
|
fn version_works() {
|
||||||
// Arrange
|
// Arrange
|
||||||
let context = test_config();
|
let context = test_config();
|
||||||
@@ -956,7 +969,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
#[ignore = "Ignored since they take a long time to run"]
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
async fn get_chain_id_from_node_should_succeed() {
|
async fn get_chain_id_from_node_should_succeed() {
|
||||||
// Arrange
|
// Arrange
|
||||||
let node = shared_node().await;
|
let node = shared_node().await;
|
||||||
@@ -975,7 +988,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
#[ignore = "Ignored since they take a long time to run"]
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
async fn can_get_gas_limit_from_node() {
|
async fn can_get_gas_limit_from_node() {
|
||||||
// Arrange
|
// Arrange
|
||||||
let node = shared_node().await;
|
let node = shared_node().await;
|
||||||
@@ -993,7 +1006,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
#[ignore = "Ignored since they take a long time to run"]
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
async fn can_get_coinbase_from_node() {
|
async fn can_get_coinbase_from_node() {
|
||||||
// Arrange
|
// Arrange
|
||||||
let node = shared_node().await;
|
let node = shared_node().await;
|
||||||
@@ -1011,7 +1024,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
#[ignore = "Ignored since they take a long time to run"]
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
async fn can_get_block_difficulty_from_node() {
|
async fn can_get_block_difficulty_from_node() {
|
||||||
// Arrange
|
// Arrange
|
||||||
let node = shared_node().await;
|
let node = shared_node().await;
|
||||||
@@ -1029,7 +1042,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
#[ignore = "Ignored since they take a long time to run"]
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
async fn can_get_block_hash_from_node() {
|
async fn can_get_block_hash_from_node() {
|
||||||
// Arrange
|
// Arrange
|
||||||
let node = shared_node().await;
|
let node = shared_node().await;
|
||||||
@@ -1047,7 +1060,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
#[ignore = "Ignored since they take a long time to run"]
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
async fn can_get_block_timestamp_from_node() {
|
async fn can_get_block_timestamp_from_node() {
|
||||||
// Arrange
|
// Arrange
|
||||||
let node = shared_node().await;
|
let node = shared_node().await;
|
||||||
@@ -1065,7 +1078,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
#[ignore = "Ignored since they take a long time to run"]
|
#[ignore = "Ignored since CI doesn't have zombienet installed"]
|
||||||
async fn can_get_block_number_from_node() {
|
async fn can_get_block_number_from_node() {
|
||||||
// Arrange
|
// Arrange
|
||||||
let node = shared_node().await;
|
let node = shared_node().await;
|
||||||
|
|||||||
@@ -62,7 +62,10 @@ where
|
|||||||
) -> TransportResult<Self::Fillable> {
|
) -> TransportResult<Self::Fillable> {
|
||||||
match self.inner.prepare(provider, tx).await {
|
match self.inner.prepare(provider, tx).await {
|
||||||
Ok(fill) => Ok(Some(fill)),
|
Ok(fill) => Ok(Some(fill)),
|
||||||
Err(_) => Ok(None),
|
Err(err) => {
|
||||||
|
tracing::debug!(error = ?err, "Gas Provider Estimation Failed, using fallback");
|
||||||
|
Ok(None)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -41,7 +41,7 @@ pub struct ReportAggregator {
|
|||||||
impl ReportAggregator {
|
impl ReportAggregator {
|
||||||
pub fn new(context: Context) -> Self {
|
pub fn new(context: Context) -> Self {
|
||||||
let (runner_tx, runner_rx) = unbounded_channel::<RunnerEvent>();
|
let (runner_tx, runner_rx) = unbounded_channel::<RunnerEvent>();
|
||||||
let (listener_tx, _) = channel::<ReporterEvent>(1024);
|
let (listener_tx, _) = channel::<ReporterEvent>(0xFFFF);
|
||||||
Self {
|
Self {
|
||||||
report: Report::new(context),
|
report: Report::new(context),
|
||||||
remaining_cases: Default::default(),
|
remaining_cases: Default::default(),
|
||||||
@@ -64,7 +64,7 @@ impl ReportAggregator {
|
|||||||
debug!("Starting to aggregate report");
|
debug!("Starting to aggregate report");
|
||||||
|
|
||||||
while let Some(event) = self.runner_rx.recv().await {
|
while let Some(event) = self.runner_rx.recv().await {
|
||||||
debug!(?event, "Received Event");
|
debug!(event = event.variant_name(), "Received Event");
|
||||||
match event {
|
match event {
|
||||||
RunnerEvent::SubscribeToEvents(event) => {
|
RunnerEvent::SubscribeToEvents(event) => {
|
||||||
self.handle_subscribe_to_events_event(*event);
|
self.handle_subscribe_to_events_event(*event);
|
||||||
@@ -412,8 +412,8 @@ impl ReportAggregator {
|
|||||||
{
|
{
|
||||||
block_information.sort_by(|a, b| {
|
block_information.sort_by(|a, b| {
|
||||||
a.ethereum_block_information
|
a.ethereum_block_information
|
||||||
.block_timestamp
|
.block_number
|
||||||
.cmp(&b.ethereum_block_information.block_timestamp)
|
.cmp(&b.ethereum_block_information.block_number)
|
||||||
});
|
});
|
||||||
|
|
||||||
// Computing the TPS.
|
// Computing the TPS.
|
||||||
@@ -466,7 +466,6 @@ impl ReportAggregator {
|
|||||||
.filter_map(|block| block.ref_time_block_fullness_percentage())
|
.filter_map(|block| block.ref_time_block_fullness_percentage())
|
||||||
.map(|v| v as u64)
|
.map(|v| v as u64)
|
||||||
.collect::<Vec<_>>();
|
.collect::<Vec<_>>();
|
||||||
dbg!(&reftime_block_fullness);
|
|
||||||
if !reftime_block_fullness.is_empty() {
|
if !reftime_block_fullness.is_empty() {
|
||||||
report
|
report
|
||||||
.metrics
|
.metrics
|
||||||
@@ -482,7 +481,6 @@ impl ReportAggregator {
|
|||||||
.filter_map(|block| block.proof_size_block_fullness_percentage())
|
.filter_map(|block| block.proof_size_block_fullness_percentage())
|
||||||
.map(|v| v as u64)
|
.map(|v| v as u64)
|
||||||
.collect::<Vec<_>>();
|
.collect::<Vec<_>>();
|
||||||
dbg!(&proof_size_block_fullness);
|
|
||||||
if !proof_size_block_fullness.is_empty() {
|
if !proof_size_block_fullness.is_empty() {
|
||||||
report
|
report
|
||||||
.metrics
|
.metrics
|
||||||
@@ -803,8 +801,9 @@ where
|
|||||||
pub fn with_list(
|
pub fn with_list(
|
||||||
&mut self,
|
&mut self,
|
||||||
platform_identifier: PlatformIdentifier,
|
platform_identifier: PlatformIdentifier,
|
||||||
mut list: Vec<T>,
|
original_list: Vec<T>,
|
||||||
) -> &mut Self {
|
) -> &mut Self {
|
||||||
|
let mut list = original_list.clone();
|
||||||
list.sort();
|
list.sort();
|
||||||
let Some(min) = list.first().copied() else {
|
let Some(min) = list.first().copied() else {
|
||||||
return self;
|
return self;
|
||||||
@@ -842,7 +841,7 @@ where
|
|||||||
.insert(platform_identifier, median);
|
.insert(platform_identifier, median);
|
||||||
self.raw
|
self.raw
|
||||||
.get_or_insert_default()
|
.get_or_insert_default()
|
||||||
.insert(platform_identifier, list);
|
.insert(platform_identifier, original_list);
|
||||||
|
|
||||||
self
|
self
|
||||||
}
|
}
|
||||||
@@ -883,6 +882,7 @@ pub struct ContractInformation {
|
|||||||
pub struct MinedBlockInformation {
|
pub struct MinedBlockInformation {
|
||||||
pub ethereum_block_information: EthereumMinedBlockInformation,
|
pub ethereum_block_information: EthereumMinedBlockInformation,
|
||||||
pub substrate_block_information: Option<SubstrateMinedBlockInformation>,
|
pub substrate_block_information: Option<SubstrateMinedBlockInformation>,
|
||||||
|
pub tx_counts: BTreeMap<StepPath, usize>,
|
||||||
}
|
}
|
||||||
|
|
||||||
impl MinedBlockInformation {
|
impl MinedBlockInformation {
|
||||||
|
|||||||
@@ -347,6 +347,16 @@ macro_rules! define_event {
|
|||||||
),*
|
),*
|
||||||
}
|
}
|
||||||
|
|
||||||
|
impl $ident {
|
||||||
|
pub fn variant_name(&self) -> &'static str {
|
||||||
|
match self {
|
||||||
|
$(
|
||||||
|
Self::$variant_ident { .. } => stringify!($variant_ident)
|
||||||
|
),*
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
$(
|
$(
|
||||||
#[derive(Debug)]
|
#[derive(Debug)]
|
||||||
$(#[$variant_meta])*
|
$(#[$variant_meta])*
|
||||||
|
|||||||
@@ -2,12 +2,13 @@
|
|||||||
|
|
||||||
use std::{
|
use std::{
|
||||||
collections::HashMap,
|
collections::HashMap,
|
||||||
|
str::FromStr,
|
||||||
sync::{LazyLock, Mutex},
|
sync::{LazyLock, Mutex},
|
||||||
};
|
};
|
||||||
|
|
||||||
use revive_dt_common::types::VersionOrRequirement;
|
use revive_dt_common::types::VersionOrRequirement;
|
||||||
|
|
||||||
use semver::Version;
|
use semver::{Version, VersionReq};
|
||||||
use sha2::{Digest, Sha256};
|
use sha2::{Digest, Sha256};
|
||||||
|
|
||||||
use crate::list::List;
|
use crate::list::List;
|
||||||
@@ -65,6 +66,9 @@ impl SolcDownloader {
|
|||||||
target: &'static str,
|
target: &'static str,
|
||||||
list: &'static str,
|
list: &'static str,
|
||||||
) -> anyhow::Result<Self> {
|
) -> anyhow::Result<Self> {
|
||||||
|
static MAXIMUM_COMPILER_VERSION_REQUIREMENT: LazyLock<VersionReq> =
|
||||||
|
LazyLock::new(|| VersionReq::from_str("<=0.8.30").unwrap());
|
||||||
|
|
||||||
let version_or_requirement = version.into();
|
let version_or_requirement = version.into();
|
||||||
match version_or_requirement {
|
match version_or_requirement {
|
||||||
VersionOrRequirement::Version(version) => Ok(Self {
|
VersionOrRequirement::Version(version) => Ok(Self {
|
||||||
@@ -79,7 +83,10 @@ impl SolcDownloader {
|
|||||||
.builds
|
.builds
|
||||||
.into_iter()
|
.into_iter()
|
||||||
.map(|build| build.version)
|
.map(|build| build.version)
|
||||||
.filter(|version| requirement.matches(version))
|
.filter(|version| {
|
||||||
|
MAXIMUM_COMPILER_VERSION_REQUIREMENT.matches(version)
|
||||||
|
&& requirement.matches(version)
|
||||||
|
})
|
||||||
.max()
|
.max()
|
||||||
else {
|
else {
|
||||||
anyhow::bail!("Failed to find a version that satisfies {requirement:?}");
|
anyhow::bail!("Failed to find a version that satisfies {requirement:?}");
|
||||||
|
|||||||
-1
Submodule polkadot-sdk deleted from dc3d0e5ab7
+1
-1
Submodule resolc-compiler-tests updated: ce77cb1166...7bc445491e
@@ -0,0 +1,246 @@
|
|||||||
|
"""
|
||||||
|
Utilities to print benchmark metrics from a report JSON into CSV.
|
||||||
|
|
||||||
|
Usage:
|
||||||
|
python scripts/print_benchmark_metrics_csv.py /absolute/path/to/report.json
|
||||||
|
|
||||||
|
The script prints, for each metadata path, case index, and mode combination,
|
||||||
|
CSV rows aligned to mined blocks with the following columns:
|
||||||
|
- block_number
|
||||||
|
- number_of_txs
|
||||||
|
- tps (transaction_per_second)
|
||||||
|
- gps (gas_per_second)
|
||||||
|
- gas_block_fullness
|
||||||
|
- ref_time (if available)
|
||||||
|
- max_ref_time (if available)
|
||||||
|
- proof_size (if available)
|
||||||
|
- max_proof_size (if available)
|
||||||
|
- ref_time_block_fullness (if available)
|
||||||
|
- proof_size_block_fullness (if available)
|
||||||
|
|
||||||
|
Important nuance: TPS and GPS arrays have (number_of_blocks - 1) items. The
|
||||||
|
first block row has no TPS/GPS; the CSV leaves those cells empty for the first
|
||||||
|
row and aligns subsequent values to their corresponding next block.
|
||||||
|
"""
|
||||||
|
|
||||||
|
from __future__ import annotations
|
||||||
|
|
||||||
|
import json
|
||||||
|
import sys
|
||||||
|
import csv
|
||||||
|
from typing import List, Mapping, TypedDict
|
||||||
|
|
||||||
|
|
||||||
|
class EthereumMinedBlockInformation(TypedDict):
|
||||||
|
"""EVM block information extracted from the report.
|
||||||
|
|
||||||
|
Attributes:
|
||||||
|
block_number: The block height.
|
||||||
|
block_timestamp: The UNIX timestamp of the block.
|
||||||
|
mined_gas: Total gas used (mined) in the block.
|
||||||
|
block_gas_limit: The gas limit of the block.
|
||||||
|
transaction_hashes: List of transaction hashes included in the block.
|
||||||
|
"""
|
||||||
|
|
||||||
|
block_number: int
|
||||||
|
block_timestamp: int
|
||||||
|
mined_gas: int
|
||||||
|
block_gas_limit: int
|
||||||
|
transaction_hashes: List[str]
|
||||||
|
|
||||||
|
|
||||||
|
class SubstrateMinedBlockInformation(TypedDict):
|
||||||
|
"""Substrate-specific block resource usage fields.
|
||||||
|
|
||||||
|
Attributes:
|
||||||
|
ref_time: The consumed ref time in the block.
|
||||||
|
max_ref_time: The maximum ref time allowed for the block.
|
||||||
|
proof_size: The consumed proof size in the block.
|
||||||
|
max_proof_size: The maximum proof size allowed for the block.
|
||||||
|
"""
|
||||||
|
|
||||||
|
ref_time: int
|
||||||
|
max_ref_time: int
|
||||||
|
proof_size: int
|
||||||
|
max_proof_size: int
|
||||||
|
|
||||||
|
|
||||||
|
class MinedBlockInformation(TypedDict):
|
||||||
|
"""Block-level information for a mined block with both EVM and optional Substrate fields."""
|
||||||
|
|
||||||
|
ethereum_block_information: EthereumMinedBlockInformation
|
||||||
|
substrate_block_information: SubstrateMinedBlockInformation
|
||||||
|
|
||||||
|
|
||||||
|
class Metric(TypedDict):
|
||||||
|
"""Metric data of integer values keyed by platform identifier.
|
||||||
|
|
||||||
|
Attributes:
|
||||||
|
minimum: Single scalar minimum per platform.
|
||||||
|
maximum: Single scalar maximum per platform.
|
||||||
|
mean: Single scalar mean per platform.
|
||||||
|
median: Single scalar median per platform.
|
||||||
|
raw: Time-series (or list) of values per platform.
|
||||||
|
"""
|
||||||
|
|
||||||
|
minimum: Mapping[str, int]
|
||||||
|
maximum: Mapping[str, int]
|
||||||
|
mean: Mapping[str, int]
|
||||||
|
median: Mapping[str, int]
|
||||||
|
raw: Mapping[str, List[int]]
|
||||||
|
|
||||||
|
|
||||||
|
class Metrics(TypedDict):
|
||||||
|
"""All metrics that may be present for a given execution report.
|
||||||
|
|
||||||
|
Note that some metrics are optional and present only for specific platforms
|
||||||
|
or execution modes.
|
||||||
|
"""
|
||||||
|
|
||||||
|
transaction_per_second: Metric
|
||||||
|
gas_per_second: Metric
|
||||||
|
gas_block_fullness: Metric
|
||||||
|
ref_time_block_fullness: Metric
|
||||||
|
proof_size_block_fullness: Metric
|
||||||
|
|
||||||
|
|
||||||
|
class ExecutionReport(TypedDict):
|
||||||
|
"""Execution report for a mode containing mined blocks and metrics.
|
||||||
|
|
||||||
|
Attributes:
|
||||||
|
mined_block_information: Mapping from platform identifier to the list of
|
||||||
|
mined blocks observed for that platform.
|
||||||
|
metrics: The computed metrics for the execution.
|
||||||
|
"""
|
||||||
|
|
||||||
|
mined_block_information: Mapping[str, List[MinedBlockInformation]]
|
||||||
|
metrics: Metrics
|
||||||
|
|
||||||
|
|
||||||
|
class CaseReport(TypedDict):
|
||||||
|
"""Report for a single case, keyed by mode string."""
|
||||||
|
|
||||||
|
mode_execution_reports: Mapping[str, ExecutionReport]
|
||||||
|
|
||||||
|
|
||||||
|
class MetadataFileReport(TypedDict):
|
||||||
|
"""Report subtree keyed by case indices for a metadata file path."""
|
||||||
|
|
||||||
|
case_reports: Mapping[str, CaseReport]
|
||||||
|
|
||||||
|
|
||||||
|
class ReportRoot(TypedDict):
|
||||||
|
"""Top-level report schema with execution information keyed by metadata path."""
|
||||||
|
|
||||||
|
execution_information: Mapping[str, MetadataFileReport]
|
||||||
|
|
||||||
|
|
||||||
|
BlockInformation = TypedDict(
|
||||||
|
"BlockInformation",
|
||||||
|
{
|
||||||
|
"Block Number": int,
|
||||||
|
"Timestamp": int,
|
||||||
|
"Datetime": None,
|
||||||
|
"Transaction Count": int,
|
||||||
|
"TPS": int | None,
|
||||||
|
"GPS": int | None,
|
||||||
|
"Ref Time": int,
|
||||||
|
"Max Ref Time": int,
|
||||||
|
"Block Fullness Ref Time": int,
|
||||||
|
"Proof Size": int,
|
||||||
|
"Max Proof Size": int,
|
||||||
|
"Block Fullness Proof Size": int,
|
||||||
|
},
|
||||||
|
)
|
||||||
|
"""A typed dictionary used to hold all of the block information"""
|
||||||
|
|
||||||
|
|
||||||
|
def load_report(path: str) -> ReportRoot:
|
||||||
|
"""Load the report JSON from disk.
|
||||||
|
|
||||||
|
Args:
|
||||||
|
path: Absolute or relative filesystem path to the JSON report file.
|
||||||
|
|
||||||
|
Returns:
|
||||||
|
The parsed report as a typed dictionary structure.
|
||||||
|
"""
|
||||||
|
|
||||||
|
with open(path, "r", encoding="utf-8") as f:
|
||||||
|
data: ReportRoot = json.load(f)
|
||||||
|
return data
|
||||||
|
|
||||||
|
|
||||||
|
def main() -> None:
|
||||||
|
report_path: str = sys.argv[1]
|
||||||
|
report: ReportRoot = load_report(report_path)
|
||||||
|
|
||||||
|
# TODO: Remove this in the future, but for now, the target is fixed.
|
||||||
|
target: str = "revive-dev-node-revm-solc"
|
||||||
|
|
||||||
|
csv_writer = csv.writer(sys.stdout)
|
||||||
|
|
||||||
|
for _, metadata_file_report in report["execution_information"].items():
|
||||||
|
for _, case_report in metadata_file_report["case_reports"].items():
|
||||||
|
for _, execution_report in case_report["mode_execution_reports"].items():
|
||||||
|
blocks_information: list[MinedBlockInformation] = execution_report[
|
||||||
|
"mined_block_information"
|
||||||
|
][target]
|
||||||
|
|
||||||
|
resolved_blocks: list[BlockInformation] = []
|
||||||
|
for i, block_information in enumerate(blocks_information):
|
||||||
|
resolved_blocks.append(
|
||||||
|
{
|
||||||
|
"Block Number": block_information[
|
||||||
|
"ethereum_block_information"
|
||||||
|
]["block_number"],
|
||||||
|
"Timestamp": block_information[
|
||||||
|
"ethereum_block_information"
|
||||||
|
]["block_timestamp"],
|
||||||
|
"Datetime": None,
|
||||||
|
"Transaction Count": len(
|
||||||
|
block_information["ethereum_block_information"][
|
||||||
|
"transaction_hashes"
|
||||||
|
]
|
||||||
|
),
|
||||||
|
"TPS": (
|
||||||
|
None
|
||||||
|
if i == 0
|
||||||
|
else execution_report["metrics"][
|
||||||
|
"transaction_per_second"
|
||||||
|
]["raw"][target][i - 1]
|
||||||
|
),
|
||||||
|
"GPS": (
|
||||||
|
None
|
||||||
|
if i == 0
|
||||||
|
else execution_report["metrics"]["gas_per_second"][
|
||||||
|
"raw"
|
||||||
|
][target][i - 1]
|
||||||
|
),
|
||||||
|
"Ref Time": block_information[
|
||||||
|
"substrate_block_information"
|
||||||
|
]["ref_time"],
|
||||||
|
"Max Ref Time": block_information[
|
||||||
|
"substrate_block_information"
|
||||||
|
]["max_ref_time"],
|
||||||
|
"Block Fullness Ref Time": execution_report["metrics"][
|
||||||
|
"ref_time_block_fullness"
|
||||||
|
]["raw"][target][i],
|
||||||
|
"Proof Size": block_information[
|
||||||
|
"substrate_block_information"
|
||||||
|
]["proof_size"],
|
||||||
|
"Max Proof Size": block_information[
|
||||||
|
"substrate_block_information"
|
||||||
|
]["max_proof_size"],
|
||||||
|
"Block Fullness Proof Size": execution_report["metrics"][
|
||||||
|
"proof_size_block_fullness"
|
||||||
|
]["raw"][target][i],
|
||||||
|
}
|
||||||
|
)
|
||||||
|
|
||||||
|
csv_writer = csv.DictWriter(sys.stdout, resolved_blocks[0].keys())
|
||||||
|
csv_writer.writeheader()
|
||||||
|
csv_writer.writerows(resolved_blocks)
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
main()
|
||||||
@@ -0,0 +1,226 @@
|
|||||||
|
"""
|
||||||
|
This script is used to turn the JSON report produced by the revive differential tests tool into an
|
||||||
|
easy to consume markdown document for the purpose of reporting this information in the Polkadot SDK
|
||||||
|
CI. The full models used in the JSON report can be found in the revive differential tests repo and
|
||||||
|
the models used in this script are just a partial reproduction of the full report models.
|
||||||
|
"""
|
||||||
|
|
||||||
|
from typing import TypedDict, Literal, Union
|
||||||
|
|
||||||
|
import json, io
|
||||||
|
|
||||||
|
|
||||||
|
class Report(TypedDict):
|
||||||
|
context: "Context"
|
||||||
|
execution_information: dict[
|
||||||
|
"MetadataFilePathString",
|
||||||
|
dict["ModeString", dict["CaseIdxString", "CaseReport"]],
|
||||||
|
]
|
||||||
|
|
||||||
|
|
||||||
|
class Context(TypedDict):
|
||||||
|
Test: "TestContext"
|
||||||
|
|
||||||
|
|
||||||
|
class TestContext(TypedDict):
|
||||||
|
corpus_configuration: "CorpusConfiguration"
|
||||||
|
|
||||||
|
|
||||||
|
class CorpusConfiguration(TypedDict):
|
||||||
|
test_specifiers: list["TestSpecifier"]
|
||||||
|
|
||||||
|
|
||||||
|
class CaseReport(TypedDict):
|
||||||
|
status: "CaseStatus"
|
||||||
|
|
||||||
|
|
||||||
|
class CaseStatusSuccess(TypedDict):
|
||||||
|
status: Literal["Succeeded"]
|
||||||
|
steps_executed: int
|
||||||
|
|
||||||
|
|
||||||
|
class CaseStatusFailure(TypedDict):
|
||||||
|
status: Literal["Failed"]
|
||||||
|
reason: str
|
||||||
|
|
||||||
|
|
||||||
|
class CaseStatusIgnored(TypedDict):
|
||||||
|
status: Literal["Ignored"]
|
||||||
|
reason: str
|
||||||
|
|
||||||
|
|
||||||
|
CaseStatus = Union[CaseStatusSuccess, CaseStatusFailure, CaseStatusIgnored]
|
||||||
|
"""A union type of all of the possible statuses that could be reported for a case."""
|
||||||
|
|
||||||
|
TestSpecifier = str
|
||||||
|
"""A test specifier string. For example resolc-compiler-tests/fixtures/solidity/test.json::0::Y+"""
|
||||||
|
|
||||||
|
ModeString = str
|
||||||
|
"""The mode string. For example Y+ >=0.8.13"""
|
||||||
|
|
||||||
|
MetadataFilePathString = str
|
||||||
|
"""The path to a metadata file. For example resolc-compiler-tests/fixtures/solidity/test.json"""
|
||||||
|
|
||||||
|
CaseIdxString = str
|
||||||
|
"""The index of a case as a string. For example '0'"""
|
||||||
|
|
||||||
|
|
||||||
|
def path_relative_to_resolc_compiler_test_directory(path: str) -> str:
|
||||||
|
"""
|
||||||
|
Given a path, this function returns the path relative to the resolc-compiler-test directory. The
|
||||||
|
following is an example of an input and an output:
|
||||||
|
|
||||||
|
Input: ~/polkadot-sdk/revive-differential-tests/resolc-compiler-tests/fixtures/solidity/test.json
|
||||||
|
Output: test.json
|
||||||
|
"""
|
||||||
|
|
||||||
|
return f"{path.split('resolc-compiler-tests/fixtures/solidity')[-1].strip('/')}"
|
||||||
|
|
||||||
|
|
||||||
|
def main() -> None:
|
||||||
|
with open("report.json", "r") as file:
|
||||||
|
report: Report = json.load(file)
|
||||||
|
|
||||||
|
# Starting the markdown document and adding information to it as we go.
|
||||||
|
markdown_document: io.TextIOWrapper = open("report.md", "w")
|
||||||
|
print("# Differential Tests Results", file=markdown_document)
|
||||||
|
|
||||||
|
# Getting all of the test specifiers from the report and making them relative to the tests dir.
|
||||||
|
test_specifiers: list[str] = list(
|
||||||
|
map(
|
||||||
|
path_relative_to_resolc_compiler_test_directory,
|
||||||
|
report["context"]["Test"]["corpus_configuration"]["test_specifiers"],
|
||||||
|
)
|
||||||
|
)
|
||||||
|
print("## Specified Tests", file=markdown_document)
|
||||||
|
for test_specifier in test_specifiers:
|
||||||
|
print(f"* `{test_specifier}`", file=markdown_document)
|
||||||
|
|
||||||
|
# Counting the total number of test cases, successes, failures, and ignored tests
|
||||||
|
total_number_of_cases: int = 0
|
||||||
|
total_number_of_successes: int = 0
|
||||||
|
total_number_of_failures: int = 0
|
||||||
|
total_number_of_ignores: int = 0
|
||||||
|
for _, mode_to_case_mapping in report["execution_information"].items():
|
||||||
|
for _, case_idx_to_report_mapping in mode_to_case_mapping.items():
|
||||||
|
for _, case_report in case_idx_to_report_mapping.items():
|
||||||
|
status: CaseStatus = case_report["status"]
|
||||||
|
|
||||||
|
total_number_of_cases += 1
|
||||||
|
if status["status"] == "Succeeded":
|
||||||
|
total_number_of_successes += 1
|
||||||
|
elif status["status"] == "Failed":
|
||||||
|
total_number_of_failures += 1
|
||||||
|
elif status["status"] == "Ignored":
|
||||||
|
total_number_of_ignores += 1
|
||||||
|
else:
|
||||||
|
raise Exception(
|
||||||
|
f"Encountered a status that's unknown to the script: {status}"
|
||||||
|
)
|
||||||
|
|
||||||
|
print("## Counts", file=markdown_document)
|
||||||
|
print(
|
||||||
|
f"* **Total Number of Test Cases:** {total_number_of_cases}",
|
||||||
|
file=markdown_document,
|
||||||
|
)
|
||||||
|
print(
|
||||||
|
f"* **Total Number of Successes:** {total_number_of_successes}",
|
||||||
|
file=markdown_document,
|
||||||
|
)
|
||||||
|
print(
|
||||||
|
f"* **Total Number of Failures:** {total_number_of_failures}",
|
||||||
|
file=markdown_document,
|
||||||
|
)
|
||||||
|
print(
|
||||||
|
f"* **Total Number of Ignores:** {total_number_of_ignores}",
|
||||||
|
file=markdown_document,
|
||||||
|
)
|
||||||
|
|
||||||
|
# Grouping the various test cases into dictionaries and groups depending on their status to make
|
||||||
|
# them easier to include in the markdown document later on.
|
||||||
|
successful_cases: dict[
|
||||||
|
MetadataFilePathString, dict[CaseIdxString, set[ModeString]]
|
||||||
|
] = {}
|
||||||
|
for metadata_file_path, mode_to_case_mapping in report[
|
||||||
|
"execution_information"
|
||||||
|
].items():
|
||||||
|
for mode_string, case_idx_to_report_mapping in mode_to_case_mapping.items():
|
||||||
|
for case_idx_string, case_report in case_idx_to_report_mapping.items():
|
||||||
|
status: CaseStatus = case_report["status"]
|
||||||
|
metadata_file_path: str = (
|
||||||
|
path_relative_to_resolc_compiler_test_directory(metadata_file_path)
|
||||||
|
)
|
||||||
|
mode_string: str = mode_string.replace(" M3", "+").replace(" M0", "-")
|
||||||
|
|
||||||
|
if status["status"] == "Succeeded":
|
||||||
|
successful_cases.setdefault(
|
||||||
|
metadata_file_path,
|
||||||
|
{},
|
||||||
|
).setdefault(
|
||||||
|
case_idx_string, set()
|
||||||
|
).add(mode_string)
|
||||||
|
|
||||||
|
print("## Failures", file=markdown_document)
|
||||||
|
print(
|
||||||
|
"The test specifiers seen in this section have the format 'path::case_idx::compilation_mode'\
|
||||||
|
and they're compatible with the revive differential tests framework and can be specified\
|
||||||
|
to it directly in the same way that they're provided through the `--test` argument of the\
|
||||||
|
framework.\n",
|
||||||
|
file=markdown_document,
|
||||||
|
)
|
||||||
|
print(
|
||||||
|
"The failures are provided in an expandable section to ensure that the PR does not get \
|
||||||
|
polluted with information. Please click on the section below for more information",
|
||||||
|
file=markdown_document,
|
||||||
|
)
|
||||||
|
print(
|
||||||
|
"<details><summary>Detailed Differential Tests Failure Information</summary>\n\n",
|
||||||
|
file=markdown_document,
|
||||||
|
)
|
||||||
|
print("| Test Specifier | Failure Reason | Note |", file=markdown_document)
|
||||||
|
print("| -- | -- | -- |", file=markdown_document)
|
||||||
|
|
||||||
|
for metadata_file_path, mode_to_case_mapping in report[
|
||||||
|
"execution_information"
|
||||||
|
].items():
|
||||||
|
for mode_string, case_idx_to_report_mapping in mode_to_case_mapping.items():
|
||||||
|
for case_idx_string, case_report in case_idx_to_report_mapping.items():
|
||||||
|
status: CaseStatus = case_report["status"]
|
||||||
|
metadata_file_path: str = (
|
||||||
|
path_relative_to_resolc_compiler_test_directory(metadata_file_path)
|
||||||
|
)
|
||||||
|
mode_string: str = mode_string.replace(" M3", "+").replace(" M0", "-")
|
||||||
|
|
||||||
|
if status["status"] != "Failed":
|
||||||
|
continue
|
||||||
|
|
||||||
|
failure_reason: str = status["reason"].replace("\n", " ")
|
||||||
|
|
||||||
|
note: str = ""
|
||||||
|
modes_where_this_case_succeeded: set[ModeString] = (
|
||||||
|
successful_cases.setdefault(
|
||||||
|
metadata_file_path,
|
||||||
|
{},
|
||||||
|
).setdefault(case_idx_string, set())
|
||||||
|
)
|
||||||
|
if len(modes_where_this_case_succeeded) != 0:
|
||||||
|
note: str = (
|
||||||
|
f"This test case succeeded with other compilation modes: {modes_where_this_case_succeeded}"
|
||||||
|
)
|
||||||
|
|
||||||
|
test_specifier: str = (
|
||||||
|
f"{metadata_file_path}::{case_idx_string}::{mode_string}"
|
||||||
|
)
|
||||||
|
print(
|
||||||
|
f"| `{test_specifier}` | `{failure_reason}` | {note} |",
|
||||||
|
file=markdown_document,
|
||||||
|
)
|
||||||
|
print("\n\n</details>", file=markdown_document)
|
||||||
|
|
||||||
|
# The primary downside of not using `with`, but I guess it's better since I don't want to over
|
||||||
|
# indent the code.
|
||||||
|
markdown_document.close()
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
main()
|
||||||
@@ -22,7 +22,6 @@ POLKADOT_SDK_DIR="${1:-}"
|
|||||||
# Binary paths (default to names in $PATH)
|
# Binary paths (default to names in $PATH)
|
||||||
REVIVE_DEV_NODE_BIN="revive-dev-node"
|
REVIVE_DEV_NODE_BIN="revive-dev-node"
|
||||||
ETH_RPC_BIN="eth-rpc"
|
ETH_RPC_BIN="eth-rpc"
|
||||||
SUBSTRATE_NODE_BIN="substrate-node"
|
|
||||||
|
|
||||||
echo -e "${GREEN}=== Revive Differential Tests Quick Start ===${NC}"
|
echo -e "${GREEN}=== Revive Differential Tests Quick Start ===${NC}"
|
||||||
echo ""
|
echo ""
|
||||||
@@ -50,14 +49,13 @@ if [ -n "$POLKADOT_SDK_DIR" ]; then
|
|||||||
|
|
||||||
REVIVE_DEV_NODE_BIN="$POLKADOT_SDK_DIR/target/release/revive-dev-node"
|
REVIVE_DEV_NODE_BIN="$POLKADOT_SDK_DIR/target/release/revive-dev-node"
|
||||||
ETH_RPC_BIN="$POLKADOT_SDK_DIR/target/release/eth-rpc"
|
ETH_RPC_BIN="$POLKADOT_SDK_DIR/target/release/eth-rpc"
|
||||||
SUBSTRATE_NODE_BIN="$POLKADOT_SDK_DIR/target/release/substrate-node"
|
|
||||||
|
|
||||||
if [ ! -x "$REVIVE_DEV_NODE_BIN" ] || [ ! -x "$ETH_RPC_BIN" ] || [ ! -x "$SUBSTRATE_NODE_BIN" ]; then
|
if [ ! -x "$REVIVE_DEV_NODE_BIN" ] || [ ! -x "$ETH_RPC_BIN" ]; then
|
||||||
echo -e "${YELLOW}Required binaries not found in release target. Building...${NC}"
|
echo -e "${YELLOW}Required binaries not found in release target. Building...${NC}"
|
||||||
(cd "$POLKADOT_SDK_DIR" && cargo build --release --package staging-node-cli --package pallet-revive-eth-rpc --package revive-dev-node)
|
(cd "$POLKADOT_SDK_DIR" && cargo build --release --package staging-node-cli --package pallet-revive-eth-rpc --package revive-dev-node)
|
||||||
fi
|
fi
|
||||||
|
|
||||||
for bin in "$REVIVE_DEV_NODE_BIN" "$ETH_RPC_BIN" "$SUBSTRATE_NODE_BIN"; do
|
for bin in "$REVIVE_DEV_NODE_BIN" "$ETH_RPC_BIN"; do
|
||||||
if [ ! -x "$bin" ]; then
|
if [ ! -x "$bin" ]; then
|
||||||
echo -e "${RED}Expected binary not found after build: $bin${NC}"
|
echo -e "${RED}Expected binary not found after build: $bin${NC}"
|
||||||
exit 1
|
exit 1
|
||||||
@@ -84,7 +82,6 @@ RUST_LOG="info,alloy_pubsub::service=error" ./target/release/retester test \
|
|||||||
--concurrency.number-of-threads 5 \
|
--concurrency.number-of-threads 5 \
|
||||||
--concurrency.number-of-concurrent-tasks 500 \
|
--concurrency.number-of-concurrent-tasks 500 \
|
||||||
--wallet.additional-keys 100000 \
|
--wallet.additional-keys 100000 \
|
||||||
--kitchensink.path "$SUBSTRATE_NODE_BIN" \
|
|
||||||
--revive-dev-node.path "$REVIVE_DEV_NODE_BIN" \
|
--revive-dev-node.path "$REVIVE_DEV_NODE_BIN" \
|
||||||
--eth-rpc.path "$ETH_RPC_BIN" \
|
--eth-rpc.path "$ETH_RPC_BIN" \
|
||||||
> logs.log \
|
> logs.log \
|
||||||
Reference in New Issue
Block a user