tests: Avoid copies of CTransaction #12949

1. 
   MarcoFalke commented at 5:54 pm on April 11, 2018: member

   Avoid the copy (or move) constructor of CTransaction in test code, whereever a simple reference can be used instead.
2. MarcoFalke force-pushed on Apr 11, 2018
3. MarcoFalke force-pushed on Apr 11, 2018
4. tests: Avoid copies of CTransaction fae58eca93
5. MarcoFalke force-pushed on Apr 11, 2018
6. 
   MarcoFalke commented at 7:02 pm on April 11, 2018: member

   Force pushed fixes to the previously wrong comments about std::shared_ptr::use_count in blockencodings_tests.cpp
7. MarcoFalke added the label Tests on Apr 11, 2018
8. MarcoFalke added the label Refactoring on Apr 11, 2018
9. 
   promag commented at 8:22 pm on April 11, 2018: member

   utACK fae58ec. The fewer copies the better.
10. 
    ryanofsky commented at 5:48 pm on April 13, 2018: member

    utACK fae58eca934b5c7165b589c3bec1751d1b432b48
11. MarcoFalke merged this on Apr 17, 2018
12. MarcoFalke closed this on Apr 17, 2018

    ---

13. MarcoFalke referenced this in commit 6b46288a08 on Apr 17, 2018
14. MarcoFalke deleted the branch on Apr 17, 2018
15. PastaPastaPasta referenced this in commit a798e1e7e5 on Apr 13, 2021
16. PastaPastaPasta referenced this in commit fa864cc3b3 on Apr 15, 2021
17. PastaPastaPasta referenced this in commit ad0349ff96 on Apr 17, 2021
18. PastaPastaPasta referenced this in commit ab1b3759f4 on Apr 18, 2021
19. kittywhiskers referenced this in commit e4429abbf8 on Apr 23, 2021
20. MarcoFalke locked this on Sep 8, 2021

Contributors
MarcoFalke promag ryanofsky

Labels
Refactoring Tests