Skip to content

statement-store: add channel replacement logic tests#11617

Merged
DenzelPenzel merged 2 commits intomasterfrom
denzelpenzel/statement-channel-replacement
Apr 6, 2026
Merged

statement-store: add channel replacement logic tests#11617
DenzelPenzel merged 2 commits intomasterfrom
denzelpenzel/statement-channel-replacement

Conversation

@DenzelPenzel
Copy link
Copy Markdown
Contributor

@DenzelPenzel DenzelPenzel commented Apr 1, 2026

Description

Implement unit tests for "Channel replacement: verify only higher-priority statements replace existing entries, corner cases of replacement logic." #11534

Summary

  • channel_replacement_only_higher_priority_succeeds -> verifies lower/equal priority rejected with ChannelPriorityTooLow, higher priority replaces, one-per-channel invariant preserved

  • channel_replacement_with_size_increase_evicts_others -> verifies that replacing a channel message with a larger one triggers additional eviction of lowest-priority non-channel statements to satisfy size constraints

@DenzelPenzel DenzelPenzel added T0-node This PR/Issue is related to the topic “node”. T10-tests This PR/Issue is related to tests. and removed T0-node This PR/Issue is related to the topic “node”. labels Apr 1, 2026
@DenzelPenzel
Copy link
Copy Markdown
Contributor Author

/cmd prdoc --audience node_dev --bump patch

@DenzelPenzel DenzelPenzel enabled auto-merge April 6, 2026 19:51
@DenzelPenzel DenzelPenzel added this pull request to the merge queue Apr 6, 2026
Merged via the queue into master with commit ae772b4 Apr 6, 2026
315 of 332 checks passed
@DenzelPenzel DenzelPenzel deleted the denzelpenzel/statement-channel-replacement branch April 6, 2026 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T10-tests This PR/Issue is related to tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants