Update TODO comment where cancelRefund completion is not invoked after failure.

Author: jaclync

Hardware/Hardware/CardReader/StripeCardReader/StripeCardReaderService.swift

@@ -563,7 +563,7 @@ extension StripeCardReaderService {
                 }
                 promise(.success(()))
             })
-            //TODO: handle timeout?
+            //TODO: 5983 - handle timeout when called from retry after refund failure
         }.eraseToAnyPublisher()
     }
 }

WooCommerce/Classes/ViewModels/CardPresentPayments/CardPresentRefundOrchestrator.swift

@@ -59,7 +59,7 @@ final class CardPresentRefundOrchestrator {
     /// Cancels the current refund.
     /// - Parameter onCompletion: called when the cancellation completes.
     func cancelRefund(onCompletion: @escaping (Result<Void, Error>) -> Void) {
-        let action = CardPresentPaymentAction.cancelRefund() { [weak self] result in
+        let action = CardPresentPaymentAction.cancelRefund { [weak self] result in
             self?.allowPassPresentation()
             onCompletion(result)
         }

WooCommerce/Classes/ViewRelated/Orders/Refund/RefundSubmissionUseCase.swift

@@ -252,7 +252,7 @@ private extension RefundSubmissionUseCase {
     /// Logs the failure reason, cancels the current refund, and offers retry if possible.
     func handleRefundFailureAndRetryRefund(_ error: Error, refundAmount: Decimal, charge: WCPayCharge, onCompletion: @escaping (Result<Void, Error>) -> ()) {
         // TODO: 5984 - tracks in-person refund error
-        DDLogError("Failed to collect payment: \(error.localizedDescription)")
+        DDLogError("Failed to refund: \(error.localizedDescription)")
         // Informs about the error.
         alerts?.error(error: error) { [weak self] in
             // Cancels current payment.