[RFC][l10n_br_sale_stock] teste depende da ordem de instalaçao dos modulos

[rvalyi]

os tests do l10n_br_sale_stock passam no Travis, pude verificar isso ontem ainda com essa melhoria: https://github.com/OCA/l10n-brazil/pull/1195

Porem se criar um banco e instalar com -i l10n_br_sale_stock e rodar depois os tests com --update=l10n_br_sale_stock --test-enable, temos esse erro. Parece que os testes so funcionam se instalar os mesmos modulos (mais modulos?) e na mesma sequencia do que o Travis:

2021-03-10 04:41:47,323 613 INFO br odoo.modules.loading: Modules loaded. 
2021-03-10 04:41:47,332 613 INFO br odoo.service.server: Starting post tests 
2021-03-10 04:41:47,333 613 INFO br odoo.modules.module: odoo.addons.l10n_br_sale_stock.tests.test_sale_stock running tests. 
2021-03-10 04:41:47,334 613 INFO br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: test_02_sale_stock_return (odoo.addons.l10n_br_sale_stock.tests.test_sale_stock.TestSaleStock) 
2021-03-10 04:41:47,334 613 INFO br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: `  Test a SO with a product invoiced on delivery. Deliver and invoice the SO, then do a return of the picking. Check that a refund invoice is well generated. 
2021-03-10 04:41:48,554 613 INFO br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: test_picking_invoicing_partner_shipping_invoiced (odoo.addons.l10n_br_sale_stock.tests.test_sale_stock.TestSaleStock) 
2021-03-10 04:41:48,554 613 INFO br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: `  Test the invoice generation grouped by partner/product with 2 picking and 3 moves per picking, but Partner to Shipping is different from Partner to Invoice. 
2021-03-10 04:41:51,287 613 INFO br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: test_picking_sale_order_product_and_service (odoo.addons.l10n_br_sale_stock.tests.test_sale_stock.TestSaleStock) 
2021-03-10 04:41:51,287 613 INFO br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: `  Test Sale Order with product and service 
2021-03-10 04:41:52,481 613 INFO br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: test_ungrouping_pickings_partner_shipping_different (odoo.addons.l10n_br_sale_stock.tests.test_sale_stock.TestSaleStock) 
2021-03-10 04:41:52,482 613 INFO br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: `  Test the invoice generation grouped by partner/product with 3 picking and 3 moves per picking, the 3 has the same Partner to Invoice but one has Partner to Shipping so shouldn't be grouping. 
2021-03-10 04:41:57,128 613 ERROR br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: FAIL 
2021-03-10 04:41:57,130 613 INFO br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: ====================================================================== 
2021-03-10 04:41:57,130 613 ERROR br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: FAIL: test_ungrouping_pickings_partner_shipping_different (odoo.addons.l10n_br_sale_stock.tests.test_sale_stock.TestSaleStock) 
2021-03-10 04:41:57,130 613 ERROR br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: `  Test the invoice generation grouped by partner/product with 3 picking and 3 moves per picking, the 3 has the same Partner to Invoice but one has Partner to Shipping so shouldn't be grouping. 
2021-03-10 04:41:57,131 613 ERROR br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: Traceback (most recent call last): 
2021-03-10 04:41:57,131 613 ERROR br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: `   File "/odoo/external-src/l10n-brazil/l10n_br_sale_stock/tests/test_sale_stock.py", line 263, in test_ungrouping_pickings_partner_shipping_different 
2021-03-10 04:41:57,131 613 ERROR br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: `     self.assertEquals(len(invoices), 2) 
2021-03-10 04:41:57,131 613 ERROR br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: ` AssertionError: 1 != 2 
2021-03-10 04:41:57,131 613 INFO br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: Ran 4 tests in 9.797s 
2021-03-10 04:41:57,131 613 ERROR br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock: FAILED 
2021-03-10 04:41:57,131 613 INFO br odoo.addons.l10n_br_sale_stock.tests.test_sale_stock:  (failures=1) 
2021-03-10 04:41:57,131 613 INFO br odoo.modules.module: odoo.addons.l10n_br_sale_stock.tests.test_sale_stock tested in 9.80s, 6438 queries 
2021-03-10 04:41:57,132 613 ERROR br odoo.modules.module: Module l10n_br_sale_stock: 1 failures, 0 errors 
2021-03-10 04:41:57,132 613 INFO br odoo.service.server: All post-tested in 9.80s, 6438 queries 
2021-03-10 04:41:57,132 613 INFO br odoo.service.server: Stopping gracefully 

Nao deveria ser assim, nao deveria se colocar nas costas do desenvolvedor a tarefa de adevinhar a sequencia magica de instalaçao dos modulos para rodar os testes.

Podemos ativar ou desativar um teste de acordo com a presencia de um outro modulo, isso sim. Mas nao quebrar feio assim.

cc @renatonlima @mbcosta

[marcos-mendez]

Baaaaa com razao me fodi tanto pra instalar essa merda. Podemos colocar na documentacao a ordem correta? Como eu adiciona a label de documentacao para corrigir este procedimento para o futuro?

[rvalyi]

Baaaaa com razao me fodi tanto pra instalar essa merda. Podemos colocar na documentacao a ordem correta? Como eu adiciona a label de documentacao para corrigir este procedimento para o futuro?

Na vdd o mais correto e que deve tomar o mesmo tanto de tempo eh de corrigir esse problema no teste mesmo para nao depender de ordem de instalacao ou presencia de outro modulo que na vdd eh opcional. Agora esse problema eh na hora de rodar os testes apenas. Pelo jeito esse modulo nao tem problema para ser instalado pelo menos (bom teria que checkar ate que ponto ele funciona bem instalado assim nessa ordem mas digamos que o teste falhando nao prova por sim so que o modulo instalado assim nao funciona corretamente).

[mbcosta]

@rvalyi não consegui reproduzir o erro que esta no LOG do seu comentário, ao rodar os comandos

$ odoo -d test -i l10n_br_sale_stock --stop-after-init e depois $ odoo -d test --update=l10n_br_sale_stock --test-enable

recebo outro erro

o erro é referente ao campo Diário/account.journal estar vazio dentro da Operação Fiscal, pelo o que entendi isso acontece com os dados de demonstração por não carregar o Plano de Contas antes da instalação do modulo e pelo que vi isso deveria estar sendo resolvido aqui https://github.com/OCA/l10n-brazil/blob/12.0/l10n_br_sale/hooks.py#L35 https://github.com/OCA/l10n-brazil/blob/12.0/l10n_br_sale/demo/fiscal_operation_simple.xml#L10 corrigi os testes no PR https://github.com/OCA/l10n-brazil/pull/1228

@marcos-mendez em teoria a ordem de instalação dos módulos deve ser resolvida de forma automática pelo próprio modulo , o erro que acredito que teve foi causado por um problema no carregamento dos dados fiscais no caso dos dados de demonstração, pode acontecer em outros módulos para contornar esse problema por enquanto o que pode ser feito é instalar o l10n_br_account antes.

[marcelsavegnago]

#1228

Fato

Empresa - Sua Empresa

[mbcosta]

@rvalyi @marcelsavegnago @marcos-mendez eu vi o problema com o @renatonlima e o que acontece é o seguinte foi incluído no l10n_br_base uma verificação para se for o caso dos Dados de Demonstração os módulos "l10n_br_coa_simple" e "l10n_br_coa_generic" serem instalados através do método button_install https://github.com/OCA/l10n-brazil/blob/12.0/l10n_br_base/init.py#L37 isso funciona porém os módulos são colocados para serem instalados no final da fila mas para funcionar eles deveriam instalar logo em seguida para os outros módulos encontrarem os IDs dos XML, eu tentei ver outros métodos do ir_module_module que pudessem fazer isso mas sem sucesso, por isso as opções que temos hoje são:

• Encontrar uma forma de fazer essa instalação logo em seguida quando o comando é chamado, talvez exista algum método ou alguém já precisou fazer algo parecido Ou
• Incluir os módulos na dependência do l10n_br_account, o efeito colateral é que sempre que for instalado o l10n_br_account irá instalar esses dois templates, talvez isso seja um problema menor

@netosjb @felipemotter esse é problema que vocês tiveram de "Dependecias não Declaradas" ao rodar os testes e o porque funciona ao instalar o l10n_br_account antes.

cc @mileo

[marcelsavegnago]

• Incluir os módulos na dependência do l10n_br_account, o efeito colateral é que sempre que for instalado o l10n_br_account irá instalar esses dois templates, talvez isso seja um problema menor

Da uma dor no peito :D

[github-actions[bot]]

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.