zebra icon indicating copy to clipboard operation
zebra copied to clipboard

Add GetMempoolTx gRPC test

Open ftm1000 opened this issue 2 years ago • 2 comments

Motivation

In https://github.com/ZcashFoundation/zebra/pull/4253 we added tests for grpc wallet methods, however we left a few TODOs in the main file (zebrad/tests/common/lightwalletd/wallet_grpc_test.rs) for missing tests.

Tasks

Add a test for:

  • GetMempoolTx
  • GetMempoolStream

The send transaction test populates the mempool, so we can add these extra gRPC calls to the end of that test.

Draft Implementation

PR #4537 has a draft implementation.

ftm1000 avatar May 06 '22 18:05 ftm1000

Hey team! Please add your planning poker estimate with ZenHub @conradoplg @dconnolly @oxarbitrage @teor2345 @upbqdn

ftm1000 avatar May 06 '22 18:05 ftm1000

Split from https://github.com/ZcashFoundation/zebra/issues/4278

ftm1000 avatar May 06 '22 18:05 ftm1000