Skip to content

Comments

Add e2e test for joining node with corrupted (truncated) ledger file#7666

Open
Copilot wants to merge 6 commits intomainfrom
copilot/add-testcase-for-issue-6612
Open

Add e2e test for joining node with corrupted (truncated) ledger file#7666
Copilot wants to merge 6 commits intomainfrom
copilot/add-testcase-for-issue-6612

Conversation

Copy link
Contributor

Copilot AI commented Feb 13, 2026

  • Understand the issue (cchost crashes in case a corrupt ledger file is found on a node that is joining the network #6612): node crashes when joining with corrupted ledger files
  • Explore existing test infrastructure (reconfiguration.py, recovery.py corruption patterns)
  • Add test_add_node_with_corrupted_ledger test to tests/reconfiguration.py
  • Add the test to the reconfiguration test suite in tests/suite/test_suite.py
  • Replace silent skips with assertions when unable to find chunks to corrupt
  • Add the test to run_all in reconfiguration.py alongside other test_add_node_* variants
  • Run code review and CodeQL security check (no alerts)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
Copilot AI changed the title [WIP] Add testcase for issue flagged in #6612 Add e2e test for joining node with corrupted (truncated) ledger file Feb 13, 2026
Copilot AI requested a review from achamayou February 13, 2026 09:31
…itions

Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
@eddyashton eddyashton marked this pull request as ready for review February 17, 2026 14:21
@eddyashton eddyashton requested a review from a team as a code owner February 17, 2026 14:21
Copilot AI review requested due to automatic review settings February 17, 2026 14:21
@eddyashton
Copy link
Member

Note that this is only a repro (not a fix) for #6612.

Co-authored-by: eddyashton <6000239+eddyashton@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants