tests: Add comprehensive tests for various endpoints and utility functions

- Implement tests for creating plans with all optional parameters.
- Add tests for listing plans with filter parameters.
- Enhance tests for updating plans with optional parameters.
- Introduce tests for creating products with optional parameters.
- Add tests for listing products with date parameters.
- Implement tests for creating refunds with all optional parameters.
- Enhance tests for listing refunds with date parameters and invalid parameters.
- Add tests for creating subaccounts with all optional parameters.
- Implement tests for listing subaccounts with date parameters.
- Enhance tests for creating subscriptions with optional parameters.
- Add tests for listing subscriptions with all parameters.
- Implement tests for creating transfer recipients with all optional parameters.
- Enhance tests for listing transfer recipients with date parameters.
- Add tests for creating virtual terminals with all optional parameters.
- Implement tests for listing virtual terminals with all parameters.
- Add tests for validating email format and amount in utility functions.
- Introduce tests for charge authorization validation.

Author: theolujay

example.py

@@ -8,7 +8,11 @@
 if not secret_key:
     raise ValueError("PAYSTACK_SECRET_KEY environment variable not set.")
 
-print("PAYSTACK_SECRET_KEY is set. Great job, Olujay!")
+elif secret_key:
+    print("PAYSTACK_SECRET_KEY is set. Great job, Olujay!")
+    
+print(secret_key)
+
 
 client = PaystackClient(secret_key=secret_key)
 
@@ -21,6 +25,7 @@
 
     print("Transaction initialized successfully:")
     print(data)
+    print(meta)
 
 except APIError as e:
     print(f"An API error occurred: {e.message}")

paystack_client/core.py

@@ -109,11 +109,19 @@ def request(
                 )
             else:
                 # Use the factory function to create appropriate exception
-                raise create_error_from_response(
-                    response=resp_json,
-                    status_code=response.status_code,
-                    request_id=request_id,
-                )
+                # For RateLimitError, pass the original response object to access headers
+                if response.status_code == 429:
+                    raise create_error_from_response(
+                        response=response, # Pass the original response object
+                        status_code=response.status_code,
+                        request_id=request_id,
+                    )
+                else:
+                    raise create_error_from_response(
+                        response=resp_json,
+                        status_code=response.status_code,
+                        request_id=request_id,
+                    )
         except requests.exceptions.ConnectTimeout:
             raise NetworkError("Connection timed out - check your internet connection")
         except requests.exceptions.ReadTimeout:

paystack_client/endpoints/direct_debit.py

@@ -54,6 +54,6 @@ def list_mandate_authorizations(
         if status:
             params["status"] = status
         if per_page:
-            params["per_page"] = per_page
+            params["perPage"] = per_page
 
         return self.request("GET", "directdebit/mandate-authorizations", params=params)

paystack_client/endpoints/disputes.py

@@ -57,7 +57,7 @@ def list_disputes(
         if status:
             payload["status"] = status
 
-        return self.request("GET", "dispute", json_data=payload)
+        return self.request("GET", "dispute", params=payload)
 
     def fetch_dispute(self, dispute_id: str) -> Tuple[Dict[str, Any], Dict[str, Any]]:
         """
@@ -240,4 +240,4 @@ def export_disputes(
         if status:
             payload["status"] = status
 
-        return self.request("GET", "dispute/export", json_data=payload)
+        return self.request("GET", "dispute/export", params=payload)

paystack_client/endpoints/refunds.py

@@ -98,7 +98,7 @@ def list_refunds(
                 raise APIError("page must be greater than 0")
             payload["page"] = page
 
-        return self.request("GET", "refund", json_data=payload)
+        return self.request("GET", "refund", params=payload)
 
     def fetch(
         self, refund_id: Union[str, int]

paystack_client/endpoints/transactions.py

@@ -63,7 +63,7 @@ def initialize(
             payload["plan"] = plan
         if invoice_limit is not None:
             payload["invoice_limit"] = invoice_limit
-        if metadata:
+        if metadata is not None:
             # Convert metadata dict to JSON string as per API requirements
             import json
 

paystack_client/utils/validators.py

@@ -1,7 +1,7 @@
 """Validation helpers for Paystack API calls."""
 
 from typing import Union
-from ..exceptions import APIError
+from ..exceptions import APIError, ValidationError
 from .helpers import validate_email
 
 
@@ -17,10 +17,10 @@ def _validate_amount_and_email(email: str, amount: Union[int, str]):
     """
     if not email:
         raise ValidationError("Email is required")
-    if not amount:
+    if amount is None:
         raise ValidationError("Amount is required")
-    if not validate_email(email):
-        raise ValidationError("Invalid email format")
+
+    validate_email(email)
 
     try:
         amount_int = int(amount)

tests/test_client.py

@@ -0,0 +1,38 @@
+import pytest
+from paystack_client import PaystackClient
+
+def test_paystack_client_initialization():
+    client = PaystackClient(secret_key="sk_test_abcdefghijklmnopqrstuvwxyz1234567890")
+    assert client.base_url == "https://api.paystack.co/"
+    assert hasattr(client, "transactions")
+    assert hasattr(client, "customers")
+    assert hasattr(client, "charge")
+    assert hasattr(client, "plans")
+    assert hasattr(client, "products")
+    assert hasattr(client, "refunds")
+    assert hasattr(client, "settlements")
+    assert hasattr(client, "subaccounts")
+    assert hasattr(client, "subscriptions")
+    assert hasattr(client, "transfers")
+    assert hasattr(client, "transfers_control")
+    assert hasattr(client, "transfer_recipients")
+    assert hasattr(client, "verification")
+    assert hasattr(client, "disputes")
+    assert hasattr(client, "payment_pages")
+    assert hasattr(client, "payment_requests")
+    assert hasattr(client, "bulk_charges")
+    assert hasattr(client, "dedicated_virtual_accounts")
+    assert hasattr(client, "direct_debit")
+    assert hasattr(client, "apple_pay")
+    assert hasattr(client, "terminal")
+    assert hasattr(client, "virtual_terminal")
+    assert hasattr(client, "transaction_splits")
+    assert hasattr(client, "integration")
+    assert hasattr(client, "miscellaneous")
+
+    # Access the integration attribute to ensure it's covered
+    assert client.integration is not None
+
+def test_paystack_client_repr():
+    client = PaystackClient(secret_key="sk_test_abcdefghijklmnopqrstuvwxyz1234567890")
+    assert repr(client) == "PaystackClient(base_url='https://api.paystack.co/')"