apollo_consensus_orchestrator: test parent proposal commitment binds parent fee_proposal

[sirandreww-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[sirandreww-starkware]

• apollo_consensus_orchestrator: test parent proposal commitment binds parent fee_proposal #14285 👈 (View in Graphite)
• main-v0.14.3

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cursor]

PR Summary

Low Risk
Test-only change with no runtime behavior modifications.

Overview
Adds an integration test in sequencer_consensus_context_test.rs that locks in how Cende blob preparation wires parent_proposal_commitment after consensus decides at height 1.

The test builds a proposal at HEIGHT_1, seeds fee_proposals_window at HEIGHT_0 with a parent fee distinct from the current block’s default, simulates decision_reached with batcher CentralObjects carrying a parent partial block hash, and asserts prepare_blob_for_next_height receives parent_proposal_commitment equal to proposal_commitment_from(PARTIAL_BLOCK_HASH, Some(PARENT_FEE_PROPOSAL)) (V0_14_3-style binding of parent fee_proposal_fri), alongside the expected current proposal_commitment.

No production logic changes—test-only coverage for parent fee binding in blob params.

^{Reviewed by Cursor Bugbot for commit e70ef63. Bugbot is set up for automated code reviews on this repo. Configure here.}

[ShahakShama]

@ShahakShama reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on sirandreww-starkware).

---

crates/apollo_consensus_orchestrator/src/sequencer_consensus_context_test.rs line 778 at r1 (raw file):

    deps.cende_ambassador.expect_prepare_blob_for_next_height().times(1).return_once(
        move |params| {
            assert_eq!(params.parent_proposal_commitment, Some(expected_parent_commitment));

Check also the proposal commitment

[sirandreww-starkware]

@sirandreww-starkware made 1 comment.
Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on ShahakShama).

---

crates/apollo_consensus_orchestrator/src/sequencer_consensus_context_test.rs line 778 at r1 (raw file):

Previously, ShahakShama wrote…

Check also the proposal commitment

Done.

[ShahakShama]

@ShahakShama reviewed 1 file and all commit messages, made 1 comment, and resolved 1 discussion.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on sirandreww-starkware).