312 lines
16 KiB
Python
312 lines
16 KiB
Python
|
# -*- coding: utf-8 -*-
|
||
|
|
# Part of Odoo. See LICENSE file for full copyright and licensing details.
|
||
|
|
|
||
|
|
import time
|
||
|
|
from odoo.tests import tagged
|
||
|
|
from odoo.addons.account.tests.common import AccountTestInvoicingCommon
|
||
|
|
|
||
|
|
|
||
|
|
class TestAngloSaxonCommon(AccountTestInvoicingCommon):
|
||
|
|
|
||
|
|
@classmethod
|
||
|
|
def setUpClass(cls, chart_template_ref=None):
|
||
|
|
super().setUpClass(chart_template_ref=chart_template_ref)
|
||
|
|
cls.PosMakePayment = cls.env['pos.make.payment']
|
||
|
|
cls.PosOrder = cls.env['pos.order']
|
||
|
|
cls.Statement = cls.env['account.bank.statement']
|
||
|
|
cls.company = cls.env.company
|
||
|
|
cls.warehouse = cls.env['stock.warehouse'].search([('company_id', '=', cls.env.company.id)], limit=1)
|
||
|
|
cls.partner = cls.env['res.partner'].create({'name': 'Partner 1'})
|
||
|
|
cls.category = cls.env.ref('product.product_category_all')
|
||
|
|
cls.category = cls.category.copy({'name': 'New category','property_valuation': 'real_time'})
|
||
|
|
cls.account = cls.env['account.account'].create({'name': 'Receivable', 'code': 'RCV00', 'account_type': 'asset_receivable', 'reconcile': True})
|
||
|
|
account_expense = cls.env['account.account'].create({'name': 'Expense', 'code': 'EXP00', 'account_type': 'expense', 'reconcile': True})
|
||
|
|
account_income = cls.env['account.account'].create({'name': 'Income', 'code': 'INC00', 'account_type': 'income', 'reconcile': True})
|
||
|
|
account_output = cls.env['account.account'].create({'name': 'Output', 'code': 'OUT00', 'account_type': 'expense', 'reconcile': True})
|
||
|
|
account_valuation = cls.env['account.account'].create({'name': 'Valuation', 'code': 'STV00', 'account_type': 'expense', 'reconcile': True})
|
||
|
|
cls.partner.property_account_receivable_id = cls.account
|
||
|
|
cls.category.property_account_income_categ_id = account_income
|
||
|
|
cls.category.property_account_expense_categ_id = account_expense
|
||
|
|
cls.category.property_stock_account_input_categ_id = cls.account
|
||
|
|
cls.category.property_stock_account_output_categ_id = account_output
|
||
|
|
cls.category.property_stock_valuation_account_id = account_valuation
|
||
|
|
cls.category.property_stock_journal = cls.env['account.journal'].create({'name': 'Stock journal', 'type': 'sale', 'code': 'STK00'})
|
||
|
|
cls.cash_journal = cls.env['account.journal'].create(
|
||
|
|
{'name': 'CASH journal', 'type': 'cash', 'code': 'CSH02'})
|
||
|
|
cls.cash_payment_method = cls.env['pos.payment.method'].create({
|
||
|
|
'name': 'Cash Test',
|
||
|
|
'journal_id': cls.cash_journal.id,
|
||
|
|
})
|
||
|
|
cls.pos_config = cls.env['pos.config'].create({
|
||
|
|
'name': 'New POS config',
|
||
|
|
'payment_method_ids': cls.cash_payment_method,
|
||
|
|
})
|
||
|
|
cls.product = cls.env['product.product'].create({
|
||
|
|
'name': 'New product',
|
||
|
|
'standard_price': 100,
|
||
|
|
'available_in_pos': True,
|
||
|
|
'type': 'product',
|
||
|
|
})
|
||
|
|
cls.company.anglo_saxon_accounting = True
|
||
|
|
cls.company.point_of_sale_update_stock_quantities = 'real'
|
||
|
|
cls.product.categ_id = cls.category
|
||
|
|
cls.product.property_account_expense_id = account_expense
|
||
|
|
cls.product.property_account_income_id = account_income
|
||
|
|
sale_journal = cls.env['account.journal'].create({'name': 'POS journal', 'type': 'sale', 'code': 'POS00'})
|
||
|
|
cls.pos_config.journal_id = sale_journal
|
||
|
|
cls.cash_journal = cls.env['account.journal'].create({'name': 'CASH journal', 'type': 'cash', 'code': 'CSH00'})
|
||
|
|
cls.sale_journal = cls.env['account.journal'].create({'name': 'SALE journal', 'type': 'sale', 'code': 'INV00'})
|
||
|
|
cls.pos_config.invoice_journal_id = cls.sale_journal
|
||
|
|
cls.cash_payment_method = cls.env['pos.payment.method'].create({
|
||
|
|
'name': 'Cash Test',
|
||
|
|
'journal_id': cls.cash_journal.id,
|
||
|
|
'receivable_account_id': cls.account.id,
|
||
|
|
})
|
||
|
|
cls.pos_config.write({'payment_method_ids': [(6, 0, cls.cash_payment_method.ids)]})
|
||
|
|
|
||
|
|
@tagged('post_install', '-at_install')
|
||
|
|
class TestAngloSaxonFlow(TestAngloSaxonCommon):
|
||
|
|
|
||
|
|
def test_create_account_move_line(self):
|
||
|
|
# This test will check that the correct journal entries are created when a product in real time valuation
|
||
|
|
# is sold in a company using anglo-saxon
|
||
|
|
self.pos_config.open_ui()
|
||
|
|
current_session = self.pos_config.current_session_id
|
||
|
|
self.cash_journal.loss_account_id = self.account
|
||
|
|
current_session.set_cashbox_pos(0, None)
|
||
|
|
|
||
|
|
# I create a PoS order with 1 unit of New product at 450 EUR
|
||
|
|
self.pos_order_pos0 = self.PosOrder.create({
|
||
|
|
'company_id': self.company.id,
|
||
|
|
'partner_id': self.partner.id,
|
||
|
|
'pricelist_id': self.company.partner_id.property_product_pricelist.id,
|
||
|
|
'session_id': self.pos_config.current_session_id.id,
|
||
|
|
'lines': [(0, 0, {
|
||
|
|
'name': "OL/0001",
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'price_unit': 450,
|
||
|
|
'discount': 0.0,
|
||
|
|
'qty': 1.0,
|
||
|
|
'price_subtotal': 450,
|
||
|
|
'price_subtotal_incl': 450,
|
||
|
|
})],
|
||
|
|
'amount_total': 450,
|
||
|
|
'amount_tax': 0,
|
||
|
|
'amount_paid': 0,
|
||
|
|
'amount_return': 0,
|
||
|
|
'last_order_preparation_change': '{}'
|
||
|
|
})
|
||
|
|
|
||
|
|
# I make a payment to fully pay the order
|
||
|
|
context_make_payment = {"active_ids": [self.pos_order_pos0.id], "active_id": self.pos_order_pos0.id}
|
||
|
|
self.pos_make_payment_0 = self.PosMakePayment.with_context(context_make_payment).create({
|
||
|
|
'amount': 450.0,
|
||
|
|
'payment_method_id': self.cash_payment_method.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
# I click on the validate button to register the payment.
|
||
|
|
context_payment = {'active_id': self.pos_order_pos0.id}
|
||
|
|
self.pos_make_payment_0.with_context(context_payment).check()
|
||
|
|
|
||
|
|
# I check that the order is marked as paid
|
||
|
|
self.assertEqual(self.pos_order_pos0.state, 'paid', 'Order should be in paid state.')
|
||
|
|
self.assertEqual(self.pos_order_pos0.amount_paid, 450, 'Amount paid for the order should be updated.')
|
||
|
|
|
||
|
|
# I close the current session to generate the journal entries
|
||
|
|
current_session_id = self.pos_config.current_session_id
|
||
|
|
current_session_id.post_closing_cash_details(450.0)
|
||
|
|
current_session_id.close_session_from_ui()
|
||
|
|
self.assertEqual(current_session_id.state, 'closed', 'Check that session is closed')
|
||
|
|
|
||
|
|
# Check if there is account_move in the order.
|
||
|
|
# There shouldn't be because the order is not invoiced.
|
||
|
|
self.assertFalse(self.pos_order_pos0.account_move, 'There should be no invoice in the order.')
|
||
|
|
|
||
|
|
# I test that the generated journal entries are correct.
|
||
|
|
account_output = self.category.property_stock_account_output_categ_id
|
||
|
|
expense_account = self.category.property_account_expense_categ_id
|
||
|
|
aml = current_session.move_id.line_ids
|
||
|
|
aml_output = aml.filtered(lambda l: l.account_id.id == account_output.id)
|
||
|
|
aml_expense = aml.filtered(lambda l: l.account_id.id == expense_account.id)
|
||
|
|
self.assertEqual(aml_output.credit, self.product.standard_price, "Cost of Good Sold entry missing or mismatching")
|
||
|
|
self.assertEqual(aml_expense.debit, self.product.standard_price, "Cost of Good Sold entry missing or mismatching")
|
||
|
|
|
||
|
|
def _prepare_pos_order(self):
|
||
|
|
""" Set the cost method of `self.product` as FIFO. Receive 5@5 and 5@1 and
|
||
|
|
create a `pos.order` record selling 7 units @ 450.
|
||
|
|
"""
|
||
|
|
# check fifo Costing Method of product.category
|
||
|
|
self.product.categ_id.property_cost_method = 'fifo'
|
||
|
|
self.product.standard_price = 5.0
|
||
|
|
self.env['stock.quant'].with_context(inventory_mode=True).create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'inventory_quantity': 5.0,
|
||
|
|
'location_id': self.warehouse.lot_stock_id.id,
|
||
|
|
}).action_apply_inventory()
|
||
|
|
self.product.standard_price = 1.0
|
||
|
|
self.env['stock.quant'].with_context(inventory_mode=True).create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'inventory_quantity': 10.0,
|
||
|
|
'location_id': self.warehouse.lot_stock_id.id,
|
||
|
|
}).action_apply_inventory()
|
||
|
|
self.assertEqual(self.product.value_svl, 30, "Value should be (5*5 + 5*1) = 30")
|
||
|
|
self.assertEqual(self.product.quantity_svl, 10)
|
||
|
|
|
||
|
|
|
||
|
|
self.pos_config.open_ui()
|
||
|
|
pos_session = self.pos_config.current_session_id
|
||
|
|
pos_session.set_cashbox_pos(0, None)
|
||
|
|
|
||
|
|
pos_order_values = {
|
||
|
|
'company_id': self.company.id,
|
||
|
|
'partner_id': self.partner.id,
|
||
|
|
'pricelist_id': self.company.partner_id.property_product_pricelist.id,
|
||
|
|
'session_id': self.pos_config.current_session_id.id,
|
||
|
|
'lines': [(0, 0, {
|
||
|
|
'name': "OL/0001",
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'price_unit': 450,
|
||
|
|
'discount': 0.0,
|
||
|
|
'qty': 7.0,
|
||
|
|
'price_subtotal': 7 * 450,
|
||
|
|
'price_subtotal_incl': 7 * 450,
|
||
|
|
})],
|
||
|
|
'amount_total': 7 * 450,
|
||
|
|
'amount_tax': 0,
|
||
|
|
'amount_paid': 0,
|
||
|
|
'amount_return': 0,
|
||
|
|
'last_order_preparation_change': '{}'
|
||
|
|
}
|
||
|
|
|
||
|
|
return self.PosOrder.create(pos_order_values)
|
||
|
|
|
||
|
|
def test_fifo_valuation_no_invoice(self):
|
||
|
|
"""Register a payment and validate a session after selling a fifo
|
||
|
|
product without making an invoice for the customer"""
|
||
|
|
pos_order_pos0 = self._prepare_pos_order()
|
||
|
|
context_make_payment = {"active_ids": [pos_order_pos0.id], "active_id": pos_order_pos0.id}
|
||
|
|
self.pos_make_payment_0 = self.PosMakePayment.with_context(context_make_payment).create({
|
||
|
|
'amount': 7 * 450.0,
|
||
|
|
'payment_method_id': self.cash_payment_method.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
# register the payment
|
||
|
|
context_payment = {'active_id': pos_order_pos0.id}
|
||
|
|
self.pos_make_payment_0.with_context(context_payment).check()
|
||
|
|
|
||
|
|
# validate the session
|
||
|
|
current_session_id = self.pos_config.current_session_id
|
||
|
|
current_session_id.post_closing_cash_details(7 * 450.0)
|
||
|
|
current_session_id.close_session_from_ui()
|
||
|
|
|
||
|
|
# check the anglo saxon move lines
|
||
|
|
# with uninvoiced orders, the account_move field of pos.order is empty.
|
||
|
|
# the accounting lines are in move_id of pos.session.
|
||
|
|
session_move = pos_order_pos0.session_id.move_id
|
||
|
|
line = session_move.line_ids.filtered(lambda l: l.debit and l.account_id == self.category.property_account_expense_categ_id)
|
||
|
|
self.assertEqual(session_move.journal_id, self.pos_config.journal_id)
|
||
|
|
self.assertEqual(line.debit, 27, 'As it is a fifo product, the move\'s value should be 5*5 + 2*1')
|
||
|
|
|
||
|
|
def test_fifo_valuation_with_invoice(self):
|
||
|
|
"""Register a payment and validate a session after selling a fifo
|
||
|
|
product and make an invoice for the customer"""
|
||
|
|
pos_order_pos0 = self._prepare_pos_order()
|
||
|
|
context_make_payment = {"active_ids": [pos_order_pos0.id], "active_id": pos_order_pos0.id}
|
||
|
|
self.pos_make_payment_0 = self.PosMakePayment.with_context(context_make_payment).create({
|
||
|
|
'amount': 7 * 450.0,
|
||
|
|
'payment_method_id': self.cash_payment_method.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
# register the payment
|
||
|
|
context_payment = {'active_id': pos_order_pos0.id}
|
||
|
|
self.pos_make_payment_0.with_context(context_payment).check()
|
||
|
|
|
||
|
|
# Create the customer invoice
|
||
|
|
pos_order_pos0.action_pos_order_invoice()
|
||
|
|
|
||
|
|
# check the anglo saxon move lines
|
||
|
|
line = pos_order_pos0.account_move.line_ids.filtered(lambda l: l.debit and l.account_id == self.category.property_account_expense_categ_id)
|
||
|
|
self.assertEqual(pos_order_pos0.account_move.journal_id, self.pos_config.invoice_journal_id)
|
||
|
|
self.assertEqual(line.debit, 27, 'As it is a fifo product, the move\'s value should be 5*5 + 2*1')
|
||
|
|
|
||
|
|
def test_cogs_with_ship_later_no_invoicing(self):
|
||
|
|
# This test will check that the correct journal entries are created when a product in real time valuation
|
||
|
|
# is sold using the ship later option and no invoice is created in a company using anglo-saxon
|
||
|
|
self.pos_config.open_ui()
|
||
|
|
current_session = self.pos_config.current_session_id
|
||
|
|
self.cash_journal.loss_account_id = self.account
|
||
|
|
current_session.set_cashbox_pos(0, None)
|
||
|
|
|
||
|
|
# 2 step delivery method
|
||
|
|
self.warehouse.delivery_steps = 'pick_ship'
|
||
|
|
|
||
|
|
# I create a PoS order with 1 unit of New product at 450 EUR
|
||
|
|
self.pos_order_pos0 = self.PosOrder.create({
|
||
|
|
'company_id': self.company.id,
|
||
|
|
'partner_id': self.partner.id,
|
||
|
|
'pricelist_id': self.company.partner_id.property_product_pricelist.id,
|
||
|
|
'session_id': self.pos_config.current_session_id.id,
|
||
|
|
'to_invoice': False,
|
||
|
|
'shipping_date': '2023-01-01',
|
||
|
|
'lines': [(0, 0, {
|
||
|
|
'name': "OL/0001",
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'price_unit': 450,
|
||
|
|
'discount': 0.0,
|
||
|
|
'qty': 1.0,
|
||
|
|
'price_subtotal': 450,
|
||
|
|
'price_subtotal_incl': 450,
|
||
|
|
})],
|
||
|
|
'amount_total': 450,
|
||
|
|
'amount_tax': 0,
|
||
|
|
'amount_paid': 0,
|
||
|
|
'amount_return': 0,
|
||
|
|
'last_order_preparation_change': '{}'
|
||
|
|
})
|
||
|
|
|
||
|
|
# I make a payment to fully pay the order
|
||
|
|
context_make_payment = {"active_ids": [self.pos_order_pos0.id], "active_id": self.pos_order_pos0.id}
|
||
|
|
self.pos_make_payment_0 = self.PosMakePayment.with_context(context_make_payment).create({
|
||
|
|
'amount': 450.0,
|
||
|
|
'payment_method_id': self.cash_payment_method.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
# I click on the validate button to register the payment.
|
||
|
|
context_payment = {'active_id': self.pos_order_pos0.id}
|
||
|
|
self.pos_make_payment_0.with_context(context_payment).check()
|
||
|
|
|
||
|
|
# I close the current session to generate the journal entries
|
||
|
|
current_session_id = self.pos_config.current_session_id
|
||
|
|
current_session_id.post_closing_cash_details(450.0)
|
||
|
|
current_session_id.close_session_from_ui()
|
||
|
|
self.assertEqual(current_session_id.state, 'closed', 'Check that session is closed')
|
||
|
|
|
||
|
|
self.assertEqual(len(current_session.picking_ids), 2, "There should be 2 pickings")
|
||
|
|
current_session.picking_ids.move_ids_without_package.write({'quantity': 1, 'picked': True})
|
||
|
|
current_session.picking_ids.button_validate()
|
||
|
|
|
||
|
|
# I test that the generated journal entries are correct.
|
||
|
|
account_output = self.category.property_stock_account_output_categ_id
|
||
|
|
expense_account = self.category.property_account_expense_categ_id
|
||
|
|
aml = current_session._get_related_account_moves().line_ids
|
||
|
|
aml_output = aml.filtered(lambda l: l.account_id.id == account_output.id)
|
||
|
|
aml_expense = aml.filtered(lambda l: l.account_id.id == expense_account.id)
|
||
|
|
|
||
|
|
self.assertEqual(len(aml_output), 3, "There should be 3 output account move lines")
|
||
|
|
# 2 moves in POS journal (Pos order + manual entry at delivery)
|
||
|
|
self.assertEqual(len(aml_output.move_id.filtered(lambda l: l.journal_id == self.pos_config.journal_id)), 2)
|
||
|
|
# 1 move in stock journal (delivery from stock layers)
|
||
|
|
self.assertEqual(len(aml_output.move_id.filtered(lambda l: l.journal_id == self.category.property_stock_journal)), 1)
|
||
|
|
#Check the lines created after the picking validation
|
||
|
|
self.assertEqual(aml_output[2].credit, self.product.standard_price, "Cost of Good Sold entry missing or mismatching")
|
||
|
|
self.assertEqual(aml_output[2].debit, 0.0, "Cost of Good Sold entry missing or mismatching")
|
||
|
|
self.assertEqual(aml_output[1].debit, self.product.standard_price, "Cost of Good Sold entry missing or mismatching")
|
||
|
|
self.assertEqual(aml_output[1].credit, 0.0, "Cost of Good Sold entry missing or mismatching")
|
||
|
|
self.assertEqual(aml_expense[1].debit, self.product.standard_price, "Cost of Good Sold entry missing or mismatching")
|
||
|
|
self.assertEqual(aml_expense[1].credit, 0.0, "Cost of Good Sold entry missing or mismatching")
|
||
|
|
#Check the lines created by the PoS session
|
||
|
|
self.assertEqual(aml_output[0].debit, 0.0, "Cost of Good Sold entry missing or mismatching")
|
||
|
|
self.assertEqual(aml_output[0].credit, 0.0, "Cost of Good Sold entry missing or mismatching")
|
||
|
|
self.assertEqual(aml_expense[0].credit, 0.0, "Cost of Good Sold entry missing or mismatching")
|
||
|
|
self.assertEqual(aml_expense[0].debit, 0.0, "Cost of Good Sold entry missing or mismatching")
|