test: Refactor tests using restart_node

[ccdle12]

fixes #19345

This PR replaces consecutive calls to stop_node() and start_node() with restart_node() where appropriate in the functional tests.

The commit messages are repetitive but focused on each file changed with the intention of squashing if applicable.

[maflcko]

There are less than 25 occurrences, so instead of 13 commits, I think this could be a single commit

[ccdle12]

There are less than 25 occurrences, so instead of 13 commits, I think this could be a single commit

Thanks, I'll squash them

[DrahtBot]

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

Conflicts

Reviewers, this pull request conflicts with the following ones:

• test: add v0.20.1, v0.21.0 and v22.0 to backwards compatibility test  #19013 (test: add v0.20.0 to backwards compatibility test by Sjors)
• wallet: ensure wallet files are not reused across chains #18554 (wallet: ensure wallet files are not reused across chains by mrwhythat)

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.

[laanwj]

Checked (with git grep -C3 stop_node\() that none of the remaining uses of stop_node(x) is trivially followed by start_node(x).
ACK 20b6e95