[13.0][FIX] l10n_ch_base_bank

OCA / l10n-switzerland

Odoo Swiss localization

GNU Affero General Public License v3.0

53 stars 164 forks source link

[13.0][FIX] l10n_ch_base_bank #603

Closed TDu closed 3 years ago

TDu commented 3 years ago

On an unrelated PR the test for l10n_ch_base_bank were broken. It seems to be related to this new change introduced by

https://github.com/odoo/odoo/pull/67599

Which test the valid of the type journal.

So this fixes the test, but I am not sure if they are still meaningfull.

TDu commented 3 years ago

Why is the fix in test_search_invoice different than in test_create_invoice ?

Well both test file started different. One created a res.partner.bank and the other not. I have change them so they prepare the environment in the same way.

TDu commented 3 years ago

@yvaucher Do you want to have another look at this ? It could probably fix master that is red.