Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(collator): ignore duplicated anchors#1087

Open
Mododo wants to merge 1 commit into
masterfrom
fix/collator/ignore-dup-anchors
Open

fix(collator): ignore duplicated anchors#1087
Mododo wants to merge 1 commit into
masterfrom
fix/collator/ignore-dup-anchors

Conversation

@Mododo
Copy link
Copy Markdown
Member

@Mododo Mododo commented May 22, 2026

may occur on engine restart

Pull Request Checklist

NODE CONFIGURATION MODEL CHANGES

None

BLOCKCHAIN CONFIGURATION MODEL CHANGES

None


COMPATIBILITY

Full

SPECIAL DEPLOYMENT ACTIONS

Not Required


PERFORMANCE IMPACT

No impact expected


TESTS

Unit Tests

Covered by behavioral cache test (updated in PR)

Network Tests

No coverage

Manual Tests

transfers-30k with node sync


Notes/Additional Comments:

may occur on engine restart
@Mododo Mododo added this to the 020 Consensus milestone May 22, 2026
@Mododo Mododo requested review from 0xdeafbeef and SmaGMan May 22, 2026 15:32
@Mododo Mododo self-assigned this May 22, 2026
@Mododo Mododo added the C-collator Collator module label May 22, 2026
@github-actions
Copy link
Copy Markdown

🧪 Network Tests

To run network tests for this PR, use:

gh workflow run network-tests.yml -f pr_number=1087

Available test options:

  • Run all tests: gh workflow run network-tests.yml -f pr_number=1087
  • Run specific test: gh workflow run network-tests.yml -f pr_number=1087 -f test_selection=ping-pong

Test types: destroyable, ping-pong, one-to-many-internal-messages, fq-deploy, nft-index, persistent-sync

Results will be posted as workflow runs in the Actions tab.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 22, 2026

Codecov Report

❌ Patch coverage is 91.66667% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.54%. Comparing base (b0a5a87) to head (dc27f80).

Files with missing lines Patch % Lines
collator/src/mempool/impls/common/cache.rs 94.28% 1 Missing and 1 partial ⚠️
collator/src/mempool/impls/std_impl/mod.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1087      +/-   ##
==========================================
- Coverage   57.57%   57.54%   -0.03%     
==========================================
  Files         464      464              
  Lines       77109    77132      +23     
  Branches    77109    77132      +23     
==========================================
- Hits        44395    44388       -7     
- Misses      30701    30728      +27     
- Partials     2013     2016       +3     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-collator Collator module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant