[16.0][FIX] l10n_br_fiscal: rename operation code - fwp 3628 #3632
Conversation
|
Hi @renatonlima, |
|
O erro parecer ser algo similar ou relacionado ao #3600 |
|
de repente amanha o erro some. Se acontecer apenas 2 dias no ano, ta OK não cuidar. Não sei se vale a pena mudar datas/dias como o Magno tentou fazer uns dias atras pois talvez apenas mudaria as datas quando os testes falham. A gente poderia tentar entender exactemente o problema ou usar o módulo freezegun como alguns testes da OCA fazem. Mas enfim temos outras prioridades... |
|
@rvalyi acabei buscando resolver o erro dos Testes no PR #3634 o problema é na funcionalidade de Alterar a Data de Vencimento de mais de 1 Linha de uma única vez, é preciso verificar se existe esse Caso de Uso até onde vi na maioria dos casos é alterado apenas uma Linha por vez( alguém tem essa informação? Ou tem esse Caso de Uso?), como eu também estava precisando rodar os Testes localmente acabei fazendo essa alteração e achei melhor subir o PR, no teste não importa se a nova Data de Vencimento tem uma diferença de Dias, Meses ou Ano porque o que esta se buscando validar é se funcionalidade ao ser chamada altera a Data de Vencimento da Linha/account.move.line e se essa nova Data não é igual a Hoje, acredito que colocando 1 Ano a mais a possibilidade desse erro voltar a acontecer é remota. |
CristianoMafraJunior
force-pushed
the
16.0-fix-code_operation
branch
from
5b3a83b to
3c21e61
Compare
|
/ocabot merge minor |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at 6e99079. Thanks a lot for contributing to OCA. ❤️ |
fwp #3628