[IMP] l10n_latam_check_ux: Added tests with instructions#835
[IMP] l10n_latam_check_ux: Added tests with instructions#835feg-adhoc wants to merge 1 commit intoingadhoc:18.0from
Conversation
|
|
There was a problem hiding this comment.
Pull Request Overview
Este PR agrega tests para validar la funcionalidad de 'Split Payment' al transferir cheques de terceros en el módulo l10n_latam_check_ux. Los tests verifican que cuando se habilita la opción split_payment, se cree un pago individual por cada cheque en lugar de agruparlos en un único pago.
- Implementa un test completo que crea el entorno necesario (diarios, métodos de pago, cheques)
- Valida que la funcionalidad de transferencia dividida funciona correctamente para múltiples cheques
- Sigue las convenciones de testing de Odoo utilizando
TransactionCasey métodos@classmethodpara setup
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| l10n_latam_check_ux/tests/test_third_party_check_transfer.py | Agrega nueva clase de test TestThirdPartyCheckTransfer con métodos auxiliares para crear cheques de terceros y un test principal que valida la funcionalidad de split payment al transferir múltiples cheques |
| l10n_latam_check_ux/tests/init.py | Importa el nuevo módulo de tests para que sea ejecutado por el framework de testing de Odoo |
| def setUpClass(cls): | ||
| super().setUpClass() | ||
|
|
||
| # Obtener la compañía principal |
There was a problem hiding this comment.
Comentario en español mientras que el resto de los comentarios del archivo están en inglés. Se recomienda mantener consistencia en el idioma. Si bien las reglas no exigen traducción, es mejor que todos los comentarios del archivo estén en el mismo idioma para mantener la consistencia.
| self.assertNotEqual(payment_1.id, payment_2.id, "Payments 1 and 2 must have different IDs") | ||
| self.assertNotEqual(payment_1.id, payment_3.id, "Payments 1 and 3 must have different IDs") | ||
| self.assertNotEqual(payment_2.id, payment_3.id, "Payments 2 and 3 must have different IDs") |
There was a problem hiding this comment.
[nitpick] Las validaciones de unicidad de IDs en las líneas 226-228 son redundantes. Si ya has verificado que len(payment_1) == 1, len(payment_2) == 1, y len(payment_3) == 1, y que cada uno tiene un monto diferente (1000, 2000, 3000), entonces necesariamente son registros distintos. Estas aserciones adicionales no agregan valor y podrían eliminarse para simplificar el código.
| self.assertNotEqual(payment_1.id, payment_2.id, "Payments 1 and 2 must have different IDs") | |
| self.assertNotEqual(payment_1.id, payment_3.id, "Payments 1 and 3 must have different IDs") | |
| self.assertNotEqual(payment_2.id, payment_3.id, "Payments 2 and 3 must have different IDs") |
feg-adhoc
force-pushed
the
18.0-test_l10n_latam_check_ux-feg
branch
from
e2064b5 to
01a2187
Compare
No description provided.