Fix WalletClient::get_transfers with Out=true in Category Selector (#127

)

* Update models.rs

Convert GotTransfer::suggested_confirmations_threshold to Option<u64> from u64 to enable Wallet.gettransfers(...) with Out: true in the Category Selector

* Update all_clients_interaction.rs

update test

---------

Co-authored-by: h4sh3d <h4sh3d@protonmail.com>

src/models.rs

@@ -516,7 +516,7 @@ pub struct GotTransfer {
     /// JSON object containing the major & minor subaddress index.
     pub subaddr_index: subaddress::Index,
     /// Estimation of the confirmations needed for the transaction to be included in a block.
-    pub suggested_confirmations_threshold: u64,
+    pub suggested_confirmations_threshold: Option<u64>,
     /// POSIX timestamp for when this transfer was first confirmed in a block (or timestamp submission if not mined yet).
     #[serde(with = "chrono::serde::ts_seconds")]
     pub timestamp: DateTime<Utc>,

tests/clients_tests/all_clients_interaction.rs

@@ -445,7 +445,7 @@ pub async fn run() {
         note: "".to_string(),
         payment_id: HashString(PaymentId::zero()),
         subaddr_index: Index { major: 0, minor: 0 },
-        suggested_confirmations_threshold: 1,
+        suggested_confirmations_threshold: Some(1),
         // this is any date, since it will not be tested against anything
         timestamp: DateTime::from_naive_utc_and_offset(
             NaiveDateTime::from_timestamp_opt(0, 0).unwrap(),