fix(provider): historical provider doesn't respect pinned block number (#360)

The fork version of the `HistoricalStateProvider` should falls back to requesting data from the forked network ONLY when the pinned block number (i.e., the block number the `HistoricalStateProvider` is created for - the `block_id` given to `<ForkedProvider as
StateFactoryProvider>::historical`) is less than or equal to the block number which katana is forked at.

Currently, `HistoricalStateProvider` would immediately falls back to the forked network. The request is then handled by the `Backend` and `Backend` is pinned to the forked block number and not the block number the state provider is for.

Author: kariy

crates/core/src/backend/storage.rs

@@ -141,7 +141,7 @@ impl Blockchain {
 
         // TODO: convert this to block number instead of BlockHashOrNumber so that it is easier to
         // check if the requested block is within the supported range or not.
-        let database = ForkedProvider::new(db, block_id, provider.clone());
+        let database = ForkedProvider::new(db, block_num, provider.clone());
 
         // initialize parent fork block
         //

crates/primitives/src/block.rs

@@ -52,7 +52,7 @@ pub enum ConfirmedBlockIdOrTag {
     L1Accepted,
 }
 
-#[derive(Debug, Copy, Clone, PartialEq, Eq)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq, Hash)]
 #[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
 pub enum BlockHashOrNumber {
     Hash(BlockHash),