endpoint patch - update transaction

Author: danielen-meli

.pre-commit-config.yaml

@@ -1,7 +1,7 @@
 repos:
   # Websec hook is MANDATORY, DO NOT comment it.
   - repo: https://github.com/melisource/fury_websec-git-hooks
-    rev: v1.1.0
+    rev: v2.0.0
     hooks:
       - id: pre_commit_hook
         stages: [commit]
@@ -10,8 +10,9 @@ repos:
 
   # Datasec hook is MANDATORY, DO NOT comment it.
   - repo: https://github.com/melisource/fury_datasec-git-hooks
-    rev: 1.0.3
+    rev: 1.2.1
     hooks:
       - id: pre_commit_hook
         stages: [commit]
-        verbose: true
+      - id: post_commit_hook
+        stages: [post-commit]

src/main/java/com/mercadopago/client/order/OrderClient.java

@@ -201,4 +201,56 @@ public OrderTransaction createTransaction(String orderId, OrderTransactionReques
             throws MPException, MPApiException {
         return this.createTransaction(orderId, request, null);
     }
+
+    /**
+     * Method responsible for update a transaction by id with request options
+     *
+     * @param orderId orderId
+     * @param transactionId transactionId
+     * @param requestOptions Metadata to customize the request
+     * @return The response for the order transaction
+     * @throws MPException an error if the request fails
+     * @throws MPApiException an error if the request fails
+     */
+    public OrderTransaction updateTransaction(String orderId, String transactionId, OrderTransactionRequest request,
+                                                     MPRequestOptions requestOptions) throws MPException, MPApiException {
+        LOGGER.info("Sending order transaction update request");
+
+        if (StringUtils.isBlank(orderId)) {
+            throw new IllegalArgumentException("Order id cannot be null or empty");
+        }
+
+        if (StringUtils.isBlank(transactionId)) {
+            throw new IllegalArgumentException("Transaction id cannot be null or empty");
+        }
+
+        String url = String.format(URL_TRANSACTION + "/%s", orderId, transactionId);
+
+        MPRequest mpRequest = MPRequest.builder()
+                .uri(url)
+                .method(HttpMethod.PUT)
+                .payload(Serializer.serializeToJson(request))
+                .build();
+
+        MPResponse response = send(mpRequest, requestOptions);
+
+        OrderTransaction order = Serializer.deserializeFromJson(OrderTransaction.class, response.getContent());
+        order.setResponse(response);
+
+        return order;
+    }
+    /**
+     * Method responsible for updating a transaction for an order
+     *
+     * @param orderId The ID of the order for which the transaction is created
+     * @param transactionId The ID of the transaction to be updated
+     * @param request The request object containing transaction details
+     * @return The response for the order transaction
+     * @throws MPException an error if the request fails
+     * @throws MPApiException an error if the request fails
+     */
+    public OrderTransaction updateTransaction(String orderId,String transactionId, OrderTransactionRequest request)
+            throws MPException, MPApiException {
+        return this.updateTransaction(orderId, transactionId, request, null);
+    }
 }

src/main/java/com/mercadopago/example/apis/order/UpdateTransaction.java

@@ -0,0 +1,5 @@
+package com.mercadopago.example.apis.order;
+
+public class UpdateTransaction {
+
+}

src/test/java/com/mercadopago/client/order/OrderClientTest.java

@@ -26,6 +26,7 @@ class OrderClientTest extends BaseClientTest {
     //File Mock Responses
     private static final String CREATE_ORDER_RESPONSE_FILE = "order/create_order_response.json";
     private static final String CREATE_TRANSACTION_RESPONSE_FILE = "order/create_transaction_response.json";
+    private static final String UPDATE_TRANSACTION_FILE = "order/update_transaction_response.json";
 
     private final OrderClient client = new OrderClient();
 
@@ -150,4 +151,37 @@ void createTransactionWithRequestOptionsSuccess() throws MPException, MPApiExcep
         Assertions.assertEquals("BRL", orderTransaction.getPayments().get(0).getCurrency());
         Assertions.assertEquals("master", orderTransaction.getPayments().get(0).getPaymentMethod().getId());
     }
+
+    @Test
+    void updateTransactionIntentSuccess() throws MPException, MPApiException, IOException {
+        HttpResponse response = MockHelper.generateHttpResponseFromFile(UPDATE_TRANSACTION_FILE, HttpStatus.OK);
+
+        Mockito.doReturn(response).when(HTTP_CLIENT).execute(any(HttpRequestBase.class), any(HttpContext.class));
+
+        String orderId = "123";
+        String transactionId = "pay_012345";
+        OrderPaymentRequest paymentRequest = OrderPaymentRequest.builder()
+                .amount("100.00")
+                .currency("BRL")
+                .paymentMethod(OrderPaymentMethodRequest.builder()
+                        .id("master")
+                        .type("credit_card")
+                        .token("some-token")
+                        .installments(1)
+                        .issuerId("701")
+                        .statementDescriptor("statement")
+                        .build())
+                .build();
+
+        OrderTransactionRequest request = OrderTransactionRequest.builder()
+                .payments(Collections.singletonList(paymentRequest))
+                .build();
+
+        OrderTransaction orderTransaction = client.updateTransaction(orderId, transactionId, request);
+
+        Assertions.assertNotNull(orderTransaction);
+        Assertions.assertEquals("100.00", orderTransaction.getPayments().get(0).getAmount());
+        Assertions.assertEquals("BRL", orderTransaction.getPayments().get(0).getCurrency());
+        Assertions.assertEquals("master", orderTransaction.getPayments().get(0).getPaymentMethod().getId());
+    }
 }

src/test/java/com/mercadopago/resources/mocks/response/order/update_transaction_response.json

@@ -0,0 +1,11 @@
+{
+  "id": "pay_012345",
+  "amount": "100.00",
+  "payment_method": {
+    "id": "master",
+    "type": "credit_card",
+    "token": "card_token",
+    "statement_descriptor": "statement",
+    "installments": 1
+  }
+}