remove explicit descriptors=true parameter for createwallet RPCs in functional tests #24550

1. 
   theStack commented at 2:52 pm on March 13, 2022: member

   Since the default wallet type was changed to descriptor wallets in #23002, there is no need to explicitly pass the descriptors=true parameter to createwallet RPCs anymore. All of those instances in functional tests should be identified and simplified by omitting the parameter (there is already a PR doing the same for documentation, see #24519).

   Useful skills:

   • Basic Python3 knowledge

   Want to work on this issue?

   For guidance on contributing, please read CONTRIBUTING.md before opening your pull request.

2. 
   theStack commented at 2:56 pm on March 13, 2022: member

   (Not sure if I have to explicitly mention this here or maintainers see it by the template I selected: this should get the “good first issue” tag).
3. fanquake added the label good first issue on Mar 13, 2022
4. 
   prusnak commented at 6:53 pm on March 13, 2022: contributor

   Implemented in https://github.com/bitcoin/bitcoin/pull/24551
5. 
   theStack commented at 1:51 pm on March 14, 2022: member

   My assumption that those parameters can be removed also in the test framework was wrong (see #24551 (comment) and #24551 (comment)), i.e. will close this now. Sorry for the noise!
6. theStack closed this on Mar 14, 2022

   ---

7. fanquake removed the label good first issue on Mar 14, 2022
8. DrahtBot locked this on Mar 14, 2023

Contributors
theStack prusnak