qa: Remove unused NodeConn members #11677

1. 
   MarcoFalke commented at 10:47 pm on November 13, 2017: member

   • ver_send and ver_recv were completely unused
   • rpc was only used once, in p2p-segwit. Imo better only pass it to the constructor in that single test
2. qa: Remove unused NodeConn members fafdad0d46
3. MarcoFalke added the label Tests on Nov 13, 2017
4. 
   promag commented at 2:18 pm on November 14, 2017: member

   Tested ACK fafdad0 (extended tests pass).
5. 
   jnewbery commented at 10:09 pm on November 14, 2017: member

   Tested ACK fafdad0d461330aee7a17db57a9d8f25906203cc.

   Thanks! This is part of #11518

6. 
   MarcoFalke commented at 10:45 pm on November 14, 2017: member

   Sorry, haven’t looked at #11518 in detail. Though, good to see we agree on this diff and did the same thing.
7. MarcoFalke merged this on Nov 14, 2017
8. MarcoFalke closed this on Nov 14, 2017

   ---

9. MarcoFalke referenced this in commit 3bdf242fc6 on Nov 14, 2017
10. MarcoFalke deleted the branch on Nov 14, 2017
11. PastaPastaPasta referenced this in commit 57c65f42d6 on Feb 13, 2020
12. PastaPastaPasta referenced this in commit 783b6831bd on Feb 13, 2020
13. PastaPastaPasta referenced this in commit 4ec4e349d2 on Feb 29, 2020
14. ckti referenced this in commit 4c5de2e01a on Mar 28, 2021
15. gades referenced this in commit 1f65e4f634 on Jun 30, 2021
16. MarcoFalke locked this on Sep 8, 2021

Contributors
MarcoFalke promag jnewbery

Labels
Tests