Skip to content

Conversation

@algorandskiy
Copy link
Contributor

Summary

testCatchpointAfterStakeLookupTxns uses double ledger after adding lots of blocks, and then checks accounts are persisted on the same round. But sometimes it checks too early so validator has not persisted the latest block yet so flushing with testCatchpointFlushRound does flush but sometimes on a previous round.
Fixed by ensuring both ledgers have the latest block persisted.

Test Plan

This is a unit test fix

@codecov
Copy link

codecov bot commented Jun 11, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.48%. Comparing base (6147ebf) to head (79b44a4).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6363      +/-   ##
==========================================
- Coverage   50.66%   50.48%   -0.18%     
==========================================
  Files         653      653              
  Lines      110377   110377              
==========================================
- Hits        55918    55729     -189     
- Misses      51592    51788     +196     
+ Partials     2867     2860       -7     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jannotti jannotti merged commit ac31f0b into algorand:master Jun 11, 2025
48 of 49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants