test: Move test_chain_listunspent wallet check from mempool_packages to wallet_basic #27735

This fixes a bug.

On master:

$ ./test/functional/mempool_packages.py  --legacy-wallet 
  File "./test/functional/mempool_packages.py", line 52, in run_test
    self.nodes[0].importaddress(self.wallet.get_address())
test_framework.authproxy.JSONRPCException: Bech32m addresses cannot be imported into legacy wallets (-5)

On this pull, all tests pass.

<!--e57a25ab6845829454e8d69fc972939a-->

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

<!--021abf342d371248e50ceaed478a90ca-->

Reviews

See the guideline for information on the review process.

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

<!--174a7506f384e20aa4161008e828411d-->

Conflicts

Reviewers, this pull request conflicts with the following ones:

• #20892 (tests: Run both descriptor and legacy tests within a single test invocation by achow101)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

(An alternative to test this would be to compile with bdb only)

.

Is there a reason that this test needs the wallet in the first place?

Yeah, the test doesn't need the wallet, but it is providing mempool-package coverage for the listunspent wallet RPC. So maybe an alternative would be to move it out into a new test, idk?

cc @glozow

Yeah, the test doesn't need the wallet, but it is providing mempool-package coverage for the listunspent wallet RPC. So maybe an alternative would be to move it out into a new test, idk?

Would be in favor of moving the listunspent coverage to a new/existing wallet_ functional test instead.

Thx, done

ACK ffffe622e9cbf926326135bb23958380dcf09df1, thanks for changing! Nice to remove wallet from another non-wallet test.