From 68eb9e3d843cb8a981e1508c9f22e05670da1980 Mon Sep 17 00:00:00 2001
From: Kilian Melcher <kilian.melcher@gmail.com>
Date: Tue, 18 Jan 2022 08:32:06 -0300
Subject: [PATCH] [FIX] Format code black and add newline at the end of readme
 files

---
 payment_pagseguro/controllers/main.py          | 11 ++++++++---
 .../data/payment_acquirer_data.xml             |  1 -
 .../models/payment_transaction.py              | 18 +++++++++---------
 payment_pagseguro/readme/DESCRIPTION.rst       |  2 +-
 payment_pagseguro/readme/INSTALL.rst           |  2 +-
 payment_pagseguro/tests/test_pagseguro.py      |  9 +++------
 6 files changed, 22 insertions(+), 21 deletions(-)

diff --git a/payment_pagseguro/controllers/main.py b/payment_pagseguro/controllers/main.py
index 7b0a8145ac58..2d23536f74fa 100644
--- a/payment_pagseguro/controllers/main.py
+++ b/payment_pagseguro/controllers/main.py
@@ -18,11 +18,16 @@ class PagseguroController(http.Controller):
     def pagseguro_s2s_create_json_3ds(self, verify_validity=False, **kwargs):
         if not kwargs.get("partner_id"):
             kwargs = dict(kwargs, partner_id=request.env.user.partner_id.id)
-        token = request.env["payment.acquirer"].browse(
-            int(kwargs.get("acquirer_id"))).s2s_process(kwargs)
+        token = (
+            request.env["payment.acquirer"]
+            .browse(int(kwargs.get("acquirer_id")))
+            .s2s_process(kwargs)
+        )
 
         if not token:
-            res = {"result": False, }
+            res = {
+                "result": False,
+            }
         else:
             res = {
                 "result": True,
diff --git a/payment_pagseguro/data/payment_acquirer_data.xml b/payment_pagseguro/data/payment_acquirer_data.xml
index a9ff679a9cac..d3b180c23ef3 100644
--- a/payment_pagseguro/data/payment_acquirer_data.xml
+++ b/payment_pagseguro/data/payment_acquirer_data.xml
@@ -18,7 +18,6 @@
             <field name="save_token">none</field>
             <field name="payment_flow">s2s</field>
             <field name="capture_manually">True</field>
-            
             <field
                 name="pre_msg"
             ><![CDATA[
diff --git a/payment_pagseguro/models/payment_transaction.py b/payment_pagseguro/models/payment_transaction.py
index d25b11143ad6..affd2bf43185 100644
--- a/payment_pagseguro/models/payment_transaction.py
+++ b/payment_pagseguro/models/payment_transaction.py
@@ -105,8 +105,8 @@ def pagseguro_s2s_capture_transaction(self):
             self.write(
                 {
                     "date": fields.datetime.now(),
-                    "acquirer_reference": res['id'],
-                    "state_message": res['payment_response']['message'],
+                    "acquirer_reference": res["id"],
+                    "state_message": res["payment_response"]["message"],
                 }
             )
             self._set_transaction_done()
@@ -114,8 +114,8 @@ def pagseguro_s2s_capture_transaction(self):
         else:
             self.sudo().write(
                 {
-                    "state_message": res['error_messages'][0]['message'],
-                    "acquirer_reference": res['error_messages'][0]['code'],
+                    "state_message": res["error_messages"][0]["message"],
+                    "acquirer_reference": res["error_messages"][0]["code"],
                     "date": fields.datetime.now(),
                 }
             )
@@ -136,7 +136,7 @@ def pagseguro_s2s_void_transaction(self):
             "charge_id": self.id,
             "amount": {
                 "value": int(self.amount * 100),
-            }
+            },
         }
 
         r = requests.post(
@@ -158,16 +158,16 @@ def pagseguro_s2s_void_transaction(self):
             self.write(
                 {
                     "date": fields.datetime.now(),
-                    "acquirer_reference": res['id'],
-                    "state_message": res['payment_response']['message'],
+                    "acquirer_reference": res["id"],
+                    "state_message": res["payment_response"]["message"],
                 }
             )
             self._set_transaction_cancel()
         else:
             self.sudo().write(
                 {
-                    "state_message": res['error_messages'][0]['message'],
-                    "acquirer_reference": res['error_messages'][0]['code'],
+                    "state_message": res["error_messages"][0]["message"],
+                    "acquirer_reference": res["error_messages"][0]["code"],
                     "date": fields.datetime.now(),
                 }
             )
diff --git a/payment_pagseguro/readme/DESCRIPTION.rst b/payment_pagseguro/readme/DESCRIPTION.rst
index 062001c73f9d..4c3132bd6c60 100644
--- a/payment_pagseguro/readme/DESCRIPTION.rst
+++ b/payment_pagseguro/readme/DESCRIPTION.rst
@@ -8,4 +8,4 @@ checkout transparent approach.
 You can learn more about that on this `youtube video <https://www.youtube.com/watch?v=3DxKEKoSPkA>`_.
 
 If you wish to use Pagseguro on production environment you need to get an approval
-from pagseguro, more on that `here <https://dev.pagseguro.uol.com.br/docs/comecando-processo-de-integracao>`_.
\ No newline at end of file
+from pagseguro, more on that `here <https://dev.pagseguro.uol.com.br/docs/comecando-processo-de-integracao>`_.
diff --git a/payment_pagseguro/readme/INSTALL.rst b/payment_pagseguro/readme/INSTALL.rst
index f703b4bb2416..d782fad429b5 100644
--- a/payment_pagseguro/readme/INSTALL.rst
+++ b/payment_pagseguro/readme/INSTALL.rst
@@ -1,4 +1,4 @@
 This module depends on:
 
 * payment
-* web_tour
\ No newline at end of file
+* web_tour
diff --git a/payment_pagseguro/tests/test_pagseguro.py b/payment_pagseguro/tests/test_pagseguro.py
index fcaa6f77398d..05f313f83de6 100644
--- a/payment_pagseguro/tests/test_pagseguro.py
+++ b/payment_pagseguro/tests/test_pagseguro.py
@@ -2,7 +2,6 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 import time
-import logging
 
 import odoo
 
@@ -18,12 +17,10 @@ def test_buy_pagseguro(self):
             timeout=20000,
         )
 
-        tx = self.env['payment.transaction'].search([], limit=1, order='id desc')
+        tx = self.env["payment.transaction"].search([], limit=1, order="id desc")
         tx.pagseguro_s2s_capture_transaction()
-        self.assertEqual(tx.state, 'done',
-                         'transaction state should be authorized')
+        self.assertEqual(tx.state, "done", "transaction state should be authorized")
 
         time.sleep(3)
         tx.pagseguro_s2s_void_transaction()
-        self.assertEqual(tx.state, 'done',
-                         'transaction state should be done')
+        self.assertEqual(tx.state, "done", "transaction state should be done")