feat(starknet_gateway): use sync state reader instead of rpc

[noamsp-starkware]

feat(starknet_gateway): use sync state reader instead of rpc

feat(starknet_sequencer_node): remove rpc state reader

Due to this change we also fix the integration tests accordingly

fix(starknet_mempool_p2p): fix directory erasing after setup

Tests were running without the temporary directory handles, causing
these directory to be erased when setup would finish running.

This commit Adds temp dir handles to the setup output for the test to
hold to prevent the erasure.

fix(starknet_gateway): sync state reader fails end to end flow

End to End flow currently provides a zero value to gas_prices.
This commit adds a default assignment to gas price if a zero value was
provided, and add a TODO to remove the default once the test is fixed

[reviewable-StarkWare]

This change is 

[noamsp-starkware]

• feat(starknet_gateway): use sync state reader instead of rpc #2990 👈 (View in Graphite)
• feat(starknet_state_sync): implement state sync get latest block number #2944
• feat(starknet_gateway): implement sync state reader get block info #2942 : 1 other dependent PR (#2903 )
• fix(starknet_state_sync): verify contract deployed at get storage/nonce #2963 : 1 other dependent PR (#2886 )
• fix(starknet_state_sync): change marker verify from casm to state #2979
• main

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

[ShahakShama]

Reviewed 1 of 3 files at r1, 2 of 9 files at r4, 5 of 7 files at r5, 1 of 1 files at r6, 1 of 1 files at r7, 1 of 8 files at r8, 6 of 7 files at r9, 1 of 1 files at r10, 1 of 1 files at r11, 1 of 1 files at r12, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @alonh5 and @noamsp-starkware)

---

crates/starknet_gateway/src/sync_state_reader.rs line 45 at r12 (raw file):

            block_timestamp: block_header.timestamp,
            sequencer_address: block_header.sequencer.0,
            // TODO(noamsp): Remove unwrap_or_default after consensus gas price fix is merged.

Fix this in consensus instead. Change the default gas price to be one instead of zero and add a TODO on @asmaastarkware to fix this properly

[noamsp-starkware]

Reviewable status: 10 of 12 files reviewed, 1 unresolved discussion (waiting on @alonh5, @asmaastarkware, and @ShahakShama)

---

crates/starknet_gateway/src/sync_state_reader.rs line 45 at r12 (raw file):

Previously, ShahakShama wrote…

Fix this in consensus instead. Change the default gas price to be one instead of zero and add a TODO on @asmaastarkware to fix this properly

Done.

[ShahakShama]

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

[alonh5]

Reviewed 1 of 3 files at r1, 1 of 9 files at r4, 1 of 8 files at r8.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @noamsp-starkware)

[ShahakShama]

Reviewed 7 of 10 files at r14, 6 of 7 files at r15, 1 of 1 files at r16, 1 of 1 files at r17, 1 of 1 files at r18, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @noamsp-starkware)