search

indexa-git / l10n-dominicana

Repositorio con las funcionalidades básicas para la emisión de facturas fiscales dominicanas en Odoo.
GNU Lesser General Public License v3.0
98 stars 121 forks source link

[17.0][TEST] l10n_do_accounting: pruebas funcionales #1156

Closed dixgrake closed 2 months ago

dixgrake commented 3 months ago

Type: General

Describe the bug

Odoo Version: 17.0

[A clear and concise description of the bug]

Este caso de prueba tiene como objetivo verificar el correcto funcionamiento de este módulo e identificar posibles errores.

Se llevaron a cabo las siguientes pruebas:

  1. Instalación de modulo.
  2. Activar la gestión de comprobante en el diario.
  3. Creación de cliente y configuración de tipo de contribuyente.
  4. Creación de Factura de Proveedor: Emitir factura con cada uno de los tipos de comprobantes fiscales.
  5. Creación de Factura de Cliente: Emitir factura con cada uno de los tipos de comprobantes fiscales.

Branch utilizado: https://github.com/indexa-git/l10n-dominicana/tree/17.0-mig-001-jl

Steps to Reproduce Steps to reproduce the behavior:

  1. Instalación del modulo: l10n_do_accounting

Fallo: Al instalar el modulo tuvimos un error que nos indicaba que no existía este dato de esta columna "name@es".

Éxito: Eliminé esta columna para verificar si así solucionábamos el problema que teníamos. Al quitarla, pudimos instalar el módulo.

Causa: 1idsequencecodename@esnamereport_nameinternal_typel10n_do_ncf_typedoc_code_prefixcountry_id/idis_vat_requiredactive

  1. Activar la gestión de comprobantes fiscales en el diario.

Una vez que logramos instalar el módulo, procedimos a activar el diario fiscal. Previamente, configuramos los datos de la empresa para continuar con el proceso.

Fallo: Después de activar el diario fiscal, no se cargaron los datos de los tipos de comprobantes.

Éxito: Nos dimos cuenta de que en el manifiesto los datos estaban comentado. Procedimos a descomentarlo y, una vez actualizado, nos cargó los tipos de comprobante en el diario.

Causa: https://github.com/indexa-git/l10n-dominicana/blob/f5b6298f31e73eac27ae2c728e158642d2864f0e/l10n_do_accounting/__manifest__.py#L19

  1. Creación de contacto y configuración de tipo de contribuyente.

Realizada la instalación del modulo y activación del diario fiscal correctamente, procedimos a crear un contacto.

Fallo: Al crear un contacto nos presento un error inmediatamente.

Error: https://pastebin.com/5FtRqGBg (Comprobantes Electrónicos)

Posible Causa: https://github.com/indexa-git/l10n-dominicana/blob/f5b6298f31e73eac27ae2c728e158642d2864f0e/l10n_do_accounting/models/res_partner.py#L105

  1. Creación de factura de proveedor:

Nos permite crear un contacto (proveedor/cliente) desde de la factura y poder configurar tipo de contribuyente y tipo de gasto/costo.

Fallos:

Error: https://pastebin.com/jzPZ92X3

Éxito: se crearon y generaron de manera correcta cada uno de los tipos de comprobantes fiscales:

image

  1. Creación de factura de cliente:

Fallos:

Error: https://pastebin.com/jzPZ92X3

Éxito: se crearon y generaron de manera correcta cada uno de los tipos de comprobantes fiscales excepto las electrónica:

image

Expected behavior

Additional context and screenshots

  1. Factura proveedor: problema en la secuencia interna

image

Sample Code / Stack Trace

Configuration information Operating System: dockerdoo Python version:

Mentions

@akevinieron @Rikylora30 @jlopezg

jlopezg commented 3 months ago

Hola @dixgrake

A continuación te respondo cada problema encontrado en tus pruebas:

Fallo: Al instalar el modulo tuvimos un error que nos indicaba que no existía este dato de esta columna "name@es". Resuelto ✅

Después de activar el diario fiscal, no se cargaron los datos de los tipos de comprobantes. Resuelto ✅

Fallo: Al crear un contacto nos presento un error inmediatamente. Resuelto ✅

Factura Proveedor

la factura fiscal no esta asignando la secuencia interna correspondiente, en su lugar se coloca "/". esto ocurre solamente en las factura que tienen el tipo de comprobante B01. Pendiente ❌

las factura con los tipos de comprobante electrónico nos presenta un error al intentar agregar un producto. Resuelto ✅

Te permite generar múltiples notas de crédito/débito para una misma factura, incluso después de haber realizado un reembolso completo en la primera nota de crédito/débito. Este comportamiento es nativo, viene desde versiones anteriores, y por el momento no contemplamos cambiarla.

Factura Cliente

La factura con los tipos de comprobante electrónico presenta un error al intentar agregar un producto, el mismo error que ocurre en la facturación de proveedor. Resuelto ✅

Te permite generar múltiples notas de crédito/débito para una misma factura, incluso después de haber realizado un reembolso completo en la primera nota de crédito/débito. Este comportamiento es nativo, viene desde versiones anteriores, y por el momento no contemplamos cambiarla.

dixgrake commented 3 months ago

@jlopezg Muchisimas gracias.

Repetimos todas las pruebas y la única que no pasó fue la creación de contacto, la cual sigue fallando pero con un error diferente.

https://github.com/indexa-git/l10n-dominicana/blob/703541250bd3e7da74b0074fe13759c27bd6fed3/l10n_do_accounting/models/res_partner.py#L113

Error: https://pastebin.com/rT6RpGUd

dixgrake commented 2 months ago

@jlopezg

Se repitieron las pruebas y pasaron. Hicimos una prueba con la impresión de factura, la cual no pasó.

Detalle del error: https://pastebin.com/zHDz0LyH

jlopezg commented 2 months ago

Recibido. @dixgrake Ya casi liberamos el branch.

jlopezg commented 2 months ago

Hola @dixgrake Ya hicimos merge en 17.0. Este es el momento de probar todo lo que podamos 💪🏻

dixgrake commented 2 months ago

De acuerdo, muchas gracias @jlopezg 💪🏻

dixgrake commented 2 months ago

Hola @jlopezg

Se repitieron todas las pruebas y pasaron correctamente. Creo que podemos dar por cerrado este issue. Si hay algo más que debamos tener en cuenta, por favor, háznoslo saber.

jlopezg commented 2 months ago

Hola @dixgrake Muchas gracias!