[16.0][MIG][l10n_br_sale_commission]

[rvalyi]

migração padrão a partir da 14.0 - work in progress

cc @renatonlima @mbcosta

[rvalyi]

/ocabot migration l10n_br_sale_commission

[mbcosta]

@rvalyi @renatonlima fiz um Force-Push para atualizar a branch, um rebase, e inclui um commit que corrigi o problema no Teste, aparentemente o Pedido de Vendas do Dado de Demonstração apesar ter o campo agent_ids preenchido e ao rodar os testes de forma simples

$ createdb test
$ odoo -d test -i l10n_br_account --workers 0 --stop-after-init
$ odoo -d test -i l10n_br_sale_commission--workers 0 --stop-after-init
$ odoo -d test -u l10n_br_sale_commission--workers 0 --stop-after-init --test-enable

Roda sem erros, mas no teste do CI o agent_ids vem vazio, ainda não identifiquei o problema mas deixei um TODO para avaliar ou agora ou no momento de separar os Testes dos Dados de Demonstração.

Uma outra alteração foi remover a dependência do account_commission porque o sale_commission já tem essa dependência

https://github.com/OCA/commission/blob/16.0/sale_commission/__manifest__.py#L12

Acredito que os últimos 5 commits podem ser juntados no commit de migração c77027d via amend mas crie um novo por enquanto para facilitar o DIFF e a revisão.

Estou buscando ver o porque algumas partes do código não estão sendo chamadas o que está baixando a Cobertura de Testes, parece que não está sendo encontrado o valor document_type_id no context

https://app.codecov.io/gh/OCA/l10n-brazil/pull/3646?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=checks&utm_campaign=pr+comments&utm_term=OCA

[mbcosta]

Atualizando o PR, segue as alterações:

• Refatorei a Visão de Configuração porque a tela estava assim

Parece que isso é causado pela TAG group removendo ( apenas para teste )

Mas achei melhor manter o group e deixei a tela assim

Seria melhor remover a TAG group?

• Criando o Domain via Compute porque retornar Domain via Onchange foi descontinuado/deprecated

WARNING db odoo.models: onchange method CommissionMakeInvoice._onchange_commission_document_type_id returned a domain, this is deprecated

Referencia https://github.com/odoo/odoo/pull/41918#issuecomment-824946980

Exemplo para confirmar o funcionamento do domain com o compute em Configurações

Com Domain

Desmarcando a opção de Gerar os Documentos Fiscais o campo do Documento Fiscal vai ser apagado e com isso não deve ter o Domain

No Wizard de Criação da Fatura de Comissões

Caso que não deve ter o Domain

Inclui o Domain na definição do campo, é possível avaliar se deveria ser apenas na visão.

• Refatorei os Testes usando Form e tirei as partes comentadas que testavam se a Fatura de Comissão criada tem o Documento e a Operação Fiscal preenchidas, isso também pode ser testado na tela com os Dados de Demonstração

• No objeto commission.settlement estava sendo sobre escrito o método make_invoices para preencher o campo settlement_id na account.move.line mas pelo o que vi isso está funcionando então se tornou desnecessário, inclui no Teste a validação se o campo está presente para confirmar, por enquanto o último commit, apenas para referencia segue imagem do código removido

É preciso avaliar:

• Juntar alguns ou todos os commits em apenas um de Migração? Deixei em separado para facilitar a Revisão vendo o Diff
• Atualizar o README? History?
• O TODO no método _prepare_invoice precisa ser resolvido agora na migração?

https://github.com/akretion/l10n-brazil/blob/16.0-mig-l10n_br_sale_commission/l10n_br_sale_commission/models/commission_settlement.py#L31

Seria importante verificar isso para alterar o status do PR para Pronto para Revisão/Ready to Review mas já é possível fazer a Revisão.

Com a alteração nos testes para usar o Form a cobertura de testes do projeto aumentou 1,07% e passou para 90,49%.

[CristianoMafraJunior]

Aqui não deveria ser class CommissionSettlement?

[mbcosta]

Sim, corrigido via amend no último commit, obrigado @CristianoMafraJunior pela revisão