Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
cf4f2fd
Squashed 'lightwallet-protocol/' content from commit 23f0768ea
pacu Apr 7, 2026
658bb9c
add: gRPC parity test suite (Zainod vs. Lightwalletd)
pacu Apr 8, 2026
55e262c
fix: address Copilot review findings in gRPC parity test suite
pacu Apr 10, 2026
3b77128
fix: rework grpc_comparison chain setup to use submitblock and minera…
pacu Apr 14, 2026
a53682e
Add cached gRPC parity fixture for Zebra-backed comparison
pacu Apr 23, 2026
43adbb1
Refactor grpc comparison test flow for readability
pacu Apr 23, 2026
78e066e
add .claude and .codex to gitignore
pacu Apr 23, 2026
6890813
Document how the gRPC comparison test was stabilized
pacu Apr 23, 2026
4fa9b8e
Refine grpc parity test maintenance and cache handling
pacu Apr 23, 2026
5d50589
Address PR review feedback for grpc parity test
pacu Apr 23, 2026
9483e91
Clarify upstream bug follow-up in grpc comparison docs
pacu Apr 23, 2026
f32f66b
Tighten shielded GetBlockRange parity checks
pacu Apr 24, 2026
9c18ffe
Make grpc compact block parity checks fully strict
pacu Apr 24, 2026
8f458ba
Show unified diffs for grpc compact block mismatches
pacu Apr 26, 2026
5c902de
Use uv consistently in test documentation
pacu Apr 26, 2026
f58389d
Avoid running the grpc comparison tests on the per-change run
pacu Apr 26, 2026
478f703
Fail fast on missing persistent wallet caches
pacu Apr 26, 2026
babe50d
Run grpc comparison with new tests
pacu Apr 28, 2026
6f5a869
Fix Python lints for grpc comparison files
pacu Jun 10, 2026
f6d7828
Check out test tree before RPC dependency sync
pacu Jun 11, 2026
f6fd81d
Avoid persisted checkout credentials for lightwalletd
pacu Jun 11, 2026
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
145 changes: 129 additions & 16 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ on:
- zebra-interop-request
- zaino-interop-request
- zallet-interop-request
- lightwalletd-interop-request

permissions:
contents: read
Expand Down Expand Up @@ -617,6 +618,84 @@ jobs:
requesting-repository: ${{ steps.repo-ids.outputs.requesting-repository }}
job-name: "Build zallet on ${{ matrix.platform }}${{ matrix.required_suffix }}"

build-lightwalletd:
name: Build lightwalletd on ${{ matrix.platform }}${{ matrix.required_suffix }}
needs: setup
runs-on: ${{ matrix.build_os }}
container:
image: ${{ matrix.container }}
env:
HOME: /root
PATH: /root/.cargo/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:$PATH
continue-on-error: ${{ !matrix.required }}
strategy:
matrix:
include: ${{ fromJson(needs.setup.outputs.build_matrix) }}

steps:
- name: Check out integration-tests to access actions
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
with:
path: integration-tests
persist-credentials: false

- name: Compute interop repo ids
id: repo-ids
uses: ./integration-tests/.github/actions/interop-repo-ids

- id: start-interop
uses: ./integration-tests/.github/actions/start-interop
with:
status-app-id: ${{ secrets.STATUS_APP_ID }}
status-app-private-key: ${{ secrets.STATUS_APP_PRIVATE_KEY }}
requesting-owner: ${{ steps.repo-ids.outputs.requesting-owner }}
requesting-repository: ${{ steps.repo-ids.outputs.requesting-repository }}
job-name: "Build lightwalletd on ${{ matrix.platform }}${{ matrix.required_suffix }}"

- name: Use specified zcash/lightwalletd commit
if: github.event.action == 'lightwalletd-interop-request'
shell: sh
env:
SHA: ${{ github.event.client_payload.sha }}
run: echo "LIGHTWALLETD_REF=${SHA}" >> $GITHUB_ENV

- name: Use zcash/lightwalletd current master
if: github.event.action != 'lightwalletd-interop-request'
run: echo "LIGHTWALLETD_REF=refs/heads/master" >> $GITHUB_ENV

- name: Check out zcash/lightwalletd
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
with:
repository: zcash/lightwalletd
ref: ${{ env.LIGHTWALLETD_REF }}
path: lightwalletd
Comment thread
github-advanced-security[bot] marked this conversation as resolved.
Fixed
persist-credentials: false

- name: Set up Go
uses: actions/setup-go@0aaccfd150d50ccaeb58ebd88d36e91967a5f35b # v5.4.0
with:
go-version-file: lightwalletd/go.mod
cache-dependency-path: lightwalletd/go.sum

- name: Build lightwalletd
run: go build -v -o lightwalletd${{ matrix.file_ext }} .
working-directory: ./lightwalletd

- name: Upload lightwalletd
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
with:
name: lightwalletd-${{ matrix.name }}
path: |
${{ format('./lightwalletd/lightwalletd{0}', matrix.file_ext) }}

- uses: ./integration-tests/.github/actions/finish-interop
if: always()
with:
app-token: ${{ steps.start-interop.outputs.app-token }}
requesting-owner: ${{ steps.repo-ids.outputs.requesting-owner }}
requesting-repository: ${{ steps.repo-ids.outputs.requesting-repository }}
job-name: "Build lightwalletd on ${{ matrix.platform }}${{ matrix.required_suffix }}"

# Not working in Windows
sec-hard:
name: sec-hard ${{ matrix.platform }}${{ matrix.required_suffix }}
Expand All @@ -625,6 +704,7 @@ jobs:
- build-zebra
- build-zaino
- build-zallet
- build-lightwalletd
runs-on: ${{ matrix.test_os }}
container:
image: ${{ matrix.container }}
Expand Down Expand Up @@ -685,12 +765,19 @@ jobs:
name: zallet-${{ matrix.name }}
path: ./src

- name: Download lightwalletd artifact
uses: actions/download-artifact@d3f86a106a0bac45b974a628896c90dbdf5c8093 # v4.3.0
with:
name: lightwalletd-${{ matrix.name }}
path: ./src

- name: Make artifact executable
if: runner.os != 'Windows'
run: |
chmod +x ${{ format('./src/zebrad{0}', matrix.file_ext) }}
chmod +x ${{ format('./src/zainod{0}', matrix.file_ext) }}
chmod +x ${{ format('./src/zallet{0}', matrix.file_ext) }}
chmod +x ${{ format('./src/lightwalletd{0}', matrix.file_ext) }}

- name: Run sec-hard test
shell: bash
Expand Down Expand Up @@ -722,18 +809,36 @@ jobs:
include: ${{ fromJson(needs.setup.outputs.rpc_test_matrix) }}

steps:
- name: Cache Python dependencies for RPC tests
- name: Select the test branch
id: test-branch
if: endsWith(github.event.action, '-interop-request')
env:
TEST_SHA: ${{ github.event.client_payload.test_sha }}
run: |
if [ "${TEST_SHA}" != "null" ] && [ -n "${TEST_SHA}" ]; then
echo "test_sha=${TEST_SHA}" >> $GITHUB_OUTPUT
fi

- name: Check out the selected test branch
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
with:
# Empty on direct repo PRs, which selects the default.
ref: ${{ steps.test-branch.outputs.test_sha }}
persist-credentials: false

- name: Cache uv project environment for RPC tests
uses: actions/cache@cdf6c1fa76f9f475f3d7449005a359c84ca0f306 # v5.0.3
with:
enableCrossOsArchive: true
path: venv
key: test-rpc-venv-${{ matrix.name }}
path: .venv
key: test-rpc-venv-${{ matrix.name }}-${{ hashFiles('pyproject.toml', 'uv.lock') }}

- name: Install uv
run: python3 -m pip install uv

- name: Get Python dependencies for RPC tests
- name: Sync Python dependencies for RPC tests
run: |
python3 -m venv ./venv
. ./venv/bin/activate
pip install zmq asyncio base58 toml embit
uv sync --frozen

test-rpc:
name: RPC tests ${{ matrix.platform }} ${{ matrix.shard }}${{ matrix.required_suffix }}
Expand All @@ -742,6 +847,7 @@ jobs:
- build-zebra
- build-zaino
- build-zallet
- build-lightwalletd
- rpc-depends
runs-on: ${{ matrix.test_os }}
container:
Expand Down Expand Up @@ -788,18 +894,19 @@ jobs:
requesting-repository: ${{ steps.repo-ids.outputs.requesting-repository }}
job-name: "RPC tests ${{ matrix.platform }} ${{ matrix.shard }}${{ matrix.required_suffix }}"

- name: Cache Python dependencies for RPC tests
- name: Cache uv project environment for RPC tests
uses: actions/cache@cdf6c1fa76f9f475f3d7449005a359c84ca0f306 # v5.0.3
with:
enableCrossOsArchive: true
path: venv
key: test-rpc-venv-${{ matrix.name }}
path: .venv
key: test-rpc-venv-${{ matrix.name }}-${{ hashFiles('pyproject.toml', 'uv.lock') }}

- name: Get Python dependencies for RPC tests if not cached
- name: Install uv
run: python3 -m pip install uv

- name: Sync Python dependencies for RPC tests
run: |
python3 -m venv ./venv
. ./venv/bin/activate
pip install zmq asyncio base58 toml embit
uv sync --frozen

- name: Download zebrad artifact
uses: actions/download-artifact@d3f86a106a0bac45b974a628896c90dbdf5c8093 # v4.3.0
Expand All @@ -819,12 +926,19 @@ jobs:
name: zallet-${{ matrix.name }}
path: ./src

- name: Download lightwalletd artifact
uses: actions/download-artifact@d3f86a106a0bac45b974a628896c90dbdf5c8093 # v4.3.0
with:
name: lightwalletd-${{ matrix.name }}
path: ./src

- name: Make artifact executable
if: runner.os != 'Windows'
run: |
chmod +x ${{ format('./src/zebrad{0}', matrix.file_ext) }}
chmod +x ${{ format('./src/zainod{0}', matrix.file_ext) }}
chmod +x ${{ format('./src/zallet{0}', matrix.file_ext) }}
chmod +x ${{ format('./src/lightwalletd{0}', matrix.file_ext) }}

- name: Get Sprout parameters
uses: actions/cache@cdf6c1fa76f9f475f3d7449005a359c84ca0f306 # v5.0.3
Expand Down Expand Up @@ -880,8 +994,7 @@ jobs:
if all_passed == False:
sys.exit(1)
EOF
. ./venv/bin/activate
ZEBRAD=$(pwd)/${{ format('src/zebrad{0}', matrix.file_ext) }} ZAINOD=$(pwd)/${{ format('src/zainod{0}', matrix.file_ext) }} ZALLET=$(pwd)/${{ format('src/zallet{0}', matrix.file_ext) }} SRC_DIR=$(pwd) python3 ./subclass.py
ZEBRAD=$(pwd)/${{ format('src/zebrad{0}', matrix.file_ext) }} ZAINOD=$(pwd)/${{ format('src/zainod{0}', matrix.file_ext) }} ZALLET=$(pwd)/${{ format('src/zallet{0}', matrix.file_ext) }} LIGHTWALLETD=$(pwd)/${{ format('src/lightwalletd{0}', matrix.file_ext) }} SRC_DIR=$(pwd) uv run python3 ./subclass.py

- uses: ./.github/actions/finish-interop
if: always()
Expand Down
5 changes: 4 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,13 @@
*.orig
*.pyc
.vscode

.claude
.codex
qa/pull-tester/tests_config.py
qa/pull-tester/tests_config.ini
qa/cache/*
qa/rpc-tests/cache/grpc_comparison/
qa/rpc-tests/cache/grpc_comparison_stage1/
src/*

poetry.lock
Expand Down
Loading
Loading