cuenca-validations 2.1.49.dev0__py3-none-any.whl → 2.1.50.dev0__py3-none-any.whl

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/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/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/version.py

@@ -1 +1 @@
-__version__ = '2.1.49.dev0'
+__version__ = '2.1.50.dev0'

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

@@ -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.dist-info → cuenca_validations-2.1.50.dev0.dist-info}/RECORD

@@ -4,27 +4,27 @@ 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=E1OKARtICq_n0Q4ukZ9yaw-H6GS0VhkKcaXf2C4kIGg,28
-cuenca_validations/types/__init__.py,sha256=M657giDGa3sejskWMPIAfCXJHDgbh4p9J9BnheKJR-A,6711
+cuenca_validations/version.py,sha256=x45bT_ftEGYO5jfeEfTF7N5RSQsa2FHic_eeg9pqCQg,28
+cuenca_validations/types/__init__.py,sha256=YCuHWRRCr1Ilmf8VVAr16Jj4e6WYdshDSMQuugd-cXQ,6775
 cuenca_validations/types/card.py,sha256=UGzz8NTFAverUmdUKAK1oGHnOnjSNTpIRUm93vKSSGY,1295
-cuenca_validations/types/enums.py,sha256=If8N4-AbM9daHmnH6txmBb9AqN0Ey7gi3jmkOZOIDAU,21104
+cuenca_validations/types/enums.py,sha256=dR24uPDVh_uD4jehYf_7GaEKXHxypWNNuNTfj8kj3B4,21206
 cuenca_validations/types/files.py,sha256=2CszbwF9ytXV9suFFwyDjYG4XxY8UhCjRw3HttVXXNw,269
 cuenca_validations/types/general.py,sha256=eYFYwyx_a4_J480GYpqW3DFbZabDFcUjvLRMQbShIUc,5622
 cuenca_validations/types/helpers.py,sha256=4veeLZbugHHqZk0ezSim978VhH6Vq8XTrEhe1eE_r3A,1531
 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=26YyfGHh1egdjAHUWTDjTtelXARWKNPB6npPHkW1Tcg,32336
-cuenca_validations-2.1.49.dev0.dist-info/licenses/LICENSE,sha256=wR76FmxBbfnQpwELkkE5iMF8sFIafEMgXLTE4N4WPTc,1063
+cuenca_validations/types/requests.py,sha256=GD9jRpVOvjVNbXWd-TL4D3gsWEuAuGNAnfOK1Mm1-EE,32264
+cuenca_validations-2.1.50.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=NlXAvKDLyVy0tYDUHy3oNkxeX1KcA-dfyqwftddHWBo,10627
+tests/test_requests.py,sha256=Av8P2npaRi49LCjCbCMFcT0wlgOtaM28V949sJKpJ60,10370
 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.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,,
+cuenca_validations-2.1.50.dev0.dist-info/METADATA,sha256=ZBDd1sB91hxxnfHTc7MGgQiGAtucTPUD164lFAqivi4,1600
+cuenca_validations-2.1.50.dev0.dist-info/WHEEL,sha256=YVMoNqKzERt-wjUZwJ33xBGAwnFl-4cqbYkTtWa4itE,91
+cuenca_validations-2.1.50.dev0.dist-info/top_level.txt,sha256=4233xdOs2HtuT-GFRjcDcwK0IwdwvWdczOtk0fPB6Gw,25
+cuenca_validations-2.1.50.dev0.dist-info/RECORD,,

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