bug fix : tx sync was incorrectly validating transactions

[tsachiherman]

Summary

The TestSyncFromClient was randomly failing. This is a real issue in production code, and the fix it pretty trivial.

Test Plan

Use existing tests to verify test.

[algonautshant]

Looks great.
Just one issue.

[codecov-commenter]

Codecov Report

Merging #3554 (49b6c39) into master (1dd40a8) will decrease coverage by 0.02%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master    #3554      +/-   ##
==========================================
- Coverage   47.59%   47.57%   -0.03%     
==========================================
  Files         370      370              
  Lines       60109    60114       +5     
==========================================
- Hits        28609    28598      -11     
- Misses      28189    28201      +12     
- Partials     3311     3315       +4     

Impacted Files	Coverage Δ
rpcs/txSyncer.go	74.54% <16.66%> (-5.46%)	⬇️
ledger/acctupdates.go	65.58% <0.00%> (-0.96%)	⬇️
data/abi/abi_type.go	87.67% <0.00%> (-0.95%)	⬇️
catchup/service.go	69.38% <0.00%> (-0.75%)	⬇️
network/wsNetwork.go	62.79% <0.00%> (-0.20%)	⬇️
network/wsPeer.go	68.05% <0.00%> (ø)
data/transactions/verify/txn.go	44.44% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1dd40a8...49b6c39. Read the comment docs.