340 lines
15 KiB
Python
340 lines
15 KiB
Python
|
# -*- coding: utf-8 -*-
|
||
|
|
# Part of Odoo. See LICENSE file for full copyright and licensing details.
|
||
|
|
|
||
|
|
from odoo.addons.account.tests.common import AccountTestInvoicingCommon
|
||
|
|
from odoo.addons.stock_account.tests.test_stockvaluation import _create_accounting_data
|
||
|
|
from odoo.tests import Form, tagged
|
||
|
|
from odoo import fields, Command
|
||
|
|
|
||
|
|
class TestAccountMoveStockCommon(AccountTestInvoicingCommon):
|
||
|
|
@classmethod
|
||
|
|
def setUpClass(cls):
|
||
|
|
super().setUpClass()
|
||
|
|
|
||
|
|
cls.other_currency = cls.setup_other_currency('EUR')
|
||
|
|
|
||
|
|
(
|
||
|
|
cls.stock_input_account,
|
||
|
|
cls.stock_output_account,
|
||
|
|
cls.stock_valuation_account,
|
||
|
|
cls.expense_account,
|
||
|
|
cls.stock_journal,
|
||
|
|
) = _create_accounting_data(cls.env)
|
||
|
|
|
||
|
|
# `all_categ` should not be altered, so we can test the `post_init` hook of `stock_account`
|
||
|
|
cls.all_categ = cls.env.ref('product.product_category_all')
|
||
|
|
|
||
|
|
cls.auto_categ = cls.env['product.category'].create({
|
||
|
|
'name': 'child_category',
|
||
|
|
'parent_id': cls.all_categ.id,
|
||
|
|
"property_stock_account_input_categ_id": cls.stock_input_account.id,
|
||
|
|
"property_stock_account_output_categ_id": cls.stock_output_account.id,
|
||
|
|
"property_stock_valuation_account_id": cls.stock_valuation_account.id,
|
||
|
|
"property_stock_journal": cls.stock_journal.id,
|
||
|
|
"property_valuation": "real_time",
|
||
|
|
"property_cost_method": "standard",
|
||
|
|
})
|
||
|
|
cls.product_A = cls.env["product.product"].create(
|
||
|
|
{
|
||
|
|
"name": "Product A",
|
||
|
|
"is_storable": True,
|
||
|
|
"default_code": "prda",
|
||
|
|
"categ_id": cls.auto_categ.id,
|
||
|
|
"taxes_id": [(5, 0, 0)],
|
||
|
|
"supplier_taxes_id": [(5, 0, 0)],
|
||
|
|
"lst_price": 100.0,
|
||
|
|
"standard_price": 10.0,
|
||
|
|
"property_account_income_id": cls.company_data["default_account_revenue"].id,
|
||
|
|
"property_account_expense_id": cls.company_data["default_account_expense"].id,
|
||
|
|
}
|
||
|
|
)
|
||
|
|
|
||
|
|
cls.branch_a = cls.setup_other_company(name="Branch A", parent_id=cls.env.company.id)
|
||
|
|
|
||
|
|
|
||
|
|
@tagged("post_install", "-at_install")
|
||
|
|
class TestAccountMove(TestAccountMoveStockCommon):
|
||
|
|
def test_standard_perpetual_01_mc_01(self):
|
||
|
|
rate = self.other_currency.rate_ids.sorted()[0].rate
|
||
|
|
|
||
|
|
move_form = Form(self.env["account.move"].with_context(default_move_type="out_invoice"))
|
||
|
|
move_form.partner_id = self.partner_a
|
||
|
|
move_form.currency_id = self.other_currency
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = self.product_A
|
||
|
|
line_form.tax_ids.clear()
|
||
|
|
invoice = move_form.save()
|
||
|
|
|
||
|
|
self.assertAlmostEqual(self.product_A.lst_price * rate, invoice.amount_total)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids")), 2)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids.currency_id")), 1)
|
||
|
|
|
||
|
|
invoice._post()
|
||
|
|
|
||
|
|
self.assertAlmostEqual(self.product_A.lst_price * rate, invoice.amount_total)
|
||
|
|
self.assertAlmostEqual(self.product_A.lst_price * rate, invoice.amount_residual)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids")), 4)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids").filtered(lambda l: l.display_type == 'cogs')), 2)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids.currency_id")), 2)
|
||
|
|
|
||
|
|
def test_fifo_perpetual_01_mc_01(self):
|
||
|
|
self.product_A.categ_id.property_cost_method = "fifo"
|
||
|
|
rate = self.other_currency.rate_ids.sorted()[0].rate
|
||
|
|
|
||
|
|
move_form = Form(self.env["account.move"].with_context(default_move_type="out_invoice"))
|
||
|
|
move_form.partner_id = self.partner_a
|
||
|
|
move_form.currency_id = self.other_currency
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = self.product_A
|
||
|
|
line_form.tax_ids.clear()
|
||
|
|
invoice = move_form.save()
|
||
|
|
|
||
|
|
self.assertAlmostEqual(self.product_A.lst_price * rate, invoice.amount_total)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids")), 2)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids.currency_id")), 1)
|
||
|
|
|
||
|
|
invoice._post()
|
||
|
|
|
||
|
|
self.assertAlmostEqual(self.product_A.lst_price * rate, invoice.amount_total)
|
||
|
|
self.assertAlmostEqual(self.product_A.lst_price * rate, invoice.amount_residual)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids")), 4)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids").filtered(lambda l: l.display_type == 'cogs')), 2)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids.currency_id")), 2)
|
||
|
|
|
||
|
|
def test_average_perpetual_01_mc_01(self):
|
||
|
|
self.product_A.categ_id.property_cost_method = "average"
|
||
|
|
rate = self.other_currency.rate_ids.sorted()[0].rate
|
||
|
|
|
||
|
|
move_form = Form(self.env["account.move"].with_context(default_move_type="out_invoice"))
|
||
|
|
move_form.partner_id = self.partner_a
|
||
|
|
move_form.currency_id = self.other_currency
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = self.product_A
|
||
|
|
line_form.tax_ids.clear()
|
||
|
|
invoice = move_form.save()
|
||
|
|
|
||
|
|
self.assertAlmostEqual(self.product_A.lst_price * rate, invoice.amount_total)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids")), 2)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids.currency_id")), 1)
|
||
|
|
|
||
|
|
invoice._post()
|
||
|
|
|
||
|
|
self.assertAlmostEqual(self.product_A.lst_price * rate, invoice.amount_total)
|
||
|
|
self.assertAlmostEqual(self.product_A.lst_price * rate, invoice.amount_residual)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids")), 4)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids").filtered(lambda l: l.display_type == 'cogs')), 2)
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids.currency_id")), 2)
|
||
|
|
|
||
|
|
def test_storno_accounting(self):
|
||
|
|
"""Storno accounting uses negative numbers on debit/credit to cancel other moves.
|
||
|
|
This test checks that we do the same for the anglosaxon lines when storno is enabled.
|
||
|
|
"""
|
||
|
|
self.env.company.account_storno = True
|
||
|
|
self.env.company.anglo_saxon_accounting = True
|
||
|
|
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_refund',
|
||
|
|
'invoice_date': fields.Date.from_string('2019-01-01'),
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'currency_id': self.other_currency.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
(0, None, {'product_id': self.product_A.id}),
|
||
|
|
]
|
||
|
|
})
|
||
|
|
move.action_post()
|
||
|
|
|
||
|
|
stock_output_line = move.line_ids.filtered(lambda l: l.account_id == self.stock_output_account)
|
||
|
|
self.assertEqual(stock_output_line.debit, 0)
|
||
|
|
self.assertEqual(stock_output_line.credit, -10)
|
||
|
|
|
||
|
|
expense_line = move.line_ids.filtered(lambda l: l.account_id == self.product_A.property_account_expense_id)
|
||
|
|
self.assertEqual(expense_line.debit, -10)
|
||
|
|
self.assertEqual(expense_line.credit, 0)
|
||
|
|
|
||
|
|
def test_standard_manual_tax_edit(self):
|
||
|
|
''' Test manually editing tax amount, cogs creation should not reset tax amount '''
|
||
|
|
move_form = Form(self.env["account.move"].with_context(default_move_type="out_invoice"))
|
||
|
|
move_form.partner_id = self.partner_a
|
||
|
|
self.company_data["default_account_revenue"].write({
|
||
|
|
'tax_ids': [(6, 0, [self.env.company.account_sale_tax_id.id])]
|
||
|
|
})
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = self.product_A
|
||
|
|
invoice = move_form.save()
|
||
|
|
|
||
|
|
self.assertEqual(invoice.amount_total, 115)
|
||
|
|
self.assertEqual(invoice.amount_untaxed, 100)
|
||
|
|
self.assertEqual(invoice.amount_tax, 15)
|
||
|
|
|
||
|
|
# simulate manual tax edit via widget
|
||
|
|
tax_totals = invoice.tax_totals
|
||
|
|
tax_totals['subtotals'][0]['tax_groups'][0]['tax_amount_currency'] = 14.0
|
||
|
|
invoice.tax_totals = tax_totals
|
||
|
|
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids")), 3)
|
||
|
|
self.assertEqual(invoice.amount_total, 114)
|
||
|
|
self.assertEqual(invoice.amount_untaxed, 100)
|
||
|
|
self.assertEqual(invoice.amount_tax, 14)
|
||
|
|
|
||
|
|
invoice._post()
|
||
|
|
|
||
|
|
self.assertEqual(len(invoice.mapped("line_ids")), 5)
|
||
|
|
self.assertEqual(invoice.amount_total, 114)
|
||
|
|
self.assertEqual(invoice.amount_untaxed, 100)
|
||
|
|
self.assertEqual(invoice.amount_tax, 14)
|
||
|
|
|
||
|
|
def test_basic_bill(self):
|
||
|
|
"""
|
||
|
|
When billing a storable product with a basic category (manual
|
||
|
|
valuation), the account used should be the expenses one. This test
|
||
|
|
checks the flow with two companies:
|
||
|
|
- One that existed before the installation of `stock_account` (to test
|
||
|
|
the post-install hook)
|
||
|
|
- One created after the module installation
|
||
|
|
"""
|
||
|
|
first_company = self.env['res.company'].browse(1)
|
||
|
|
self.env.user.company_ids |= first_company
|
||
|
|
basic_product = self.env['product.product'].create({
|
||
|
|
'name': 'SuperProduct',
|
||
|
|
'is_storable': True,
|
||
|
|
'categ_id': self.all_categ.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
for company in (self.env.company | first_company):
|
||
|
|
bill_form = Form(self.env['account.move'].with_company(company.id).with_context(default_move_type='in_invoice'))
|
||
|
|
bill_form.partner_id = self.partner_a
|
||
|
|
bill_form.invoice_date = fields.Date.today()
|
||
|
|
with bill_form.invoice_line_ids.new() as line:
|
||
|
|
line.product_id = basic_product
|
||
|
|
line.price_unit = 100
|
||
|
|
bill = bill_form.save()
|
||
|
|
bill.action_post()
|
||
|
|
|
||
|
|
product_accounts = basic_product.product_tmpl_id.with_company(company.id).get_product_accounts()
|
||
|
|
self.assertEqual(bill.invoice_line_ids.account_id, product_accounts['expense'])
|
||
|
|
|
||
|
|
def test_product_valuation_method_change_to_automated_negative_on_hand_qty(self):
|
||
|
|
""" We have a product whose category has manual valuation and on-hand quantity is negative:
|
||
|
|
Upon switching to an automated valuation method for the product category, the following
|
||
|
|
entries should be generated in the stock journal:
|
||
|
|
1. CREDIT to valuation account
|
||
|
|
2. DEBIT to stock output account
|
||
|
|
"""
|
||
|
|
stock_location = self.env['stock.warehouse'].search([
|
||
|
|
('company_id', '=', self.env.company.id),
|
||
|
|
], limit=1).lot_stock_id
|
||
|
|
categ = self.env['product.category'].create({'name': 'categ'})
|
||
|
|
product = self.product_a
|
||
|
|
product.write({
|
||
|
|
'is_storable': True,
|
||
|
|
'categ_id': categ.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
out_picking = self.env['stock.picking'].create({
|
||
|
|
'location_id': stock_location.id,
|
||
|
|
'location_dest_id': self.ref('stock.stock_location_customers'),
|
||
|
|
'picking_type_id': stock_location.warehouse_id.out_type_id.id,
|
||
|
|
})
|
||
|
|
sm = self.env['stock.move'].create({
|
||
|
|
'name': product.name,
|
||
|
|
'product_id': product.id,
|
||
|
|
'product_uom_qty': 1,
|
||
|
|
'product_uom': product.uom_id.id,
|
||
|
|
'location_id': out_picking.location_id.id,
|
||
|
|
'location_dest_id': out_picking.location_dest_id.id,
|
||
|
|
'picking_id': out_picking.id,
|
||
|
|
})
|
||
|
|
out_picking.action_confirm()
|
||
|
|
sm.quantity = 1
|
||
|
|
out_picking.button_validate()
|
||
|
|
|
||
|
|
categ.write({
|
||
|
|
'property_valuation': 'real_time',
|
||
|
|
'property_stock_account_input_categ_id': self.stock_input_account.id,
|
||
|
|
'property_stock_account_output_categ_id': self.stock_output_account.id,
|
||
|
|
'property_stock_valuation_account_id': self.stock_valuation_account.id,
|
||
|
|
'property_stock_journal': self.stock_journal.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
amls = self.env['account.move.line'].search([('product_id', '=', product.id)]).sorted(
|
||
|
|
# ensure the aml with the stock_valuation_account is the first one
|
||
|
|
lambda amls: amls.account_id != self.stock_valuation_account
|
||
|
|
)
|
||
|
|
|
||
|
|
expected_valuation_line = {
|
||
|
|
'account_id': self.stock_valuation_account.id,
|
||
|
|
'credit': product.standard_price,
|
||
|
|
'debit': 0,
|
||
|
|
}
|
||
|
|
expected_output_line = {
|
||
|
|
'account_id': self.stock_output_account.id,
|
||
|
|
'credit': 0,
|
||
|
|
'debit': product.standard_price,
|
||
|
|
}
|
||
|
|
self.assertRecordValues(amls, [expected_valuation_line, expected_output_line])
|
||
|
|
|
||
|
|
def test_stock_account_move_automated_not_standard_with_branch_company(self):
|
||
|
|
"""
|
||
|
|
Test that the validation of a stock picking does not fail `_check_company`
|
||
|
|
at the creation of the account move with sub company
|
||
|
|
"""
|
||
|
|
branch_a = self.branch_a['company']
|
||
|
|
self.env.user.company_id = branch_a
|
||
|
|
|
||
|
|
self.auto_categ.write({'property_cost_method': 'average', 'property_valuation': 'real_time'})
|
||
|
|
product = self.product_A
|
||
|
|
product.write({'categ_id': self.auto_categ.id, 'standard_price': 300, 'company_id': branch_a.id})
|
||
|
|
|
||
|
|
stock_location = self.env['stock.warehouse'].search([
|
||
|
|
('company_id', '=', self.env.company.id),
|
||
|
|
], limit=1).lot_stock_id
|
||
|
|
|
||
|
|
in_picking = self.env['stock.picking'].create({
|
||
|
|
'location_id': stock_location.id,
|
||
|
|
'location_dest_id': self.ref('stock.stock_location_customers'),
|
||
|
|
'picking_type_id': stock_location.warehouse_id.in_type_id.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
sm = self.env['stock.move'].create({
|
||
|
|
'name': product.name,
|
||
|
|
'product_id': product.id,
|
||
|
|
'product_uom_qty': 1,
|
||
|
|
'product_uom': product.uom_id.id,
|
||
|
|
'location_id': in_picking.location_id.id,
|
||
|
|
'location_dest_id': in_picking.location_dest_id.id,
|
||
|
|
'picking_id': in_picking.id,
|
||
|
|
})
|
||
|
|
in_picking.button_validate()
|
||
|
|
self.assertEqual(sm.state, 'done')
|
||
|
|
self.assertEqual(sm.account_move_ids.company_id, self.env.company)
|
||
|
|
|
||
|
|
def test_cogs_analytic_accounting(self):
|
||
|
|
"""Check analytic distribution is correctly propagated to COGS lines"""
|
||
|
|
self.env.company.anglo_saxon_accounting = True
|
||
|
|
default_plan = self.env['account.analytic.plan'].create({
|
||
|
|
'name': 'Default',
|
||
|
|
})
|
||
|
|
analytic_account = self.env['account.analytic.account'].create({
|
||
|
|
'name': 'Account 1',
|
||
|
|
'plan_id': default_plan.id,
|
||
|
|
'company_id': False,
|
||
|
|
})
|
||
|
|
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_refund',
|
||
|
|
'invoice_date': fields.Date.from_string('2019-01-01'),
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_A.id,
|
||
|
|
'analytic_distribution': {
|
||
|
|
analytic_account.id: 100,
|
||
|
|
},
|
||
|
|
}),
|
||
|
|
]
|
||
|
|
})
|
||
|
|
move.action_post()
|
||
|
|
|
||
|
|
cogs_line = move.line_ids.filtered(lambda l: l.account_id == self.product_A.property_account_expense_id)
|
||
|
|
self.assertEqual(cogs_line.analytic_distribution, {str(analytic_account.id): 100})
|