merge-script 0fbb8043ce
Merge bitcoin/bitcoin#30252: test: Remove redundant verack check
0000276b31cea5e443a59d94a98c569293ada951 test: Remove redundant verack check (MarcoFalke)

Pull request description:

  Currently the sync in `connect_nodes` mentions the `version` and `verack` message types, but only checks the `verack`. Neither check is required, as the `pong` check implies both. In case of failure, the debug log will have to be consulted anyway, so the redundant check doesn't add value.

  Also clarify in the comments that the goal is to check the flag `fSuccessfullyConnected` indirectly.

ACKs for top commit:
  furszy:
    utACK 0000276b31ce
  brunoerg:
    ACK 0000276b31cea5e443a59d94a98c569293ada951
  tdb3:
    ACK 0000276b31cea5e443a59d94a98c569293ada951

Tree-SHA512: f9ddcb1436d2f70da462a8dd470ecfc90a534dd6507c23877ef7626e7c02326c077001a42ad0171a87fba5c5275d1970d8c5e5d82c56c8412de944856fdfd6db
2024-06-11 14:11:34 +01:00
..
2024-05-14 10:24:31 +02:00