Return OutOfRange instead of StarknetApiError whenever a method can only return OutOfRange

[ShahakShama]

This change is 

[ShahakShama]

+reviewer:@giladchase

Reviewable status: 0 of 7 files reviewed, all discussions resolved (waiting on @giladchase, @nagmo-starkware, and @yair-starkware)

[giladchase]

Reviewed 7 of 7 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @nagmo-starkware, @ShahakShama, and @yair-starkware)

---

src/lib.rs line 41 at r1 (raw file):

pub struct OutOfRangeError {
    string: String,
}

"out of error" string is redundant, it appears in the variant.
string field also not necessary I think, it is clear what the string means.

Suggestion:

#[error("{0}")]
pub struct OutOfRangeError(String)

---

src/lib.rs line 47 at r1 (raw file):

        Self::OutOfRange(error)
    }
}

i think that #[from] generates the code removed

Suggestion:

    #[error(transparent)]
    OutOfRange(#[from] OutOfRangeError),
    /// Error when serializing into number.
    #[error(transparent)]
    ParseIntError(#[from] ParseIntError),
    /// Missing resource type / duplicated resource type.
    #[error("Missing resource type / duplicated resource type; got {0}.")]
    InvalidResourceMappingInitializer(String),
}

#[derive(thiserror::Error, Clone, Debug)]
#[error("Out of range {string}.")]
pub struct OutOfRangeError {
    string: String,
}