cuenca-validations 2.1.48.dev1__py3-none-any.whl → 2.1.49.dev0__py3-none-any.whl

cuenca_validations/types/__init__.py

@@ -83,7 +83,6 @@ __all__ = [
     'Profession',
     'QueryParams',
     'ReferencedClabeRequest',
-    'RejectTransferOrderRequest',
     'LegalRfc',
     'Rfc',
     'QuestionnairesRequest',
@@ -114,6 +113,7 @@ __all__ = [
     'TransferOrderStatus',
     'TransferQuery',
     'TransferRequest',
+    'UpdateTransferOrderRequest',
     'UpdateTransferRequest',
     'UserCardNotification',
     'UserCredentialRequest',
@@ -280,7 +280,6 @@ from .requests import (
     PlatformRequest,
     QuestionnairesRequest,
     ReferencedClabeRequest,
-    RejectTransferOrderRequest,
     SavingRequest,
     SavingUpdateRequest,
     SessionRequest,
@@ -289,6 +288,7 @@ from .requests import (
     TransferOrderLineRequest,
     TransferOrderRequest,
     TransferRequest,
+    UpdateTransferOrderRequest,
     UpdateTransferRequest,
     UserCredentialRequest,
     UserCredentialUpdateRequest,

cuenca_validations/types/requests.py

@@ -1,5 +1,5 @@
 import datetime as dt
-from typing import Annotated, Any, Optional, Union
+from typing import Annotated, Any, Literal, Optional, Union
 
 from clabe import BANK_NAMES, Clabe
 from pydantic import (
@@ -50,6 +50,7 @@ from ..types.enums import (
     TrackDataMethod,
     TransactionStatus,
     TransactionTokenValidationStatus,
+    TransferOrderStatus,
     UserCardNotification,
     UserStatus,
     VerificationStatus,
@@ -1021,10 +1022,6 @@ class PhoneVerificationAssociationRequest(BaseRequest):
     verification_id: str
 
 
-MAX_TRANSFER_ORDER_BATCH_LINES = 5000
-MAX_TRANSFER_ORDER_EXPIRATION_HOURS = 24 * 7
-
-
 class TransferOrderLineRequest(BaseRequest):
     account_number: str
     recipient_name: str
@@ -1044,65 +1041,62 @@ class TransferOrderLineRequest(BaseRequest):
 
 
 class TransferOrderRequest(BaseRequest):
-    account_number: Optional[str] = None
-    recipient_name: Optional[str] = None
-    amount: Optional[int] = None
-    descriptor: Optional[str] = None
     idempotency_key: str
     legal_person_id: Optional[str] = None
-    items: Optional[list[TransferOrderLineRequest]] = None
-    expires_in_hours: Optional[int] = Field(
-        default=None,
-        gt=0,
-        le=MAX_TRANSFER_ORDER_EXPIRATION_HOURS,
-    )
+    items: list[TransferOrderLineRequest]
 
     model_config = ConfigDict(
         json_schema_extra={
             'example': {
-                'account_number': '646180157034181180',
-                'recipient_name': 'Doroteo Arango',
-                'amount': 100_00,
-                'descriptor': 'Mezcal, pulque y tequila',
                 'idempotency_key': 'UNIQUE-KEY-003',
                 'legal_person_id': 'USWqY5cvkISJOxHyEKjAKf8w',
+                'items': [
+                    {
+                        'account_number': '646180157034181180',
+                        'recipient_name': 'Doroteo Arango',
+                        'amount': 100_00,
+                        'descriptor': 'Mezcal, pulque y tequila',
+                    }
+                ],
             }
         },
     )
 
     @model_validator(mode='after')
-    def require_unitario_or_batch(self) -> 'TransferOrderRequest':
-        has_items = bool(self.items)
-        has_flat = all(
-            (
-                self.account_number is not None,
-                self.recipient_name is not None,
-                self.amount is not None,
-                self.descriptor is not None,
-            )
-        )
-        if has_items == has_flat:
-            raise ValueError(
-                'Provide either a single transfer (account_number, '
-                'recipient_name, amount, descriptor) or items[] for a batch, '
-                'not both'
-            )
-        if self.items is not None:
-            if len(self.items) > MAX_TRANSFER_ORDER_BATCH_LINES:
-                raise ValueError(
-                    'Batch exceeds maximum of '
-                    f'{MAX_TRANSFER_ORDER_BATCH_LINES} lines'
-                )
-            if len(self.items) == 0:
-                raise ValueError('items must not be empty')
+    def validate_items(self) -> 'TransferOrderRequest':
+        if len(self.items) == 0:
+            raise ValueError('items must not be empty')
         return self
 
 
-class RejectTransferOrderRequest(BaseRequest):
-    rejection_reason: str
+class UpdateTransferOrderRequest(BaseRequest):
+    status: Literal[
+        TransferOrderStatus.authorized, TransferOrderStatus.rejected
+    ]
+    rejection_reason: Optional[str] = None
 
     model_config = ConfigDict(
         json_schema_extra={
-            'example': {'rejection_reason': 'Monto incorrecto'},
+            'examples': [
+                {'status': 'authorized'},
+                {
+                    'status': 'rejected',
+                    'rejection_reason': 'Monto incorrecto',
+                },
+            ]
         },
     )
+
+    @model_validator(mode='after')
+    def validate_rejection_reason(self) -> 'UpdateTransferOrderRequest':
+        if self.status is TransferOrderStatus.rejected:
+            if (
+                self.rejection_reason is None
+                or not self.rejection_reason.strip()
+            ):
+                raise ValueError('rejection_reason is required')
+        elif self.rejection_reason is not None:
+            raise ValueError(
+                'rejection_reason is only allowed when status is rejected'
+            )
+        return self

cuenca_validations/version.py

@@ -1 +1 @@
-__version__ = '2.1.48.dev1'
+__version__ = '2.1.49.dev0'

{cuenca_validations-2.1.48.dev1.dist-info → cuenca_validations-2.1.49.dev0.dist-info}/METADATA

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

{cuenca_validations-2.1.48.dev1.dist-info → cuenca_validations-2.1.49.dev0.dist-info}/RECORD

@@ -4,8 +4,8 @@ cuenca_validations/errors.py,sha256=OtM8EgiKqYdz9Hn66AbBO96orL1or7efkyt0vh0Zxbs,
 cuenca_validations/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 cuenca_validations/typing.py,sha256=1QCu81IbVZZpyInjyeAuO-nF36gpT5Gi4o6V9PozuOU,204
 cuenca_validations/validators.py,sha256=zXSnU5-EMbQZBD-PfFXgP4Z6G7cm7KFDWXB7Nie6WUk,2682
-cuenca_validations/version.py,sha256=umPOhd32a7pnK7Db2pIyLe4_2kcNXGaPq1FCO0MRMS4,28
-cuenca_validations/types/__init__.py,sha256=VZu01odevifIR7SR9-lpJC2Ysee8QLwVnIOj7cjEUTU,6711
+cuenca_validations/version.py,sha256=E1OKARtICq_n0Q4ukZ9yaw-H6GS0VhkKcaXf2C4kIGg,28
+cuenca_validations/types/__init__.py,sha256=M657giDGa3sejskWMPIAfCXJHDgbh4p9J9BnheKJR-A,6711
 cuenca_validations/types/card.py,sha256=UGzz8NTFAverUmdUKAK1oGHnOnjSNTpIRUm93vKSSGY,1295
 cuenca_validations/types/enums.py,sha256=If8N4-AbM9daHmnH6txmBb9AqN0Ey7gi3jmkOZOIDAU,21104
 cuenca_validations/types/files.py,sha256=2CszbwF9ytXV9suFFwyDjYG4XxY8UhCjRw3HttVXXNw,269
@@ -14,17 +14,17 @@ cuenca_validations/types/helpers.py,sha256=4veeLZbugHHqZk0ezSim978VhH6Vq8XTrEhe1
 cuenca_validations/types/identities.py,sha256=j2xxh3UYHJe6IbAwr9yNXJkebTth_-g3SUmHeiPez8M,5513
 cuenca_validations/types/morals.py,sha256=davabh5hAnFVQyM7-yCyDaT5ewXnm0cr1BtqDIwzkX8,1833
 cuenca_validations/types/queries.py,sha256=yJZGW3p6uWGaBm6mjDp7KBgg7oLh0CUlAJeeVLRzowk,6134
-cuenca_validations/types/requests.py,sha256=6c9EUQMLCThX9RcU8VFpAY0Yuyc6erZpTbjosEmbD2s,32569
-cuenca_validations-2.1.48.dev1.dist-info/licenses/LICENSE,sha256=wR76FmxBbfnQpwELkkE5iMF8sFIafEMgXLTE4N4WPTc,1063
+cuenca_validations/types/requests.py,sha256=26YyfGHh1egdjAHUWTDjTtelXARWKNPB6npPHkW1Tcg,32336
+cuenca_validations-2.1.49.dev0.dist-info/licenses/LICENSE,sha256=wR76FmxBbfnQpwELkkE5iMF8sFIafEMgXLTE4N4WPTc,1063
 tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tests/test_card.py,sha256=QAfRz7e11gWICPnFJZ2tiYgUsFV3C9TwzJXrDnDNXFw,1202
 tests/test_errors.py,sha256=ixiIgEuBuzfsL5p4uCFdF32XqFRtTPF6EVhGJ0keOrI,930
 tests/test_helpers.py,sha256=fx2R7y6oQO8XeQzevJe826NuHkBgvtznZicomchB3Gg,899
-tests/test_requests.py,sha256=mjH-VbU_jCdgjQPAalTcJo76iL6tSBnmyengiAY-M20,8671
+tests/test_requests.py,sha256=NlXAvKDLyVy0tYDUHy3oNkxeX1KcA-dfyqwftddHWBo,10627
 tests/test_statement.py,sha256=IOE0rRRBgBZSJv_FLaETEyn5NzzXKMNTqgjv99GX-68,1436
 tests/test_types.py,sha256=rPjqPGfPU73MSdbeJpoW_ai5TAO25hWSnQAt_H7ZIeI,23055
 tests/test_validators.py,sha256=Jjr9gWTT4cRntGiKvQK4fncqx3JkEuTWkKm1VqpRHTs,1829
-cuenca_validations-2.1.48.dev1.dist-info/METADATA,sha256=Dv9rhKncVEqcSZk1z8OPXIJDU9reQEh76kaAHBHMe_U,1600
-cuenca_validations-2.1.48.dev1.dist-info/WHEEL,sha256=YVMoNqKzERt-wjUZwJ33xBGAwnFl-4cqbYkTtWa4itE,91
-cuenca_validations-2.1.48.dev1.dist-info/top_level.txt,sha256=4233xdOs2HtuT-GFRjcDcwK0IwdwvWdczOtk0fPB6Gw,25
-cuenca_validations-2.1.48.dev1.dist-info/RECORD,,
+cuenca_validations-2.1.49.dev0.dist-info/METADATA,sha256=7CzNDGl4cHPMNcq-p0u36AMzFYj2tFFmM34sc-pfn_E,1600
+cuenca_validations-2.1.49.dev0.dist-info/WHEEL,sha256=YVMoNqKzERt-wjUZwJ33xBGAwnFl-4cqbYkTtWa4itE,91
+cuenca_validations-2.1.49.dev0.dist-info/top_level.txt,sha256=4233xdOs2HtuT-GFRjcDcwK0IwdwvWdczOtk0fPB6Gw,25
+cuenca_validations-2.1.49.dev0.dist-info/RECORD,,

tests/test_requests.py

@@ -5,17 +5,18 @@ from pydantic_extra_types.phone_numbers import PhoneNumber
 from cuenca_validations.types.enums import (
     Country,
     OperatorRole,
+    TransferOrderStatus,
     VerificationType,
 )
 from cuenca_validations.types.queries import OperatorQuery
 from cuenca_validations.types.requests import (
-    MAX_TRANSFER_ORDER_BATCH_LINES,
     OperatorLoginRequest,
     OperatorLoginUpdateRequest,
     OperatorRequest,
     OperatorUpdateRequest,
     PasswordResetRequest,
     TransferOrderRequest,
+    UpdateTransferOrderRequest,
     UpdateTransferRequest,
     UserTOSAgreementRequest,
     UserUpdateRequest,
@@ -244,26 +245,73 @@ def test_transfer_order_request_shape() -> None:
         'descriptor': 'Mezcal',
     }
     single = TransferOrderRequest.model_validate(
-        {**line, 'idempotency_key': 'single'}
+        {'idempotency_key': 'single', 'items': [line]}
     )
-    assert single.items is None
+    assert single.items is not None
+    assert len(single.items) == 1
     batch = TransferOrderRequest.model_validate(
-        {'idempotency_key': 'batch', 'items': [line]}
+        {'idempotency_key': 'batch', 'items': [line, line]}
     )
     assert batch.items is not None
+    assert len(batch.items) == 2
 
-    with pytest.raises(ValidationError, match='not both'):
-        TransferOrderRequest.model_validate(
-            {**line, 'idempotency_key': 'both', 'items': [line]}
-        )
     with pytest.raises(ValidationError, match='items must not be empty'):
         TransferOrderRequest.model_validate(
-            {**line, 'idempotency_key': 'empty', 'items': []}
+            {'idempotency_key': 'empty', 'items': []}
         )
-    with pytest.raises(ValidationError, match='Batch exceeds maximum'):
-        TransferOrderRequest.model_validate(
-            {
-                'idempotency_key': 'too-many',
-                'items': [line] * (MAX_TRANSFER_ORDER_BATCH_LINES + 1),
-            }
+    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'
+
+
+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': ''}
+        )
+    with pytest.raises(ValidationError, match='rejection_reason is required'):
+        UpdateTransferOrderRequest.model_validate(
+            {'status': 'rejected', 'rejection_reason': '   '}
         )
+
+
+def test_update_transfer_order_request_authorized_forbids_reason() -> None:
+    with pytest.raises(
+        ValidationError,
+        match='rejection_reason is only allowed when status is rejected',
+    ):
+        UpdateTransferOrderRequest.model_validate(
+            {'status': 'authorized', 'rejection_reason': 'no'}
+        )
+
+
+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)