Skip to content

feat: getTotalDiscounts() and tests#1913

Merged
NeoRazorX merged 1 commit intoNeoRazorX:masterfrom
abdedarghal111:tarea4518
Mar 24, 2026
Merged

feat: getTotalDiscounts() and tests#1913
NeoRazorX merged 1 commit intoNeoRazorX:masterfrom
abdedarghal111:tarea4518

Conversation

@abdedarghal111
Copy link
Contributor

@abdedarghal111 abdedarghal111 commented Mar 24, 2026

Descripción

https://facturascripts.com/roadmap/4518

  • Añadido el método que devuelve los descuentos totales de todo el documento (independientemente de si son descuentos de lineas o descuentos del documento)
  • Añadido también el redondeo a ese método (redondeo por defecto) para evitar los errores de coma flotante y que se pueda usar de manera apropiada.
  • Añadidos los tests que prueban todos los casos (descuentos en lineas, descuentos en documento, sin descuento y un mix de ambos)

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

@NeoRazorX NeoRazorX merged commit a711d9a into NeoRazorX:master Mar 24, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants