cuenca-validations 2.1.49.dev0__tar.gz → 2.1.50.dev0__tar.gz

{cuenca_validations-2.1.49.dev0 → cuenca_validations-2.1.50.dev0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cuenca_validations
-Version: 2.1.49.dev0
+Version: 2.1.50.dev0
 Summary: Cuenca common validations
 Home-page: https://github.com/cuenca-mx/cuenca-validations
 Author: Cuenca

{cuenca_validations-2.1.49.dev0 → cuenca_validations-2.1.50.dev0}/cuenca_validations/types/__init__.py

@@ -114,6 +114,7 @@ __all__ = [
     'TransferQuery',
     'TransferRequest',
     'UpdateTransferOrderRequest',
+    'UpdateTransferOrderStatus',
     'UpdateTransferRequest',
     'UserCardNotification',
     'UserCredentialRequest',
@@ -191,6 +192,7 @@ from .enums import (
     TransactionStatus,
     TransferNetwork,
     TransferOrderStatus,
+    UpdateTransferOrderStatus,
     UserCardNotification,
     UserStatus,
     UserType,

{cuenca_validations-2.1.49.dev0 → cuenca_validations-2.1.50.dev0}/cuenca_validations/types/enums.py

@@ -768,3 +768,8 @@ class TransferOrderStatus(str, Enum):
     authorized = 'authorized'
     rejected = 'rejected'
     expired = 'expired'
+
+
+class UpdateTransferOrderStatus(str, Enum):
+    authorized = 'authorized'
+    rejected = 'rejected'

{cuenca_validations-2.1.49.dev0 → cuenca_validations-2.1.50.dev0}/cuenca_validations/types/requests.py

@@ -1,5 +1,5 @@
 import datetime as dt
-from typing import Annotated, Any, Literal, Optional, Union
+from typing import Annotated, Any, Optional, Union
 
 from clabe import BANK_NAMES, Clabe
 from pydantic import (
@@ -50,7 +50,7 @@ from ..types.enums import (
     TrackDataMethod,
     TransactionStatus,
     TransactionTokenValidationStatus,
-    TransferOrderStatus,
+    UpdateTransferOrderStatus,
     UserCardNotification,
     UserStatus,
     VerificationStatus,
@@ -1042,7 +1042,7 @@ class TransferOrderLineRequest(BaseRequest):
 
 class TransferOrderRequest(BaseRequest):
     idempotency_key: str
-    legal_person_id: Optional[str] = None
+    legal_person_id: str
     items: list[TransferOrderLineRequest]
 
     model_config = ConfigDict(
@@ -1070,9 +1070,7 @@ class TransferOrderRequest(BaseRequest):
 
 
 class UpdateTransferOrderRequest(BaseRequest):
-    status: Literal[
-        TransferOrderStatus.authorized, TransferOrderStatus.rejected
-    ]
+    status: UpdateTransferOrderStatus
     rejection_reason: Optional[str] = None
 
     model_config = ConfigDict(
@@ -1089,7 +1087,7 @@ class UpdateTransferOrderRequest(BaseRequest):
 
     @model_validator(mode='after')
     def validate_rejection_reason(self) -> 'UpdateTransferOrderRequest':
-        if self.status is TransferOrderStatus.rejected:
+        if self.status is UpdateTransferOrderStatus.rejected:
             if (
                 self.rejection_reason is None
                 or not self.rejection_reason.strip()

cuenca_validations-2.1.50.dev0/cuenca_validations/version.py

@@ -0,0 +1 @@
+__version__ = '2.1.50.dev0'

{cuenca_validations-2.1.49.dev0 → cuenca_validations-2.1.50.dev0}/cuenca_validations.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cuenca_validations
-Version: 2.1.49.dev0
+Version: 2.1.50.dev0
 Summary: Cuenca common validations
 Home-page: https://github.com/cuenca-mx/cuenca-validations
 Author: Cuenca

{cuenca_validations-2.1.49.dev0 → cuenca_validations-2.1.50.dev0}/tests/test_requests.py

@@ -5,7 +5,7 @@ from pydantic_extra_types.phone_numbers import PhoneNumber
 from cuenca_validations.types.enums import (
     Country,
     OperatorRole,
-    TransferOrderStatus,
+    UpdateTransferOrderStatus,
     VerificationType,
 )
 from cuenca_validations.types.queries import OperatorQuery
@@ -245,49 +245,59 @@ def test_transfer_order_request_shape() -> None:
         'descriptor': 'Mezcal',
     }
     single = TransferOrderRequest.model_validate(
-        {'idempotency_key': 'single', 'items': [line]}
+        {
+            'idempotency_key': 'single',
+            'legal_person_id': 'USWqY5cvkISJOxHyEKjAKf8w',
+            'items': [line],
+        }
     )
+    assert single.legal_person_id == 'USWqY5cvkISJOxHyEKjAKf8w'
     assert single.items is not None
     assert len(single.items) == 1
     batch = TransferOrderRequest.model_validate(
-        {'idempotency_key': 'batch', 'items': [line, line]}
+        {
+            'idempotency_key': 'batch',
+            'legal_person_id': 'USWqY5cvkISJOxHyEKjAKf8w',
+            'items': [line, line],
+        }
     )
     assert batch.items is not None
     assert len(batch.items) == 2
 
     with pytest.raises(ValidationError, match='items must not be empty'):
         TransferOrderRequest.model_validate(
-            {'idempotency_key': 'empty', 'items': []}
+            {
+                'idempotency_key': 'empty',
+                'legal_person_id': 'USWqY5cvkISJOxHyEKjAKf8w',
+                'items': [],
+            }
         )
     with pytest.raises(ValidationError):
-        TransferOrderRequest.model_validate({'idempotency_key': 'missing'})
-
-
-def test_update_transfer_order_request_authorized() -> None:
-    req = UpdateTransferOrderRequest.model_validate({'status': 'authorized'})
-    assert req.status is TransferOrderStatus.authorized
-    assert req.rejection_reason is None
-
-
-def test_update_transfer_order_request_rejected() -> None:
-    req = UpdateTransferOrderRequest.model_validate(
-        {'status': 'rejected', 'rejection_reason': 'Monto incorrecto'}
-    )
-    assert req.status is TransferOrderStatus.rejected
-    assert req.rejection_reason == 'Monto incorrecto'
+        TransferOrderRequest.model_validate(
+            {
+                'idempotency_key': 'missing-items',
+                'legal_person_id': 'USWqY5cvkISJOxHyEKjAKf8w',
+            }
+        )
+    with pytest.raises(ValidationError, match='legal_person_id'):
+        TransferOrderRequest.model_validate(
+            {'idempotency_key': 'missing-person', 'items': [line]}
+        )
 
 
-def test_update_transfer_order_request_rejected_requires_reason() -> None:
-    with pytest.raises(ValidationError, match='rejection_reason is required'):
-        UpdateTransferOrderRequest.model_validate({'status': 'rejected'})
-    with pytest.raises(ValidationError, match='rejection_reason is required'):
-        UpdateTransferOrderRequest.model_validate(
-            {'status': 'rejected', 'rejection_reason': ''}
-        )
+@pytest.mark.parametrize(
+    'payload',
+    [
+        {'status': 'rejected'},
+        {'status': 'rejected', 'rejection_reason': ''},
+        {'status': 'rejected', 'rejection_reason': '   '},
+    ],
+)
+def test_update_transfer_order_request_rejected_requires_reason(
+    payload: dict,
+) -> None:
     with pytest.raises(ValidationError, match='rejection_reason is required'):
-        UpdateTransferOrderRequest.model_validate(
-            {'status': 'rejected', 'rejection_reason': '   '}
-        )
+        UpdateTransferOrderRequest.model_validate(payload)
 
 
 def test_update_transfer_order_request_authorized_forbids_reason() -> None:
@@ -300,18 +310,18 @@ def test_update_transfer_order_request_authorized_forbids_reason() -> None:
         )
 
 
-def test_update_transfer_order_request_rejects_invalid_status() -> None:
-    with pytest.raises(ValidationError):
-        UpdateTransferOrderRequest.model_validate({'status': 'created'})
-    with pytest.raises(ValidationError):
-        UpdateTransferOrderRequest.model_validate({'status': 'expired'})
-    with pytest.raises(ValidationError):
-        UpdateTransferOrderRequest.model_validate({})
-
-
-def test_update_transfer_order_request_forbids_extra() -> None:
-    with pytest.raises(ValidationError) as ex:
-        UpdateTransferOrderRequest.model_validate(
-            {'status': 'authorized', 'operator_id': 'OP123'}
-        )
-    assert 'Extra inputs are not permitted' in str(ex.value)
+@pytest.mark.parametrize(
+    'payload, status',
+    [
+        ({'status': 'authorized'}, UpdateTransferOrderStatus.authorized),
+        (
+            {'status': 'rejected', 'rejection_reason': 'Monto incorrecto'},
+            UpdateTransferOrderStatus.rejected,
+        ),
+    ],
+)
+def test_update_transfer_order_request_valid(
+    payload: dict, status: UpdateTransferOrderStatus
+) -> None:
+    req = UpdateTransferOrderRequest.model_validate(payload)
+    assert req.status is status

cuenca_validations-2.1.49.dev0/cuenca_validations/version.py

@@ -1 +0,0 @@
-__version__ = '2.1.49.dev0'