Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][REF] l10n_br_stock_account: Não informar a Operação Fiscal Padrão quando a Empresa não é do Brasil #3599

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

mbcosta
Copy link
Contributor

@mbcosta mbcosta commented Jan 28, 2025

Not inform Fiscal Operation when Company country is not Brazil.

Alteração para não informar a Operação Fiscal Padrão quando a Empresa não do Brasil, aproveitei para remover a atualização do invoice_state das Linhas porque na v16 isso já é feito no super() aqui https://github.com/OCA/account-invoicing/blob/16.0/stock_picking_invoicing/models/stock_picking.py#L47

Como comentado no PR #3598 isso é necessário para não gerar erros nos testes do módulo delivery e consequentemente na migração do #3571

2025-01-28 16:45:41,056 18 ERROR test_br odoo.addons.delivery.tests.test_delivery_cost: FAIL: TestDeliveryCost.test_00_delivery_cost
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/odoo/addons/delivery/tests/test_delivery_cost.py", line 109, in test_00_delivery_cost
    self.assertEqual(len(line), 1, "Delivery cost is not Added")
AssertionError: 0 != 1 : Delivery cost is not Added

2025-01-28 17:06:29,522 24 ERROR test_br odoo.addons.delivery.tests.test_delivery_cost: FAIL: TestDeliveryCost.test_01_taxes_on_delivery_cost
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/odoo/addons/delivery/tests/test_delivery_cost.py", line 293, in test_01_taxes_on_delivery_cost
    self.assertRecordValues(sale_order.order_line, [{'price_subtotal': 9.09, 'price_total': 10.45}])
  File "/usr/local/lib/python3.10/site-packages/odoo/tests/common.py", line 611, in assertRecordValues
    self.fail('\n'.join(errors))
AssertionError: The records and expected_values do not match.

==== Differences at index 0 ====
--- 

+++ 

@@ -1 +1 @@

-price_total:9.09
+price_total:10.450000000000001
 


2025-01-28 17:06:41,952 24 ERROR test_br odoo.addons.delivery.tests.test_delivery_cost: FAIL: TestDeliveryCost.test_delivery_real_cost
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/odoo/addons/delivery/tests/test_delivery_cost.py", line 364, in test_delivery_real_cost
    self.assertEqual(delivery_line.price_unit, picking.carrier_price)
AssertionError: 0.0 != 40.0


2025-01-28 17:07:14,559 24 ERROR test_br odoo.addons.delivery.tests.test_delivery_stock_move: FAIL: StockMoveInvoice.test_01_delivery_stock_move
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/odoo/addons/delivery/tests/test_delivery_stock_move.py", line 98, in test_01_delivery_stock_move
    self.assertEqual(moves[0].move_line_ids.sale_price, 1725.0, 'wrong shipping value')
AssertionError: 1500.0 != 1725.0 : wrong shipping value


2025-01-28 17:07:17,937 24 ERROR test_br odoo.addons.delivery.tests.test_delivery_stock_move: FAIL: StockMoveInvoice.test_02_delivery_stock_move
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/odoo/addons/delivery/tests/test_delivery_stock_move.py", line 141, in test_02_delivery_stock_move
    self.assertEqual(moves[0].move_line_ids[0].sale_price, 862.5, 'wrong shipping value')
AssertionError: 750.0 != 862.5 : wrong shipping value

cc @OCA/local-brazil-maintainers

@OCA-git-bot
Copy link
Contributor

Hi @renatonlima,
some modules you are maintaining are being modified, check this out!

@mbcosta mbcosta force-pushed the 16.0-FIX-br_stock_account_without_fiscal_operation branch from 449ad28 to 2d85c82 Compare January 30, 2025 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants