From bdcc138e8d84577fc99f9a9aef3484b66f98209a Mon Sep 17 00:00:00 2001
From: DEEPANSHU BANSAL <41580413+deepanshu-iiitu@users.noreply.github.com>
Date: Fri, 17 Nov 2023 21:38:52 +0530
Subject: [PATCH] feat(connector): [BANKOFAMERICA] PSYNC Bugfix (#2897)

---
 .../connector/bankofamerica/transformers.rs   | 21 +++++++++++++------
 1 file changed, 15 insertions(+), 6 deletions(-)

diff --git a/crates/router/src/connector/bankofamerica/transformers.rs b/crates/router/src/connector/bankofamerica/transformers.rs
index 20b2af48b168..a6fa8652b27d 100644
--- a/crates/router/src/connector/bankofamerica/transformers.rs
+++ b/crates/router/src/connector/bankofamerica/transformers.rs
@@ -273,7 +273,8 @@ pub enum BankofamericaPaymentStatus {
 impl ForeignFrom<(BankofamericaPaymentStatus, bool)> for enums::AttemptStatus {
     fn foreign_from((status, auto_capture): (BankofamericaPaymentStatus, bool)) -> Self {
         match status {
-            BankofamericaPaymentStatus::Authorized => {
+            BankofamericaPaymentStatus::Authorized
+            | BankofamericaPaymentStatus::AuthorizedPendingReview => {
                 if auto_capture {
                     // Because BankOfAmerica will return Payment Status as Authorized even in AutoCapture Payment
                     Self::Pending
@@ -281,7 +282,6 @@ impl ForeignFrom<(BankofamericaPaymentStatus, bool)> for enums::AttemptStatus {
                     Self::Authorized
                 }
             }
-            BankofamericaPaymentStatus::AuthorizedPendingReview => Self::Authorized,
             BankofamericaPaymentStatus::Succeeded | BankofamericaPaymentStatus::Transmitted => {
                 Self::Charged
             }
@@ -321,7 +321,7 @@ pub struct BankOfAmericaErrorInformationResponse {
 #[derive(Debug, Deserialize)]
 pub struct BankOfAmericaErrorInformation {
     reason: Option<String>,
-    message: String,
+    message: Option<String>,
 }
 
 impl<F>
@@ -369,7 +369,10 @@ impl<F>
             BankOfAmericaPaymentsResponse::ErrorInformation(error_response) => Ok(Self {
                 response: Err(types::ErrorResponse {
                     code: consts::NO_ERROR_CODE.to_string(),
-                    message: error_response.error_information.message,
+                    message: error_response
+                        .error_information
+                        .message
+                        .unwrap_or(consts::NO_ERROR_MESSAGE.to_string()),
                     reason: error_response.error_information.reason,
                     status_code: item.http_code,
                     attempt_status: None,
@@ -422,7 +425,10 @@ impl<F>
             BankOfAmericaPaymentsResponse::ErrorInformation(error_response) => Ok(Self {
                 response: Err(types::ErrorResponse {
                     code: consts::NO_ERROR_CODE.to_string(),
-                    message: error_response.error_information.message,
+                    message: error_response
+                        .error_information
+                        .message
+                        .unwrap_or(consts::NO_ERROR_MESSAGE.to_string()),
                     reason: error_response.error_information.reason,
                     status_code: item.http_code,
                     attempt_status: None,
@@ -475,7 +481,10 @@ impl<F>
             BankOfAmericaPaymentsResponse::ErrorInformation(error_response) => Ok(Self {
                 response: Err(types::ErrorResponse {
                     code: consts::NO_ERROR_CODE.to_string(),
-                    message: error_response.error_information.message,
+                    message: error_response
+                        .error_information
+                        .message
+                        .unwrap_or(consts::NO_ERROR_MESSAGE.to_string()),
                     reason: error_response.error_information.reason,
                     status_code: item.http_code,
                     attempt_status: None,