Merge pull request #2 from algorandskiy/pavel/cce/lookup-all-resource…

…s-fix Make tests working
algorandskiy · Jan 4, 2022 · bfe3a38 · bfe3a38
2 parents 8f588bb + 6b04344
commit bfe3a38
Show file tree

Hide file tree

Showing 6 changed files with 202 additions and 141 deletions.
diff --git a/cmd/catchpointdump/file.go b/cmd/catchpointdump/file.go
@@ -295,17 +295,22 @@ func printAccountsDatabase(databaseName string, fileHeader ledger.CatchpointFile
 			}
 			err = rows.Err()
 		} else {
+			acctCount := 0
 			acctCb := func(addr basics.Address, data basics.AccountData) {
 				err = printer(addr, data, progress)
 				if err != nil {
 					return
 				}
 				progress++
+				acctCount++
 			}
 			_, err = ledger.LoadAllFullAccounts(context.Background(), tx, balancesTable, resourcesTable, acctCb)
 			if err != nil {
 				return
 			}
+			if acctCount != int(rowsCount) {
+				return fmt.Errorf("expected %d accounts but got only %d", rowsCount, acctCount)
+			}
 		}
 
 		// increase the deadline warning to disable the warning message.