[16.0][FIX] l10n_br_account_payment_order: Teste de Alteração da Data de Vencimento, erro no CI

[mbcosta]

Test Due Date.

Teste de Alteração da Data de Vencimento passou a retornar erro agora a noite no PR #3599 , o LOG pode ser visto em

https://github.com/OCA/l10n-brazil/actions/runs/13021744581/job/36323658172?pr=3599#step:9:2865

2025-01-29 00:05:30,190 407 ERROR odoo odoo.addons.l10n_br_account_payment_order.tests.test_payment_order_change: ERROR: TestPaymentOrderChange.test_change_date_maturity_multiple
Traceback (most recent call last):
  File "/__w/l10n-brazil/l10n-brazil/l10n_br_account_payment_order/tests/test_payment_order_change.py", line 65, in test_change_date_maturity_multiple
    self._send_and_check_new_cnab_code(
  File "/__w/l10n-brazil/l10n-brazil/l10n_br_account_payment_order/tests/test_base_class.py", line 136, in _send_and_check_new_cnab_code
    self._send_new_cnab_code(aml_to_change, code_to_send, warning_error)
  File "/__w/l10n-brazil/l10n-brazil/l10n_br_account_payment_order/tests/test_base_class.py", line 126, in _send_new_cnab_code
    change_wizard.doit()
  File "/__w/l10n-brazil/l10n-brazil/l10n_br_account_payment_order/wizards/account_move_line_change.py", line 63, in doit
    self.account_move_line_ids._identify_cnab_change(
  File "/__w/l10n-brazil/l10n-brazil/l10n_br_account_payment_order/models/l10n_br_cnab_change_methods.py", line 33, in _identify_cnab_change
    cnab_code = record._get_cnab_date_maturity(new_date)
  File "/__w/l10n-brazil/l10n-brazil/l10n_br_account_payment_order/models/l10n_br_cnab_change_methods.py", line 206, in _get_cnab_date_maturity
    raise UserError(
odoo.exceptions.UserError: New Date Maturity 2025-02-28 is equal to actual Date Maturity 2025-02-28

Não parece ser algum problema do método em si apenas do teste, talvez por uma questão de calendário ( alguma relação com o mês de Fevereiro? ) porque a solução até simples foi alterar a quantidade de dias somados de 30 para 40, pelos testes locais isso deve corrigir.

cc @OCA/local-brazil-maintainers

[antoniospneto]

Que coisa esquisita esse erro, tbm não consegui entender o motivo

[antoniospneto]

@mbcosta eu não debuguei o código, apenas dei uma lida, mas se você for ver ali nos testes tá sendo passado a lista inteira das linhas financeiras: self.financial_move_line_ids, e o termo de pagamento dessa fatura é o account_payment_term_advance 30% a vista e o resto é pra 30 dias, ou seja a segunda parcela vai ser a data de hoje + 30 dias, e a data que tá sendo pra ser alterada é hoje + 30 dias, ai realmente não é pra fechar..

O que eu não entendo é o porque que antes tava passando os testes, será que por está passando uma lista, por algum motivo antes o teste tava analisando só a primeira? e agora por aleatoridade passou a verificar a segunda linha?

[antoniospneto]

Comentei ali antes, mas de qualquer forma eu acho que essa é uma ótima solução, a alteração da data não poderia ser hoje + 30 dias, pois esse já é o termo de pagamento definido na fatura, temos que garantir que as datas sejam diferente.

[antoniospneto]

Pessoal estou dando merge para liberar o CI ok, me parece bem trivial
/ocabot merge patch

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-3600-by-antoniospneto-bump-patch, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 9a869d5. Thanks a lot for contributing to OCA. ❤️

[mbcosta]

valeu @antoniospneto existe dois testes esse com Múltiplas Datas e logo abaixo com apenas uma Data, que acredito ser o caso padrão, pelo o que lembro esse caso com mais de uma data foi incluído pelo pessoal da KMEE e não tenho certeza como isso funcionaria em um caso de uso, quer dizer tem algum caso onde a Empresa muda diversas Datas de Vencimento de uma única vez? Parece que aconteceu alguma coincidência de datas que acabou gerando o erro, talvez ao invés do parâmetro days=40 poderia ser um months=1, mas acabei fazendo algo simples para não travar o CI