[11.0][MIG] l10n_es_aeat_mod130

[victormmtorres]

No description provided.

[oca-clabot]

Hey @victormmtorres, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/cla
Here is a list of the users:

• Launchpad Translations on behalf of openerp-spain-team (no github login found)
• agaldona ainaragaldona@avanzosc.es (no github login found)

Appreciation of efforts,
OCA CLAbot

[victormmtorres]

cc @Tecnativa

[pedrobaeza]

Está ya listo para revisar?

[victormmtorres]

Creo que ya esta bastante avanzado y se puede revisar, pero si veo que aun no pasa Travis ni Runbot por lo que ire machacando los errores

[pedrobaeza]

OK, me espero a que avises que está ya en verde.

[victormmtorres]

Travis is green but not Runbot.

Help wanted with Runbot, no Warning or Errors inside Full logs

[victormmtorres]

Fails every time try force rebuild runbot

[pedrobaeza]

@victormmtorres podrías añadir tests al módulo? Echa un vistazo al del 111 para inspirarte (https://github.com/OCA/l10n-spain/blob/11.0/l10n_es_aeat_mod111/tests/test_l10n_es_aeat_mod111.py):

• Heredar del TestL10nEsAeatModBase que ya crea el plan contable y trae herramientas para rápida creación de facturas.
• Crear alguna factura de compra y de venta declarando el diccionario de impuestos taxes_purchase y el de taxes_sale que no ves en el 111. Pon los impuestos normales SIVA21B y PIVA21B como ejemplo (aquí no es lo más importante, ya que se miden ingresos y gastos).
• Crea el modelo.
• Ejecuta el cálculo.
• Comprueba los valores de lo que se debería haber calculado..
• Puedes hacer variantes de marcar la casilla de deducción para ver cómo varía el cálculo.
• Copia de aquí para "testear" la exportación a BOE:

  l10n-spain/l10n_es_aeat_mod303/tests/test_l10n_es_aeat_mod303.py

  Lines 348 to 361 in f5b2f14

  	# Export to BOE
  	export_to_boe = self.env['l10n.es.aeat.report.export_to_boe'].create({
  	'name': 'test_export_to_boe.txt',
  	})
  	export_config_xml_ids = [
  	'l10n_es_aeat_mod303.aeat_mod303_main_export_config',
  	'l10n_es_aeat_mod303.aeat_mod303_2017_main_export_config',
  	'l10n_es_aeat_mod303.aeat_mod303_2018_main_export_config',
  	]
  	for xml_id in export_config_xml_ids:
  	export_config = self.env.ref(xml_id)
  	self.assertTrue(
  	export_to_boe._export_config(self.model303, export_config)
  	)

  (lo pongo entre comillas porque lo que estamos probando es que no falla únicamente).

[victormmtorres]

Changes done @pedrobaeza

[victormmtorres]

@pedrobaeza

[victormmtorres]

@pedrobaeza please review

[pedrobaeza]

Con los últimos cambios realizados debería estar todo correcto, aunque mantengo sin fusionar hasta confirmación del cliente