Fix linting errors

- Fix line length issues in client.py
- Auto-format all files with ruff

Author: shihweilo

tappay/__init__.py

@@ -1,9 +1,14 @@
-
 __version__ = "0.5.0"
 
 from tappay.client import Client
+from tappay.exceptions import (
+    AuthenticationError,
+    ClientError,
+    Error,
+    Exceptions,
+    ServerError,
+)
 from tappay.models import Models
-from tappay.exceptions import Exceptions, Error, ClientError, ServerError, AuthenticationError
 
 __all__ = [
     "Client",

tappay/client.py

@@ -1,22 +1,23 @@
 import logging
 import os
 from platform import python_version
-from typing import Optional, Dict, Any, Union
+from typing import Any, Dict, Optional
 
 import requests
 
-from tappay.models import Models
 from tappay.exceptions import Exceptions
+from tappay.models import Models
 
 logger = logging.getLogger(__name__)
 
-# We need to access __version__ from somewhere, commonly from a package level or hardcoded here then imported.
-# For now I will hardcode it here or pass it in.
-# Better yet, I will define __version__ in __init__.py and import it here? No, circular import.
-# I will put __version__ in a separate file or keep it in __init__ and pass it to Client if needed,
-# or just duplicate/move it.
-# Let's define it in client.py for now to avoid circular dependency if __init__ imports client.
-# Actually, the original code used it in User-Agent.
+# We need to access __version__ from somewhere, commonly from a package
+# level or hardcoded here then imported. For now I will hardcode it here
+# or pass it in. Better yet, I will define __version__ in __init__.py and
+# import it here? No, circular import. I will put __version__ in a
+# separate file or keep it in __init__ and pass it to Client if needed,
+# or just duplicate/move it. Let's define it in client.py for now to
+# avoid circular dependency if __init__ imports client. Actually, the
+# original code used it in User-Agent.
 VERSION = "0.5.0"
 
 
@@ -81,7 +82,8 @@ def pay_by_prime(
         """
         if not isinstance(card_holder_data, Models.CardHolderData):
             raise TypeError(
-                f"expected `CardHolderData` type for parameter `card_holder_data`, {type(card_holder_data)} found"
+                f"expected `CardHolderData` type for parameter "
+                f"`card_holder_data`, {type(card_holder_data)} found"
             )
 
         params = {
@@ -126,9 +128,7 @@ def pay_by_token(
             "/tpc/payment/pay-by-token", params
         )
 
-    def refund(
-        self, rec_trade_id: str, amount: int, **kwargs: Any
-    ) -> Dict[str, Any]:
+    def refund(self, rec_trade_id: str, amount: int, **kwargs: Any) -> Dict[str, Any]:
         """
         Refund a payment
         Ref: https://docs.tappaysdk.com/tutorial/zh/back.html#refund-api
@@ -196,9 +196,7 @@ def get_trade_history(self, rec_trade_id: str) -> Dict[str, Any]:
             "rec_trade_id": rec_trade_id,
         }
 
-        return self.__post_with_partner_key(
-            "/tpc/transaction/trade-history", params
-        )
+        return self.__post_with_partner_key("/tpc/transaction/trade-history", params)
 
     def bind_card(
         self,
@@ -212,7 +210,8 @@ def bind_card(
         """
         if not isinstance(card_holder_data, Models.CardHolderData):
             raise TypeError(
-                f"expected `CardHolderData` type for parameter `card_holder_data`, {type(card_holder_data)} found"
+                f"expected `CardHolderData` type for parameter "
+                f"`card_holder_data`, {type(card_holder_data)} found"
             )
 
         params = {
@@ -224,9 +223,7 @@ def bind_card(
         if kwargs:
             params.update(**kwargs)
 
-        return self.__post_with_partner_key_and_merchant_id(
-            "/tpc/card/bind", params
-        )
+        return self.__post_with_partner_key_and_merchant_id("/tpc/card/bind", params)
 
     def remove_card(self, card_key: str, card_token: str) -> Dict[str, Any]:
         """
@@ -240,9 +237,7 @@ def remove_card(self, card_key: str, card_token: str) -> Dict[str, Any]:
 
         return self.__post_with_partner_key("/tpc/card/remove", params)
 
-    def cancel_refund(
-        self, rec_trade_id: str, **kwargs: Any
-    ) -> Dict[str, Any]:
+    def cancel_refund(self, rec_trade_id: str, **kwargs: Any) -> Dict[str, Any]:
         """
         Cancel a single refund
         Ref: https://docs.tappaysdk.com/tutorial/zh/advanced.html#refund-cancel-api
@@ -254,9 +249,7 @@ def cancel_refund(
         if kwargs:
             params.update(**kwargs)
 
-        return self.__post_with_partner_key(
-            "/tpc/transaction/refund/cancel", params
-        )
+        return self.__post_with_partner_key("/tpc/transaction/refund/cancel", params)
 
     def __post_with_partner_key(
         self, request_uri: str, params: Dict[str, Any]
@@ -297,7 +290,7 @@ def __parse(self, response: requests.Response) -> Optional[Dict[str, Any]]:
         elif 500 <= response.status_code < 600:
             message = f"{response.status_code} response from {self.api_host}"
             raise Exceptions.ServerError(message)
-        
+
         # Fallback for unexpected status codes
         message = f"Unexpected status code {response.status_code} from {self.api_host}"
         raise Exceptions.ServerError(message)

tappay/exceptions.py

@@ -1,19 +1,22 @@
-
 class Error(Exception):
     """Base exception for all TapPay errors."""
+
     pass
 
 
 class ClientError(Error):
     """Error raised when the client sends an invalid request."""
+
     pass
 
 
 class ServerError(Error):
     """Error raised when the TapPay server encounters an issue."""
+
     pass
 
 
 class AuthenticationError(ClientError):
     """Error raised when authentication fails."""
+
     pass

tappay/models.py

@@ -1,10 +1,12 @@
-from typing import Optional, Dict, Any
+from typing import Any, Dict, Optional
+
 
 class Models:
     """Namespace for TapPay models."""
 
     class Currencies:
         """Currency constants."""
+
         TWD = "TWD"
 
     class CardHolderData:

tests/conftest.py

@@ -1,27 +1,22 @@
-
 import pytest
+
 from tappay.client import Client, Models
 
+
 @pytest.fixture
 def sandbox_client():
-    return Client(
-        is_sandbox=True,
-        partner_key="partner_key",
-        merchant_id="merchant_id"
-    )
+    return Client(is_sandbox=True, partner_key="partner_key", merchant_id="merchant_id")
+
 
 @pytest.fixture
 def production_client():
     return Client(
-        is_sandbox=False,
-        partner_key="partner_key",
-        merchant_id="merchant_id"
+        is_sandbox=False, partner_key="partner_key", merchant_id="merchant_id"
     )
 
+
 @pytest.fixture
 def card_holder():
     return Models.CardHolderData(
-        phone_number="0912345678",
-        name="Wang Xiao Ming",
-        email="[email protected]"
+        phone_number="0912345678", name="Wang Xiao Ming", email="[email protected]"
     )

tests/test_client.py

@@ -1,19 +1,23 @@
+from unittest.mock import Mock, patch
 
 import pytest
-from unittest.mock import patch, Mock
+
 from tappay.client import Client, Models
 from tappay.exceptions import AuthenticationError, ClientError, ServerError
 
+
 def test_client_initialization_sandbox():
     client = Client(is_sandbox=True, partner_key="pk", merchant_id="mid")
     assert client.api_host == "sandbox.tappaysdk.com"
     assert client.partner_key == "pk"
     assert client.merchant_id == "mid"
 
+
 def test_client_initialization_production():
     client = Client(is_sandbox=False, partner_key="pk", merchant_id="mid")
     assert client.api_host == "prod.tappaysdk.com"
 
+
 def test_pay_by_prime(sandbox_client, card_holder):
     with patch("tappay.client.requests.post") as mock_post:
         mock_response = Mock()
@@ -22,10 +26,7 @@ def test_pay_by_prime(sandbox_client, card_holder):
         mock_post.return_value = mock_response
 
         response = sandbox_client.pay_by_prime(
-            prime="test_prime",
-            amount=100,
-            details="test",
-            card_holder_data=card_holder
+            prime="test_prime", amount=100, details="test", card_holder_data=card_holder
         )
 
         assert response["status"] == 0
@@ -35,15 +36,14 @@ def test_pay_by_prime(sandbox_client, card_holder):
         assert kwargs["json"]["amount"] == 100
         assert kwargs["json"]["currency"] == "TWD"
 
+
 def test_pay_by_prime_invalid_cardholder(sandbox_client):
     with pytest.raises(TypeError):
         sandbox_client.pay_by_prime(
-            prime="p",
-            amount=100,
-            details="d",
-            card_holder_data={}
+            prime="p", amount=100, details="d", card_holder_data={}
         )
 
+
 def test_api_error_handling_401(sandbox_client):
     with patch("tappay.client.requests.post") as mock_post:
         mock_response = Mock()
@@ -55,17 +55,19 @@ def test_api_error_handling_401(sandbox_client):
                 prime="p",
                 amount=1,
                 details="d",
-                card_holder_data=Models.CardHolderData("p", "n", "e")
+                card_holder_data=Models.CardHolderData("p", "n", "e"),
             )
 
+
 def test_api_error_handling_400(sandbox_client):
     with patch("tappay.client.requests.post") as mock_post:
         mock_response = Mock()
         mock_response.status_code = 400
         mock_post.return_value = mock_response
 
         with pytest.raises(ClientError):
-             sandbox_client.capture_today("id")
+            sandbox_client.capture_today("id")
+
 
 def test_api_error_handling_500(sandbox_client):
     with patch("tappay.client.requests.post") as mock_post:
@@ -74,4 +76,4 @@ def test_api_error_handling_500(sandbox_client):
         mock_post.return_value = mock_response
 
         with pytest.raises(ServerError):
-             sandbox_client.capture_today("id")
+            sandbox_client.capture_today("id")