Merge pull request #75 from mercadopago/feature/adjusts_tests

eltinMeli · web-flow · commit 481d2cd184cc · 2024-06-26T16:49:05.000-03:00
Adjusts credential tests
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -14,7 +14,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ['3.7', '3.8', '3.9', '3.10', '3.11']
+        python-version: ['3.7', '3.8', '3.9', '3.10', '3.11', '3.12']
 
     steps:
     - uses: actions/checkout@v2
@@ -42,3 +42,5 @@ jobs:
     - name: Test with unittest
       run: |
         python -m unittest discover tests/
+      env:
+        ACCESS_TOKEN: ${{secrets.ACCESS_TOKEN}}
diff --git a/tests/test_advanced_payment.py b/tests/test_advanced_payment.py
@@ -5,18 +5,17 @@
     datetime,
     timedelta,
 )
+import os
 import unittest
 import uuid
-
 import mercadopago
 
 
 class TestAdvancedPayment(unittest.TestCase):
     """
     Test Module: Advanced Payment
     """
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     @unittest.skip(reason="Outdated API usage")
     def test_all(self):
diff --git a/tests/test_card.py b/tests/test_card.py
@@ -2,9 +2,9 @@
     Module: test_card
 """
 from datetime import datetime
+import os
 import random
 import unittest
-
 import mercadopago
 
 
@@ -14,13 +14,11 @@ class TestCard(unittest.TestCase):
     """
 
     _customer_id = None
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     @classmethod
     def setUpClass(cls):
         customer_data = cls.create_customer()
-        print(customer_data)
         cls._customer_id = customer_data["response"]["id"]
 
     @classmethod
@@ -46,7 +44,6 @@ def test_all(self):
         }
 
         card_token_created = self.sdk.card_token().create(card_token_object)
-        print(card_token_created)
 
         card_object = {
             "customer_id": self._customer_id,
@@ -58,8 +55,7 @@ def test_all(self):
         self.assertEqual(self.sdk.card().get(
             self._customer_id, card_created["response"]["id"])["status"], 200)
 
-        self.sdk.card().delete(self._customer_id,
-                               card_created["response"]["id"])
+        self.sdk.card().delete(self._customer_id, card_created["response"]["id"])
 
     @classmethod
     def create_customer(cls):
diff --git a/tests/test_card_token.py b/tests/test_card_token.py
@@ -2,17 +2,16 @@
     Module: test_card_token
 """
 from datetime import datetime
+import os
 import unittest
-
 import mercadopago
 
 
 class TestCardToken(unittest.TestCase):
     """
     Test Module: Card Token
     """
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     def test_all(self):
         """
diff --git a/tests/test_chargeback.py b/tests/test_chargeback.py
@@ -1,17 +1,16 @@
 """
     Module: test_chargeback
 """
+import os
 import unittest
-
 import mercadopago
 
 
 class TestChargeback(unittest.TestCase):
     """
     Test Module: Chargeback
     """
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     def test_search_chargeback(self):
         """
diff --git a/tests/test_customer.py b/tests/test_customer.py
@@ -1,18 +1,17 @@
 """
     Module: test_customer
 """
+import os
 import random
 import unittest
-
 import mercadopago
 
 
 class TestCustomer(unittest.TestCase):
     """
     Test Module: Customer
     """
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     def test_all(self):
         """
@@ -41,12 +40,10 @@ def test_all(self):
             customer_saved["response"]["id"], {"last_name": "Payer"})
         self.assertEqual(200, customer_update["status"])
 
-        customer_updated = self.sdk.customer().get(
-            customer_saved["response"]["id"])
+        customer_updated = self.sdk.customer().get(customer_saved["response"]["id"])
         self.assertEqual(customer_updated["response"]["last_name"], "Payer")
 
-        customer_deleted = self.sdk.customer().delete(
-            customer_saved["response"]["id"])
+        customer_deleted = self.sdk.customer().delete(customer_saved["response"]["id"])
         self.assertEqual(200, customer_deleted["status"])
 
 
diff --git a/tests/test_identification_type.py b/tests/test_identification_type.py
@@ -1,17 +1,16 @@
 """
     Module: test_identification_type
 """
+import os
 import unittest
-
 import mercadopago
 
 
 class TestIdentificationType(unittest.TestCase):
     """
     Test Module: Identification Type
     """
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     def test_find_all(self):
         """
diff --git a/tests/test_merchant_order.py b/tests/test_merchant_order.py
@@ -1,18 +1,17 @@
 """
     Module: test_merchant_order
 """
+import os
 import unittest
 import uuid
-
 import mercadopago
 
 
 class TestMerchantOrder(unittest.TestCase):
     """
     Test Module: Merchant Order
     """
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     def test_all(self):
         """
diff --git a/tests/test_payment.py b/tests/test_payment.py
@@ -2,17 +2,16 @@
     Module: test_payment
 """
 from datetime import datetime
+import os
 import unittest
-
 import mercadopago
 
 
 class TestPayment(unittest.TestCase):
     """
     Test Module: Payment
     """
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     def test_create_and_find(self):
         """
diff --git a/tests/test_payment_methods.py b/tests/test_payment_methods.py
@@ -1,17 +1,16 @@
 """
     Module: test_payment_methods
 """
+import os
 import unittest
-
 import mercadopago
 
 
 class TestPaymentMethods(unittest.TestCase):
     """
     Test Module: Payment Methods
     """
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     def test_find(self):
         """
diff --git a/tests/test_plan.py b/tests/test_plan.py
@@ -1,8 +1,8 @@
 """
     Module: test_plan
 """
+import os
 import unittest
-
 import random
 import mercadopago
 
@@ -12,8 +12,7 @@ class TestPlan(unittest.TestCase):
     Test Module: Preference
     """
 
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     def test_all(self):
         """
diff --git a/tests/test_preapproval.py b/tests/test_preapproval.py
@@ -1,17 +1,16 @@
 """
     Module: test_preapproval
 """
+import os
 import unittest
-
 import mercadopago
 
 
 class TestPreApproval(unittest.TestCase):
     """
     Test Module: PreApproval
     """
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     def test_create(self):
         """
diff --git a/tests/test_preference.py b/tests/test_preference.py
@@ -1,8 +1,8 @@
 """
     Module: test_preference
 """
+import os
 import unittest
-
 import time
 import mercadopago
 
@@ -11,8 +11,7 @@ class TestPreference(unittest.TestCase):
     """
     Test Module: Preference
     """
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     def test_all(self):
         """
@@ -33,26 +32,24 @@ def test_all(self):
         }
         preference_saved = self.sdk.preference().create(preference_object)
         self.assertEqual(preference_saved["status"], 201)
-
         time.sleep(1)
 
         preference_object["items"][0]["title"] = "Testando 1 2 3"
 
-        preference_update = self.sdk.preference().update(
-            preference_saved["response"]["id"], preference_object)
+        preference_id = preference_saved["response"]["id"]
+        preference_update = self.sdk.preference().update(preference_id, preference_object)
         self.assertEqual(preference_update["status"], 200)
 
         time.sleep(1)
-        preference_saved = self.sdk.preference().get(
-            preference_saved["response"]["id"])
-
+        preference_saved = self.sdk.preference().get(preference_id)
+        self.assertEqual(preference_saved["status"], 200)
         self.assertEqual(preference_saved["response"]["items"][0]["title"],
-                         preference_object["items"][0]["title"])
-
-        time.sleep(1)
-        preference_saved = self.sdk.preference().search()
+                         preference_object["items"][0]["title"],
+                         "Validate title")
 
-        self.assertEqual(preference_saved["response"]["elements"][0]["items"][0],
+        time.sleep(3)
+        preference_search = self.sdk.preference().search()
+        self.assertEqual(preference_search["response"]["elements"][0]["items"][0],
                          preference_object["items"][0]["title"])
 
 
diff --git a/tests/test_subscription.py b/tests/test_subscription.py
@@ -2,8 +2,8 @@
     Module: test_plan
 """
 from datetime import datetime
+import os
 import unittest
-
 import random
 import mercadopago
 
@@ -15,8 +15,7 @@ class TestSubscription(unittest.TestCase):
     _customer_id = None
     _customer_email = None
     _plan_id = None
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     @classmethod
     def setUpClass(cls):
diff --git a/tests/test_user.py b/tests/test_user.py
@@ -1,14 +1,13 @@
 """
     Module: test_user
 """
+import os
 import unittest
-
 import mercadopago
 
 
 class TestUser(unittest.TestCase):
-    sdk = mercadopago.SDK(
-        "APP_USR-558881221729581-091712-44fdc612e60e3e638775d8b4003edd51-471763966")
+    sdk = mercadopago.SDK(os.environ['ACCESS_TOKEN'])
 
     def test_find_user(self):
         self.assertEqual(self.sdk.user().get()["status"], 200)
