test: remove qt byteswap compattests #21531

1. 
   fanquake commented at 4:40 am on March 26, 2021: member

   These were added as part of #9366 when with fixing issues with Protobuf.

   Now that we no-longer use Protobuf, there’s no reason to maintain a duplicate set of byteswap tests in the qt tests. Our other set of byteswap tests are here: https://github.com/bitcoin/bitcoin/blob/master/src/test/bswap_tests.cpp.

2. fanquake added the label Tests on Mar 26, 2021
3. 
   hebasto commented at 5:15 pm on March 26, 2021: member

   I think of this PR like a follow up after 8c6081a884cd0969160955ce8687d4d4ed074db3 from #17165, right?
4. test: remove qt byteswap compattests

   These were added as part of #9366 to fix issues with Protobuf.
   
   Now that we no-longer use Protobuf, there's no reason to maintain a
   duplicate set of byteswap tests for qt.

   9ac86bcc0d
5. fanquake force-pushed on Mar 29, 2021
6. 
   fanquake commented at 3:13 am on March 29, 2021: member

   I think of this PR like a follow up after 8c6081a from #17165, right?

   Sure

7. 
   laanwj commented at 9:42 am on March 29, 2021: member

   +0 -56 perfect score

   Code review ACK 9ac86bcc0d72cdcecdbf8884a557095bbc3cdc11

8. MarcoFalke merged this on Mar 29, 2021
9. MarcoFalke closed this on Mar 29, 2021

   ---

10. fanquake deleted the branch on Mar 29, 2021
11. sidhujag referenced this in commit 20a1c64bc6 on Mar 29, 2021
12. DrahtBot locked this on Aug 16, 2022

Contributors
fanquake hebasto laanwj

Labels
Tests