[WIP] Basic migration OVL 7.0 to 8.0

[suniagajose]

Buen dia @hbto y @moylop260

Como saben este PR fue creado para hacer una migracion parcial de la OVL, pero he llegado al punto donde tengo ciertas dudas funcionales para poder continuar. A continuacion mostrare una lista de los detalles restante para los que requiero su colaboracion:

build Travis CI: https://travis-ci.org/Vauxoo/odoo-venezuela/builds/91320703

Incidencias

Fiscal Requirements Archivo de TEST: fr_vat_test.yml

• El 5.4 para comprobacion del constraint VAT Mandatory, en mi instancia local el test pasa sin problemas, pero en el travis esta fallando. SOLUCIONADO [17/11/15]

Withholding ISRL Archivo de TEST: supplier_wh_islr_invoice.yml

• Falla en 1.1: El test fallaba porque se habia comentado una data demo que causaba un error por duplicidad de xml_id en pylint. Se reestablecio esta data demo para evitar fallo del test, pero persiste la duplicidad de xml_id.
• Falla en 1.2, 1.5. El test espera la creacion de un objeto del modelo 'islr.wh.doc' el cual nunca se crea y causa esta falla.

Archivo de TEST: supplier_wh_islr_doc.yml

• Falla en 2.2 y 2.3. Al igual que el caso anterior tampoco se genera el objeto del modelo 'islr.wh.doc' al confirmar la factura lo que causa el siguiente error Assertion "The invoice was not confirmed or the withholding document was created wrong way."

Withholding SRC y Withholding Muni

• Algunos test fallaban porque hacian referencia a la data demo, la cual no estaba incluida en manifest file. Por lo tanto se incluyo y se corrigieron algunos detalles de migracion para que los test pasaran. Sobre esto quisiera saber si habia algun motivo por lo que no estaban incluida la data demo en el manifest?

Withholding IVA

• Los test para este modulo se encontraron comentados en el manifest file y se dejaron asi. Deberiamos habilitar estos test para incrementar el coverage?

[hbto]

pendiente por ser respondido

Quien suscribe,

Hbto

2015-11-16 2:41 GMT-04:30 suniagajose notifications@github.com:

Buen dia @hbto https://github.com/hbto y @moylop260 https://github.com/moylop260

Como saben este PR fue creado para hacer una migracion parcial de la OVL, pero he llegado al punto donde tengo ciertas dudas funcionales para poder continuar. A continuacion mostrare una lista de los detalles restante para los que requiero su colaboracion:

build Travis CI: https://travis-ci.org/Vauxoo/odoo-venezuela/builds/91320703 Incidencias

Fiscal Requirements Archivo de TEST: fr_vat_test.yml https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_fiscal_requirements/test/fr_vat_test.yml

• El 5.4 https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_fiscal_requirements/test/fr_vat_test.yml#L69 para comprobacion del constraint VAT Mandatory, en mi instancia local el test pasa sin problemas, pero en el travis esta fallando

Withholding ISRL Archivo de TEST: supplier_wh_islr_invoice.yml https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_invoice.yml

• Falla en 1.1 https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_invoice.yml#L36: El test fallaba porque se habia comentado una data demo https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/demo/l10n_ve_islr_withholding_demo.xml#L32-L193 que causaba un error por duplicidad de xml_id en pylint. Se reestablecio esta data demo para evitar fallo del test, pero persiste la duplicidad de xml_id.
• Falla en 1.2 https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_invoice.yml#L91-L94, 1.5 https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_invoice.yml#L186-L191. El test espera la creacion de un objeto del modelo 'islr.wh.doc' el cual nunca se crea y causa esta falla.

Archivo de TEST: supplier_wh_islr_doc.yml https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_doc.yml

• Falla en 2.2 https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_doc.yml#L76-L78 y 2.3 https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_doc.yml#L120-L126. Al igual que el caso anterior tampoco se genera el objeto del modelo 'islr.wh.doc' al confirmar la factura lo que causa el siguiente error Assertion "The invoice was not confirmed or the withholding document was created wrong way."

Withholding SRC y Withholding Muni

• Algunos test fallaban porque hacian referencia a la data demo, la cual no estaba incluida en manifest file. Por lo tanto se incluyo y se corrigieron algunos detalles de migracion para que los test pasaran. Sobre esto quisiera saber si habia algun motivo por lo que no estaban incluida la data demo en el manifest?

Withholding IVA

• Los test para este modulo se encontraron comentados en el manifest file y se dejaron asi. Deberiamos habilitar estos test para incrementar el coverage?

— Reply to this email directly or view it on GitHub https://github.com/Vauxoo/odoo-venezuela/pull/33#issuecomment-156943597.

[suniagajose]

Retomando este PR, fueron resuelto los errores en los test de fiscal_requirements, asi como tambien el error de duplicidad en la data de withholding islr, para lo cual se tuvieron que modificar algunos test. Ambos casos estan descritos en mi post anterior

A la fecha de hoy quedan estos detalle pendiente.

Withholding ISRL Archivo de TEST: supplier_wh_islr_invoice.yml

• Falla en 1.2, 1.5. El test espera la creacion de un objeto del modelo 'islr.wh.doc' el cual nunca se crea y causa esta falla.

Archivo de TEST: supplier_wh_islr_doc.yml

• Falla en 2.2 y 2.3. Al igual que el caso anterior tampoco se genera el objeto del modelo 'islr.wh.doc' al confirmar la factura lo que causa el siguiente error Assertion "The invoice was not confirmed or the withholding document was created wrong way."

Hice una revision tecnica del codigo y el metodo que crea este objeto les disparado por un action server, que a su vez espera ser ejecutado por el workflow de account.invoice, al cambiar el state a 'open'. Funcionalmente se realizo esta prueba y no se pudo generar el documento.

Cualquier comentario tecnico o funcional al respecto seria de gran ayuda @moylop260 y @hbto

Gracias

Enlace travis del error

[hbto]

Hola José

Te he reenviado el correo con el asunto:

Sabias que... Las acciones de servidor son ignoradas desde unit tests, a menos que se corra la acción explícitamente desde el test. Espero que allí consigas la información que requieres para poder superar este issue

Quien suscribe,

Hbto

2015-11-25 21:36 GMT-04:30 suniagajose notifications@github.com:

Retomando este PR, fueron resuelto los errores en los test de fiscal_requirements, asi como tambien el error de duplicidad en la data de withholding islr, para lo cual se tuvieron que modificar algunos test. Ambos casos estan descritos en mi post anterior

A la fecha de hoy quedan estos detalle pendiente.

Withholding ISRL Archivo de TEST: supplier_wh_islr_invoice.yml https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_invoice.yml

• Falla en 1.2 https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_invoice.yml#L91-L94, 1.5 https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_invoice.yml#L186-L191. El test espera la creacion de un objeto del modelo 'islr.wh.doc' el cual nunca se crea y causa esta falla.

Archivo de TEST: supplier_wh_islr_doc.yml https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_doc.yml

• Falla en 2.2 https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_doc.yml#L76-L78 y 2.3 https://github.com/vauxoo-dev/odoo-venezuela/blob/8.0-FIX-duplicated-dev-suniagajose/l10n_ve_withholding_islr/test/supplier_wh_islr_doc.yml#L120-L126. Al igual que el caso anterior tampoco se genera el objeto del modelo 'islr.wh.doc' al confirmar la factura lo que causa el siguiente error Assertion "The invoice was not confirmed or the withholding document was created wrong way."

Hice una revision tecnica del codigo y el metodo que crea este objeto les disparado por un action server https://github.com/Vauxoo/odoo-venezuela/blob/8.0/l10n_ve_withholding_islr/workflow/wh_action_server.xml, que a su vez espera ser ejecutado por el workflow https://github.com/Vauxoo/odoo-venezuela/blob/8.0/l10n_ve_withholding/workflow/wh_action_server.xml#L14-L15 de account.invoice, al cambiar el state a 'open'. Funcionalmente se realizo esta prueba y no se pudo generar el documento.

Cualquier comentario tecnico o funcional al respecto seria de gran ayuda @moylop260 https://github.com/moylop260 y @hbto https://github.com/hbto

Gracias

Enlace travis del error https://travis-ci.org/Vauxoo/odoo-venezuela/jobs/93278011#L759

— Reply to this email directly or view it on GitHub https://github.com/Vauxoo/odoo-venezuela/pull/33#issuecomment-159781524.

[JulioSerna]

@hbto @suniagajose

Eso puede resolver agregando la dependencia "base_action_rule"

[hbto]

Excelente gracias Julio.

Quien suscribe,

Hbto

On Wed, Nov 25, 2015 at 10:48 PM, Julio [Vauxoo] notifications@github.com wrote:

@hbto https://github.com/hbto @suniagajose https://github.com/suniagajose

Eso puede resolver agregando la dependencia "base_action_rule"

— Reply to this email directly or view it on GitHub https://github.com/Vauxoo/odoo-venezuela/pull/33#issuecomment-159792107.

[suniagajose]

Gracias @julio y @hbto El nov 25, 2015 9:20 PM, "Hbto [ Vauxoo ]" notifications@github.com escribió:

Excelente gracias Julio.

Quien suscribe,

Hbto

On Wed, Nov 25, 2015 at 10:48 PM, Julio [Vauxoo] <notifications@github.com

wrote:

@hbto https://github.com/hbto @suniagajose https://github.com/suniagajose

Eso puede resolver agregando la dependencia "base_action_rule"

— Reply to this email directly or view it on GitHub <https://github.com/Vauxoo/odoo-venezuela/pull/33#issuecomment-159792107 .

— Reply to this email directly or view it on GitHub https://github.com/Vauxoo/odoo-venezuela/pull/33#issuecomment-159792298.

[moylop260]

@suniagajose Could you rebase it?

@hbto Could we merge it?

[suniagajose]

@moylop260 rebase ready!

[moylop260]

@suniagajose Could you fix the warning of runbot, please?

2015-12-17 18:21:37,605 89 WARNING openerp_test openerp.modules.module: Found deprecated fast_suite or checks attribute in test module openerp.addons.l10n_ve_withholding_islr.tests. These have no effect in or after version 8.0.

[suniagajose]

How i fix this kind of warnings??? @moylop260

WARNING openerp_test openerp.modules.module: Found deprecated fast_suite or checks attribute in test module openerp.addons.l10n_ve_withholding_islr.tests. These have no effect in or after version 8.0.