fix: hive

[greged93]

Time spent on this PR: 1 day

On track for: #1310

Pull Request type

Please check the type of change your PR introduces:

• Bugfix
• Feature
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no API changes)
• Build-related changes
• Documentation content changes
• Testing

What is the new behavior?

Fixes the hive testing using execute from outside.

Does this introduce a breaking change?

• Yes
• No

[tcoratger]

Here, shouldn't it be:

let chain_id: u64 = (Felt::from(u32::MAX).to_bigint() & chain_id.to_bigint()).try_into()?;

to be consistent with what's here

kakarot-rpc/src/client/mod.rs

Lines 51 to 54 in 9ea6096

	let chain = (starknet_provider.chain_id().await.map_err(KakarotError::from)?.to_bigint()
	& Felt::from(u32::MAX).to_bigint())
	.to_u64()
	.unwrap();

?

[greged93]

you're right, as discussed this is an error in the code and needs to be fixed. We should split into one ETH_CHAIN_ID which is taken modulo u32::MAX and one STARKNET_CHAIN_ID which is just the "real" starknet chain id

[ClementWalter]

2**53 and not 2**32 though