Skip to content

2/3 Add manual gRPC comparison test#165

Open
pacu wants to merge 1 commit into
grpc-proto-bindingsfrom
grpc-comparison-manual
Open

2/3 Add manual gRPC comparison test#165
pacu wants to merge 1 commit into
grpc-proto-bindingsfrom
grpc-comparison-manual

Conversation

@pacu

@pacu pacu commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator

Stack order:

  1. 1/3 Add lightwallet protocol Python bindings #164: add lightwallet-protocol Python bindings.
  2. This PR: add the manual gRPC comparison test and framework support.
  3. grpc-comparison-ci -> grpc-comparison-manual: run the gRPC comparison test in CI.

Goal:

Add a manually runnable gRPC parity test that compares zainod and lightwalletd responses against the same zebrad regtest chain, without enabling it in CI yet.

What is included:

  • qa/rpc-tests/grpc_comparison.py parity test.
  • qa/zcash/grpc_comparison_tests.py convenience wrapper.
  • Framework support for launching lightwalletd and standalone zcashd fixture builders.
  • Cached gRPC comparison fixtures under qa/rpc-tests/cache/grpc_comparison*.
  • Documentation for running and maintaining the test.

Review focus:

This PR is where the handwritten test behavior lives. CI wiring is intentionally deferred to PR 3 so this diff can be reviewed on its own.

Local checks run:

  • python -m py_compile qa/rpc-tests/grpc_comparison.py qa/rpc-tests/test_framework/authproxy.py qa/rpc-tests/test_framework/config.py qa/rpc-tests/test_framework/test_framework.py qa/rpc-tests/test_framework/util.py qa/zcash/grpc_comparison_tests.py
  • timeout 900s uv run ./qa/zcash/grpc_comparison_tests.py --nocleanup

Note:

The wrapper in this PR invokes qa/rpc-tests/grpc_comparison.py directly so the manual test can run before PR 3 registers it in qa/pull-tester/rpc-tests.py.

Co-Authored-By: OpenAI Codex codex@openai.com

Add framework support for starting lightwalletd and standalone zcashd fixtures, then add a manual gRPC parity test comparing zainod and lightwalletd responses.

Include cached regtest fixtures and documentation so the test can run locally without rebuilding the fixture chain every time.

Co-Authored-By: OpenAI Codex <codex@openai.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant