[BUG]: check_pending_account_resources must match pending transaction acceptance

[sgerbino]

Is there an existing issue for this?

• I have searched the existing issues

Current behavior

This RPC call is used at the beginning of submit_transaction and determines if the transaction would be accepted so it should match the implementation that actually adds the pending transaction.

Right now it is possible to have enough available RC left on the head node while not having enough RC on another fork which results in the rejection of the pending transaction.

koinos-mempool/libraries/mempool/mempool.cpp

Lines 251 to 267 in b0e3c2d

	bool mempool_impl::check_pending_account_resources(
	const account_type& payer,
	uint64_t max_payer_resources,
	uint64_t trx_resource_limit,
	std::optional< crypto::multihash > block_id ) const
	{
	auto lock = _db.get_shared_lock();
	auto node = relevant_node( block_id, lock );
	KOINOS_ASSERT(
	node,
	pending_transaction_unknown_block,
	"cannot check pending account resources from an unknown block"
	);
	return check_pending_account_resources_on_node( node, payer, max_payer_resources, trx_resource_limit );
	}

Expected behavior

If check_pending_account_resources returns true then the resulting pending transaction should be accepted.
If check_pending_account_resources returns false then the resulting pending transaction should fail.

Steps to reproduce

No response

Environment

- OS:

Anything else?

No response