test: wait until new block is committed to BlockQueue

[ahangsu]

Summary

By #4349, we noticed a datarace in testMaxInnerTxnSingleAppCall. This PR waits the ledger to commit every new blocks in blockQueue to be committed in function endBlock.

[codecov]

Codecov Report

Merging #4381 (01f78c9) into master (60fb4f6) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4381   +/-   ##
=======================================
  Coverage   55.60%   55.61%           
=======================================
  Files         403      403           
  Lines       50801    50801           
=======================================
+ Hits        28248    28251    +3     
+ Misses      20159    20157    -2     
+ Partials     2394     2393    -1     

Impacted Files	Coverage Δ
agreement/cryptoVerifier.go	67.60% <0.00%> (-2.12%)	⬇️
agreement/proposalManager.go	96.07% <0.00%> (-1.97%)	⬇️
ledger/acctupdates.go	69.92% <0.00%> (-0.61%)	⬇️
ledger/acctonline.go	78.47% <0.00%> (-0.53%)	⬇️
catchup/service.go	69.38% <0.00%> (ø)
cmd/tealdbg/debugger.go	73.49% <0.00%> (+0.80%)	⬆️
util/db/dbutil.go	50.30% <0.00%> (+1.21%)	⬆️
network/wsPeer.go	67.67% <0.00%> (+2.19%)	⬆️
cmd/algoh/blockWatcher.go	80.95% <0.00%> (+3.17%)	⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

[michaeldiamant]

@ahangsu Thanks for the effort here - Hopefully, we've addressed a source of flakyness.

Additionally, I performed a cursory call hierarchy inspection for AddValidatedBlock. I found no production code paths liable for the same class of data race.