Merge bitcoin#18496: test: remove redundant sync_with_ping after add_p2p_connection

4670006 test: remove redundant sync_with_ping after add_p2p_connection (Jon Atack)

Pull request description:

  Now that bitcoin#18247 is merged, these calls are redundant.

ACKs for top commit:
  vasild:
    utACK 4670006

Tree-SHA512: bdbfe8bcf9dbdde0a8115e3a62bfe359910798d7a3010d920ffca07049cb5f97bf8fb9b6f70079b0607105192b61a6d665774e59a2b678597b47ad6237595ad5

Author: MarcoFalke

test/functional/p2p_sendheaders.py

@@ -243,10 +243,6 @@ def run_test(self):
         # will occur outside of direct fetching
         test_node = self.nodes[0].add_p2p_connection(BaseNode(), services=NODE_WITNESS)
 
-        # Ensure verack's have been processed by our peer
-        inv_node.sync_with_ping()
-        test_node.sync_with_ping()
-
         self.test_null_locators(test_node, inv_node)
         self.test_nonnull_locators(test_node, inv_node)