Skip to content

tests: fix a race in TestP2PMetainfoExchange#6336

Merged
algorandskiy merged 1 commit intoalgorand:masterfrom
algorandskiy:pavel/p2p-test-metainfo-fix
May 21, 2025
Merged

tests: fix a race in TestP2PMetainfoExchange#6336
algorandskiy merged 1 commit intoalgorand:masterfrom
algorandskiy:pavel/p2p-test-metainfo-fix

Conversation

@algorandskiy
Copy link
Copy Markdown
Contributor

Summary

Fix a race between making connections and propagating peer info into peer store in TestP2PMetainfoExchange

Test Plan

This is a test fix

@codecov
Copy link
Copy Markdown

codecov bot commented May 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.87%. Comparing base (64ae8b0) to head (2f3e4a4).
Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6336   +/-   ##
=======================================
  Coverage   51.86%   51.87%           
=======================================
  Files         653      653           
  Lines       87547    87547           
=======================================
+ Hits        45408    45411    +3     
- Misses      39264    39265    +1     
+ Partials     2875     2871    -4     

☔ 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.

@algorandskiy algorandskiy merged commit b278dfd into algorand:master May 21, 2025
20 checks passed
cce pushed a commit to cce/go-algorand that referenced this pull request May 29, 2025
@algorandskiy algorandskiy deleted the pavel/p2p-test-metainfo-fix branch March 16, 2026 20:06
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