v0_8 get_storage_proof returning proofs out of order

[GMKrieger]

The methods get_class_proofs, get_contract_proofs and get_contract_storage_proofs methods inside get_storage_proof are returning the proofs out of order and flattened into a single Vec. This bug occurs on this snippets of code:

pathfinder/crates/rpc/src/method/get_storage_proof.rs

Lines 372 to 383 in a054a03

	let nodes: Vec<NodeHashToNodeMapping> =
	ClassCommitmentTree::get_proofs(tx, block_number, class_hashes, class_root_idx)?
	.into_iter()
	.flatten()
	.map(|(node, node_hash)| NodeHashToNodeMapping {
	node_hash,
	node: ProofNode(node),
	})
	.collect::<HashSet<_>>()
	.into_iter()
	.collect();
	let classes_proof = NodeHashToNodeMappings(nodes);

pathfinder/crates/rpc/src/method/get_storage_proof.rs

Lines 417 to 427 in 2097a13

	let nodes =
	StorageCommitmentTree::get_proofs(tx, block_number, contract_addresses, storage_root_idx)?
	.into_iter()
	.flatten()
	.map(|(node, node_hash)| NodeHashToNodeMapping {
	node_hash,
	node: ProofNode(node),
	})
	.collect::<HashSet<_>>()
	.into_iter()
	.collect();

pathfinder/crates/rpc/src/method/get_storage_proof.rs

Lines 469 to 486 in 2097a13

	let nodes: Vec<NodeHashToNodeMapping> = ContractsStorageTree::get_proofs(
	&tx,
	csk.contract_address,
	block_number,
	&csk.storage_keys,
	root,
	)?
	.into_iter()
	.flatten()
	.map(|(node, node_hash)| NodeHashToNodeMapping {
	node_hash,
	node: ProofNode(node),
	})
	.collect::<HashSet<_>>()
	.into_iter()
	.collect();
	proofs.push(NodeHashToNodeMappings(nodes));