search

OCA / l10n-spain

Odoo Spain Localization
https://www.aeodoo.org/estado-localizacion
GNU Affero General Public License v3.0
292 stars 520 forks source link

[17.0][FIX] l10n_es_account_statement_import_n43: Tests compatible with base_bank_from_iban #3585

Closed manuelregidor closed 6 months ago

manuelregidor commented 6 months ago

Before this fix, tests were not compatible with module base_bank_from_iban as it was using bank accounts with a wrong IBAN format.

This fix adds bannk accounts with righthly formatted IBANs so tests are compatible.

OCA-git-bot commented 6 months ago

Hi @pedrobaeza, some modules you are maintaining are being modified, check this out!

pedrobaeza commented 6 months ago

Pero no entiendo por qué aquí no falla si en el entorno está base_bank_from_iban. ¿Acaso en vuestro CI no replicais la misma forma de pasar los tests que aquí?

HaraldPanten commented 6 months ago

Pero no entiendo por qué aquí no falla si en el entorno está base_bank_from_iban. ¿Acaso en vuestro CI no replicais la misma forma de pasar los tests que aquí?

No te acabo de entender, perdona.

Este FIX viene por este warning en el formato del IBAN --> https://github.com/OCA/l10n-spain/pull/3583

ValentinVinagre commented 6 months ago

@manuelregidor cierra este FIX please 👍🏻