fix(test): Fixes bugs in the lightwalletd integration tests

[arya2]

Motivation

This PR fixes a bug in the sending_transactions_using_lightwalletd test where transactions from multiple future blocks are being sent to the mempool without committing prior blocks to the chain.

It also fixes a bug in the lightwalletd_wallet_grpc_tests where the test expects the ZF funding stream address balance to always be greater than zero.

Solution

• Submits blocks to the state after sending their transactions to the mempool in the sending_transactions_using_lightwalletd test, and
• Only check that active funding stream addresses have a balance that's greater than zero in the lightwalletd_wallet_grpc_tests test.

Tests

The test should pass in CI.

PR Author's Checklist

• The PR name will make sense to users.
• The PR provides a CHANGELOG summary.
• The solution is tested.
• The documentation is up to date.
• The PR has a priority label.

PR Reviewer's Checklist

• The PR Author's checklist is complete.
• The PR resolves the issue.

[conradoplg]

@Mergifyio update

[mergify]

update

✅ Branch has been successfully updated

[conradoplg]

Sorry, I missed this PR before. It looks good, I just have some questions about parts of it.

[conradoplg]

FYI I got this error on another PR:

  left: SendResponse { error_code: 0, error_message: "failed to validate tx: transaction::Hash(\"private\"), error: transaction did not pass consensus validation: could not validate nullifiers and anchors on best chain: immature transparent coinbase spend: attempt to spend OutPoint { hash: transaction::Hash(\"47bc05a53a5f270604f9d6a52bdc48cb9a5c7a1efc4ff6eb00f8842f287a3f2c\"), index: 0 } at Height(2739104), but spends are invalid before Height(2739107), which is 100 blocks after it was created at Height(2739007)" }
 right: SendResponse { error_code: 0, error_message: "\"0f85fada75a7a59a02e15363f8bd62150c9148d133b50d570952bb89bc713a1f\"" }

which does seems like something that might be fixed by this PR

[conradoplg]

Looks good, just needs solving conflicts

[arya2]

I still need to fix the new test failure before this merges (error trying to connect when calling Zebra's RPC server) first, but I'm happy for this to merge without a fix for the invalid script issue.

[arya2]

The test is now passing locally.

[conradoplg]

Looks good! I left a question just to be sure about the fix of the last issue

[mergify]

This pull request has been removed from the queue for the following reason: pull request dequeued.

Pull request #9052 has been dequeued. The pull request rule doesn't match anymore.

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.

If you want to requeue this pull request, you need to post a comment with the text: @mergifyio requeue

[conradoplg]

(It was dequeued because I noticed that issue I commented just after I approved it)