catchpoints: add accountsRound check against R-320 horizon values to catchpointFileWriter

[cce]

Summary

Fix issue with onlineaccounts updateRound column for certain accounts in catchpoint files when updateRound < R-320. Renames MakeOnlineAccountsIter to MakeOrderedOnlineAccountsIter (without logic changes) so it is more clear the DB provides an ordering on (account, updround).

Test Plan

Update tests and add new ones.

[codecov]

Codecov Report

Attention: Patch coverage is 60.86957% with 9 lines in your changes missing coverage. Please review.

Project coverage is 51.78%. Comparing base (121df6f) to head (76676f0).
Report is 3 commits behind head on master.

Files with missing lines	Patch %	Lines
ledger/catchpointfilewriter.go	83.33%	1 Missing and 1 partial ⚠️
...edger/store/trackerdb/sqlitedriver/sqlitedriver.go	0.00%	2 Missing ⚠️
cmd/catchpointdump/file.go	0.00%	1 Missing ⚠️
ledger/catchupaccessor.go	0.00%	1 Missing ⚠️
ledger/store/trackerdb/dualdriver/dualdriver.go	0.00%	1 Missing ⚠️
ledger/store/trackerdb/generickv/reader.go	0.00%	1 Missing ⚠️
ledger/store/trackerdb/sqlitedriver/kvsIter.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6240      +/-   ##
==========================================
+ Coverage   51.76%   51.78%   +0.02%     
==========================================
  Files         644      644              
  Lines       86523    86533      +10     
==========================================
+ Hits        44791    44815      +24     
+ Misses      38864    38854      -10     
+ Partials     2868     2864       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jannotti]

Seems right to me! I don't feel strongly about the Address vs *Address thing. You can resolve if you prefer the pointer.