starknet_os_flow_tests: split test_new_class_flow experimental contract flow

[dorimedini-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[dorimedini-starkware]

• starknet_os_flow_tests: migrate test_initial_sierra_gas #9539 : 2 dependent PRs (#9544 , #9546 )
• starknet_os_flow_tests: migrate test_block_info #9529
• blockifier_test_utils: add block_info_test_contract contracts #9528
• starknet_os_flow_tests: migrate test_deploy_syscall #9527
• starknet_os_flow_tests: migrate test_deprecated_tx_info #9525
• blockifier_test_utils: add tx_info_writer cairo0 contract #9524
• starknet_os_flow_tests: rename test #9823
• starknet_os_flow_tests: split test_new_class_flow new account flow #9822
• starknet_os_flow_tests: split test_new_class_flow experimental contract flow #9821 👈 (View in Graphite)
• starknet_os_flow_tests: split test_new_class_flow exec-info txs #9820
• starknet_os_flow_tests: use nonce as deployment salt #9819
• starknet_os_flow_tests: implement get_builtin_usage #9818
• starknet_os_flow_tests: implement update_expected_storage_updates_for_block_hash_contract #9817
• starknet_os_flow_tests: migrate test_new_class_flow #9518
• starknet_os_flow_tests: add update_expected_storage util #9805
• blockifier_test_utils: add experimental contract and it's libfuncs list #9517
• blockifier_test_utils: put tx hash in signature for exec info tests #9516
• blockifier: extract get_expected_execution_info to function #9515
• blockifier: update test contract for test_new_contract_flow #9514
• main-v0.14.1-committer

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

[meship-starkware]

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware)

---

crates/starknet_os_flow_tests/src/tests.rs line 1218 at r1 (raw file):

        .assert_debug_eq(&poseidons);
    }
}

There is no need to check the Poseidon usage in all three functions. Keeping it in this one is enough

Code quote:

    let poseidons = test_output.get_builtin_usage(&BuiltinName::poseidon);
    if use_kzg_da {
        expect![[r#"
            58
        "#]]
        .assert_debug_eq(&poseidons);
    } else {
        expect![[r#"
            49
        "#]]
        .assert_debug_eq(&poseidons);
    }
}

[dorimedini-starkware]

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware and @meship-starkware)

---

crates/starknet_os_flow_tests/src/tests.rs line 1218 at r1 (raw file):

Previously, meship-starkware (Meshi Peled) wrote…

There is no need to check the Poseidon usage in all three functions. Keeping it in this one is enough

deleted other poseidon validations

[meship-starkware]

@meship-starkware reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dorimedini-starkware)

[graphite-app]

Merge activity

• Nov 13, 9:53 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.