Merge pull request #14 from payjp/sdk-update-2025-12-15-b528390

feat: Update SDK from generator (2025-12-15)

Author: wozozo

tests/test_api_mocks.py

@@ -124,7 +124,7 @@ def test_create_payment_flow_success(self, mock_call_api, mock_deserialize, api_
         mock_api_response.data = sample_payment_flow_data
         mock_deserialize.return_value = mock_api_response
 
-        request = PaymentFlowCreateRequest(amount=1000)
+        request = PaymentFlowCreateRequest(amount=1000, currency="jpy")
         response = payment_flows_api.create_payment_flow(request)
 
         # Verify API call
@@ -288,6 +288,7 @@ def test_payment_flow_model_creation(self):
         """Test payment flow model creation."""
         request = PaymentFlowCreateRequest(
             amount=1000,
+            currency="jpy",
             customer_id="cus_test123",
             description="Test payment flow"
         )

tests/test_error_handling.py

@@ -210,10 +210,11 @@ def test_minimum_amount_validation(self):
 
     def test_invalid_currency_validation(self):
         """Test validation of currency codes (if implemented)."""
-        # PaymentFlowCreateRequest doesn't have currency field in this implementation
-        # Test with a different field that exists
+        # PaymentFlowCreateRequest now has currency as a required field
+        # Test with description field
         request = PaymentFlowCreateRequest(
             amount=1000,
+            currency="jpy",
             description="INVALID_DESCRIPTION_TEST"
         )
         assert request.description == "INVALID_DESCRIPTION_TEST"

tests/test_models.py

@@ -39,13 +39,14 @@ class TestPaymentFlowModels:
 
     def test_payment_flow_create_request_basic(self):
         """Test basic payment flow creation request."""
-        request = PaymentFlowCreateRequest(amount=1000)
+        request = PaymentFlowCreateRequest(amount=1000, currency="jpy")
         assert request.amount == 1000
 
     def test_payment_flow_create_request_with_customer(self):
         """Test payment flow creation with customer."""
         request = PaymentFlowCreateRequest(
             amount=1000,
+            currency="jpy",
             customer_id="cus_test123"
         )
         assert request.amount == 1000
@@ -55,6 +56,7 @@ def test_payment_flow_create_request_with_description(self):
         """Test payment flow creation with description."""
         request = PaymentFlowCreateRequest(
             amount=1500,
+            currency="jpy",
             description="Test payment for SDK"
         )
         assert request.amount == 1500
@@ -63,21 +65,21 @@ def test_payment_flow_create_request_with_description(self):
     def test_payment_flow_amount_types(self):
         """Test different amount values."""
         # Minimum valid amount (50)
-        request1 = PaymentFlowCreateRequest(amount=50)
+        request1 = PaymentFlowCreateRequest(amount=50, currency="jpy")
         assert request1.amount == 50
 
         # Normal amount
-        request2 = PaymentFlowCreateRequest(amount=1000)
+        request2 = PaymentFlowCreateRequest(amount=1000, currency="jpy")
         assert request2.amount == 1000
 
         # Maximum valid amount (9999999)
-        request3 = PaymentFlowCreateRequest(amount=9999999)
+        request3 = PaymentFlowCreateRequest(amount=9999999, currency="jpy")
         assert request3.amount == 9999999
 
     def test_payment_flow_capture_method(self):
         """Test capture method settings if available."""
         # Test that capture_method field exists and can be set
-        request = PaymentFlowCreateRequest(amount=1000)
+        request = PaymentFlowCreateRequest(amount=1000, currency="jpy")
         if hasattr(request, 'capture_method'):
             # Only test if the field exists in this implementation
             assert request.capture_method is None  # Default value
@@ -106,6 +108,7 @@ def test_payment_flow_request_serialization(self):
         """Test payment flow request serialization."""
         request = PaymentFlowCreateRequest(
             amount=1000,
+            currency="jpy",
             description="Test payment"
         )
 
@@ -150,7 +153,7 @@ def test_amount_flexibility(self):
         amounts = [50, 100, 1000, 9999999]
 
         for amount in amounts:
-            request = PaymentFlowCreateRequest(amount=amount)
+            request = PaymentFlowCreateRequest(amount=amount, currency="jpy")
             assert request.amount == amount
 
     def test_model_optional_fields(self):
@@ -160,7 +163,7 @@ def test_model_optional_fields(self):
         assert customer_request.email == "test@example.com"
         assert customer_request.description is None
 
-        # Test payment flow with just amount
-        payment_request = PaymentFlowCreateRequest(amount=1000)
+        # Test payment flow with just amount and currency (required fields)
+        payment_request = PaymentFlowCreateRequest(amount=1000, currency="jpy")
         assert payment_request.amount == 1000
         assert payment_request.customer_id is None

tests/test_sdk_functionality.py

@@ -90,13 +90,14 @@ class TestPaymentFlowOperations:
 
     def test_payment_flow_create_request_basic(self):
         """Test basic payment flow creation request."""
-        request = PaymentFlowCreateRequest(amount=1000)
+        request = PaymentFlowCreateRequest(amount=1000, currency="jpy")
         assert request.amount == 1000
 
     def test_payment_flow_create_request_with_customer(self):
         """Test payment flow creation with customer."""
         request = PaymentFlowCreateRequest(
             amount=1000,
+            currency="jpy",
             customer_id="cus_test123"
         )
         assert request.amount == 1000
@@ -106,6 +107,7 @@ def test_payment_flow_create_request_with_description(self):
         """Test payment flow creation with description."""
         request = PaymentFlowCreateRequest(
             amount=1500,
+            currency="jpy",
             description="Test payment for SDK"
         )
         assert request.amount == 1500
@@ -145,6 +147,7 @@ def test_payment_flow_request_serialization(self):
         """Test payment flow request serialization."""
         request = PaymentFlowCreateRequest(
             amount=1000,
+            currency="jpy",
             description="Test payment"
         )
 
@@ -188,11 +191,11 @@ def test_model_validation_flexibility(self):
         request1 = CustomerCreateRequest(email="")
         assert request1.email == ""
 
-        request2 = PaymentFlowCreateRequest(amount=50)
+        request2 = PaymentFlowCreateRequest(amount=50, currency="jpy")
         assert request2.amount == 50
 
         # Test with max valid value (PaymentFlowCreateRequest requires <= 9999999)
-        request3 = PaymentFlowCreateRequest(amount=9999999)
+        request3 = PaymentFlowCreateRequest(amount=9999999, currency="jpy")
         assert request3.amount == 9999999
 
 
@@ -250,6 +253,7 @@ def test_payment_flow_workflow_simulation(self, api_client):
         # Test creating request objects for workflow
         create_request = PaymentFlowCreateRequest(
             amount=2500,
+            currency="jpy",
             customer_id="cus_workflow_test",
             description="Workflow test payment"
         )
@@ -274,6 +278,7 @@ def test_multi_api_coordination(self, api_client):
         customer_request = CustomerCreateRequest(email="coordination@example.com")
         payment_request = PaymentFlowCreateRequest(
             amount=1500,
+            currency="jpy",
             customer_id="cus_coordination_test"
         )
 

tests/test_simple_functionality.py

@@ -117,13 +117,14 @@ def test_customer_create_request_with_description(self):
 
     def test_payment_flow_create_request_basic(self):
         """Test basic payment flow creation request."""
-        request = PaymentFlowCreateRequest(amount=1000)
+        request = PaymentFlowCreateRequest(amount=1000, currency="jpy")
         assert request.amount == 1000
 
     def test_payment_flow_create_request_with_customer(self):
         """Test payment flow creation request with customer."""
         request = PaymentFlowCreateRequest(
             amount=1000,
+            currency="jpy",
             customer_id="cus_test123"
         )
         assert request.amount == 1000
@@ -133,6 +134,7 @@ def test_payment_flow_create_request_with_options(self):
         """Test payment flow creation request with various options."""
         request = PaymentFlowCreateRequest(
             amount=1500,
+            currency="jpy",
             customer_id="cus_test123",
             description="Test payment for SDK",
             confirm=True
@@ -162,14 +164,14 @@ def test_customer_request_to_json(self):
 
     def test_payment_flow_request_to_string(self):
         """Test payment flow request string representation."""
-        request = PaymentFlowCreateRequest(amount=1000)
+        request = PaymentFlowCreateRequest(amount=1000, currency="jpy")
         str_repr = request.to_str()
         assert isinstance(str_repr, str)
         assert "1000" in str_repr
 
     def test_payment_flow_request_to_json(self):
         """Test payment flow request JSON serialization."""
-        request = PaymentFlowCreateRequest(amount=1000)
+        request = PaymentFlowCreateRequest(amount=1000, currency="jpy")
         json_str = request.to_json()
         assert isinstance(json_str, str)
         assert "1000" in json_str