test(sequencing): add non-default values to central objects test

[Yael-Starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[Yael-Starkware]

• test(starknet_batcher): verify execution info order matches the order of transactions in a block #3616
• feat(starknet_consensus_orchestrator): add compressed_state_diff to cende objects #3557
• feat(starknet_consensus_orchestrator): pass central objects from batcher to cende #3384
• test(starknet_consensus_orchestrator): central transaction execution info object test #3337
• feat(sequencing): add execution info central object #3238
• chore(sequencing): reorder central transactions fields #3230
• test(sequencing): add non-default values to central objects test #3198 👈 (View in Graphite)
• test(sequencing): add non-default values to central objects test #3197
• main

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

[DvirYo-starkware]

Reviewed 3 of 4 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @dafnamatsry and @Yael-Starkware)

---

a discussion (no related file):
Same as the previous PR, can you also do a test for the default variants?

---

crates/sequencing/papyrus_consensus_orchestrator/src/cende/central_objects.rs line 111 at r2 (raw file):

            state_diff.deprecated_declared_classes.is_empty(),
            "Deprecated classes are not supported"
        );

Consider adding a new line here

---

crates/sequencing/papyrus_consensus_orchestrator/src/cende/central_objects.rs line 113 at r2 (raw file):

        );
        let mut address_to_class_hash = state_diff.deployed_contracts;
        address_to_class_hash.extend(state_diff.replaced_classes);

This is a bug fix?

Code quote:

        let mut address_to_class_hash = state_diff.deployed_contracts;
        address_to_class_hash.extend(state_diff.replaced_classes);

[Yael-Starkware]

Reviewable status: 2 of 4 files reviewed, 2 unresolved discussions (waiting on @dafnamatsry and @DvirYo-starkware)

---

a discussion (no related file):

Previously, DvirYo-starkware wrote…

Same as the previous PR, can you also do a test for the default variants?

same, please add a task to monday

---

crates/sequencing/papyrus_consensus_orchestrator/src/cende/central_objects.rs line 111 at r2 (raw file):

Previously, DvirYo-starkware wrote…

Consider adding a new line here

done.

---

crates/sequencing/papyrus_consensus_orchestrator/src/cende/central_objects.rs line 113 at r2 (raw file):

Previously, DvirYo-starkware wrote…

This is a bug fix?

yes

[DvirYo-starkware]

Reviewed 2 of 2 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dafnamatsry)