123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445 |
- # -*- coding: utf-8 -*-
- # Part of Odoo. See LICENSE file for full copyright and licensing details.
- from freezegun import freeze_time
- from odoo.addons.stock_account.tests.test_anglo_saxon_valuation_reconciliation_common import ValuationReconciliationTestCommon
- from odoo.tests.common import Form, tagged
- from odoo import Command, fields
- @tagged('post_install', '-at_install')
- class TestValuationReconciliation(ValuationReconciliationTestCommon):
- @classmethod
- def setup_company_data(cls, company_name, chart_template=None, **kwargs):
- company_data = super().setup_company_data(company_name, chart_template=chart_template, **kwargs)
- # Create stock config.
- company_data.update({
- 'default_account_stock_price_diff': cls.env['account.account'].create({
- 'name': 'default_account_stock_price_diff',
- 'code': 'STOCKDIFF',
- 'reconcile': True,
- 'account_type': 'asset_current',
- 'company_id': company_data['company'].id,
- }),
- })
- return company_data
- def _create_purchase(self, product, date, quantity=1.0, set_tax=False, price_unit=66.0):
- with freeze_time(date):
- rslt = self.env['purchase.order'].create({
- 'partner_id': self.partner_a.id,
- 'currency_id': self.currency_data['currency'].id,
- 'order_line': [
- (0, 0, {
- 'name': product.name,
- 'product_id': product.id,
- 'product_qty': quantity,
- 'product_uom': product.uom_po_id.id,
- 'price_unit': price_unit,
- 'date_planned': date,
- 'taxes_id': [(6, 0, product.supplier_taxes_id.ids)] if set_tax else False,
- })],
- 'date_order': date,
- })
- rslt.button_confirm()
- return rslt
- def _create_invoice_for_po(self, purchase_order, date):
- with freeze_time(date):
- move_form = Form(self.env['account.move'].with_context(default_move_type='in_invoice', default_date=date))
- move_form.invoice_date = date
- move_form.partner_id = self.partner_a
- move_form.currency_id = self.currency_data['currency']
- move_form.purchase_vendor_bill_id = self.env['purchase.bill.union'].browse(-purchase_order.id)
- return move_form.save()
- def test_shipment_invoice(self):
- """ Tests the case into which we receive the goods first, and then make the invoice.
- """
- test_product = self.test_product_delivery
- date_po_and_delivery = '2018-01-01'
- purchase_order = self._create_purchase(test_product, date_po_and_delivery)
- self._process_pickings(purchase_order.picking_ids, date=date_po_and_delivery)
- invoice = self._create_invoice_for_po(purchase_order, '2018-02-02')
- invoice.action_post()
- picking = self.env['stock.picking'].search([('purchase_id','=',purchase_order.id)])
- self.check_reconciliation(invoice, picking)
- # cancel the invoice
- invoice.button_cancel()
- def test_invoice_shipment(self):
- """ Tests the case into which we make the invoice first, and then receive the goods.
- """
- # Create a PO and an invoice for it
- test_product = self.test_product_order
- purchase_order = self._create_purchase(test_product, '2017-12-01')
- invoice = self._create_invoice_for_po(purchase_order, '2017-12-23')
- move_form = Form(invoice)
- with move_form.invoice_line_ids.edit(0) as line_form:
- line_form.quantity = 1
- invoice = move_form.save()
- # Validate the invoice and refund the goods
- invoice.action_post()
- self._process_pickings(purchase_order.picking_ids, date='2017-12-24')
- picking = self.env['stock.picking'].search([('purchase_id', '=', purchase_order.id)])
- self.check_reconciliation(invoice, picking)
- # Return the goods and refund the invoice
- with freeze_time('2018-01-13'):
- stock_return_picking_form = Form(self.env['stock.return.picking'].with_context(
- active_ids=picking.ids, active_id=picking.ids[0], active_model='stock.picking'))
- stock_return_picking = stock_return_picking_form.save()
- stock_return_picking.product_return_moves.quantity = 1.0
- stock_return_picking_action = stock_return_picking.create_returns()
- return_pick = self.env['stock.picking'].browse(stock_return_picking_action['res_id'])
- return_pick.action_assign()
- return_pick.move_ids.quantity_done = 1
- return_pick._action_done()
- # Refund the invoice
- refund_invoice_wiz = self.env['account.move.reversal'].with_context(active_model="account.move", active_ids=[invoice.id]).create({
- 'reason': 'test_invoice_shipment_refund',
- 'refund_method': 'cancel',
- 'date': '2018-03-15',
- 'journal_id': invoice.journal_id.id,
- })
- refund_invoice = self.env['account.move'].browse(refund_invoice_wiz.reverse_moves()['res_id'])
- # Check the result
- self.assertEqual(invoice.payment_state, 'reversed', "Invoice should be in 'reversed' state")
- self.assertEqual(refund_invoice.payment_state, 'paid', "Refund should be in 'paid' state")
- self.check_reconciliation(refund_invoice, return_pick)
- def test_multiple_shipments_invoices(self):
- """ Tests the case into which we receive part of the goods first, then 2 invoices at different rates, and finally the remaining quantities
- """
- test_product = self.test_product_delivery
- date_po_and_delivery0 = '2017-01-01'
- purchase_order = self._create_purchase(test_product, date_po_and_delivery0, quantity=5.0)
- self._process_pickings(purchase_order.picking_ids, quantity=2.0, date=date_po_and_delivery0)
- picking = self.env['stock.picking'].search([('purchase_id', '=', purchase_order.id)], order="id asc", limit=1)
- invoice = self._create_invoice_for_po(purchase_order, '2017-01-15')
- move_form = Form(invoice)
- with move_form.invoice_line_ids.edit(0) as line_form:
- line_form.quantity = 3.0
- invoice = move_form.save()
- invoice.action_post()
- self.check_reconciliation(invoice, picking, full_reconcile=False)
- invoice2 = self._create_invoice_for_po(purchase_order, '2017-02-15')
- move_form = Form(invoice2)
- with move_form.invoice_line_ids.edit(0) as line_form:
- line_form.quantity = 2.0
- invoice2 = move_form.save()
- invoice2.action_post()
- self.check_reconciliation(invoice2, picking, full_reconcile=False)
- # We don't need to make the date of processing explicit since the very last rate
- # will be taken
- self._process_pickings(purchase_order.picking_ids.filtered(lambda x: x.state != 'done'), quantity=3.0)
- picking = self.env['stock.picking'].search([('purchase_id', '=', purchase_order.id)], order='id desc', limit=1)
- self.check_reconciliation(invoice2, picking)
- def test_rounding_discount(self):
- self.env.ref("product.decimal_discount").digits = 5
- tax_exclude_id = self.env["account.tax"].create(
- {
- "name": "Exclude tax",
- "amount": "0.00",
- "type_tax_use": "purchase",
- }
- )
- test_product = self.test_product_delivery
- test_product.supplier_taxes_id = [(6, 0, tax_exclude_id.ids)]
- date_po_and_delivery = '2018-01-01'
- purchase_order = self._create_purchase(test_product, date_po_and_delivery, quantity=10000, set_tax=True)
- self._process_pickings(purchase_order.picking_ids, date=date_po_and_delivery)
- invoice = self._create_invoice_for_po(purchase_order, '2018-01-01')
- # Set a discount
- move_form = Form(invoice)
- with move_form.invoice_line_ids.edit(0) as line_form:
- line_form.discount = 0.92431
- move_form.save()
- invoice.action_post()
- # Check the price difference amount.
- invoice_layer = self.env['stock.valuation.layer'].search([('account_move_line_id', 'in', invoice.line_ids.ids)])
- self.assertTrue(len(invoice_layer) == 1, "A price difference line should be created")
- self.assertAlmostEqual(invoice_layer.value, -3050.22)
- picking = self.env['stock.picking'].search([('purchase_id', '=', purchase_order.id)])
- self.assertAlmostEqual(invoice_layer.value + picking.move_ids.stock_valuation_layer_ids.value, invoice.line_ids[0].debit)
- self.assertAlmostEqual(invoice_layer.value + picking.move_ids.stock_valuation_layer_ids.value, invoice.invoice_line_ids.price_subtotal/2, 2)
- self.check_reconciliation(invoice, picking)
- def test_rounding_price_unit(self):
- self.env.ref("product.decimal_price").digits = 6
- test_product = self.test_product_delivery
- date_po_and_delivery = '2018-01-01'
- purchase_order = self._create_purchase(test_product, date_po_and_delivery, quantity=1000000, price_unit=0.0005)
- self._process_pickings(purchase_order.picking_ids, date=date_po_and_delivery)
- invoice = self._create_invoice_for_po(purchase_order, '2018-01-01')
- # Set a discount
- move_form = Form(invoice)
- with move_form.invoice_line_ids.edit(0) as line_form:
- line_form.price_unit = 0.0006
- move_form.save()
- invoice.action_post()
- # Check the price difference amount. It's expected that price_unit * qty != price_total.
- invoice_layer = self.env['stock.valuation.layer'].search([('account_move_line_id', 'in', invoice.line_ids.ids)])
- self.assertTrue(len(invoice_layer) == 1, "A price difference line should be created")
- # self.assertAlmostEqual(invoice_layer.price_unit, 0.0001)
- self.assertAlmostEqual(invoice_layer.value, 50.0)
- picking = self.env['stock.picking'].search([('purchase_id', '=', purchase_order.id)])
- self.check_reconciliation(invoice, picking)
- @freeze_time('2021-01-03')
- def test_price_difference_exchange_difference_accounting_date(self):
- self.stock_account_product_categ.property_account_creditor_price_difference_categ = self.company_data['default_account_stock_price_diff']
- test_product = self.test_product_delivery
- test_product.categ_id.write({"property_cost_method": "standard"})
- test_product.write({'standard_price': 100.0})
- date_po_receipt = '2021-01-02'
- rate_po_receipt = 25.0
- date_bill = '2021-01-01'
- rate_bill = 30.0
- date_accounting = '2021-01-03'
- rate_accounting = 26.0
- foreign_currency = self.currency_data['currency']
- company_currency = self.env.company.currency_id
- self.env['res.currency.rate'].create([
- {
- 'name': date_po_receipt,
- 'rate': rate_po_receipt,
- 'currency_id': foreign_currency.id,
- 'company_id': self.env.company.id,
- }, {
- 'name': date_bill,
- 'rate': rate_bill,
- 'currency_id': foreign_currency.id,
- 'company_id': self.env.company.id,
- }, {
- 'name': date_accounting,
- 'rate': rate_accounting,
- 'currency_id': foreign_currency.id,
- 'company_id': self.env.company.id,
- }, {
- 'name': date_po_receipt,
- 'rate': 1.0,
- 'currency_id': company_currency.id,
- 'company_id': self.env.company.id,
- }, {
- 'name': date_accounting,
- 'rate': 1.0,
- 'currency_id': company_currency.id,
- 'company_id': self.env.company.id,
- }, {
- 'name': date_bill,
- 'rate': 1.0,
- 'currency_id': company_currency.id,
- 'company_id': self.env.company.id,
- }])
- #purchase order created in foreign currency
- purchase_order = self._create_purchase(test_product, date_po_receipt, quantity=10, price_unit=3000)
- with freeze_time(date_po_receipt):
- self._process_pickings(purchase_order.picking_ids)
- invoice = self._create_invoice_for_po(purchase_order, date_bill)
- with Form(invoice) as move_form:
- move_form.invoice_date = fields.Date.from_string(date_bill)
- move_form.date = fields.Date.from_string(date_accounting)
- invoice.action_post()
- price_diff_line = invoice.line_ids.filtered(lambda l: l.account_id == self.stock_account_product_categ.property_account_creditor_price_difference_categ)
- self.assertTrue(len(price_diff_line) == 1, "A price difference line should be created")
- self.assertAlmostEqual(price_diff_line.balance, 192.31)
- self.assertAlmostEqual(price_diff_line.price_subtotal, 5000.0)
- picking = self.env['stock.picking'].search([('purchase_id', '=', purchase_order.id)])
- interim_account_id = self.company_data['default_account_stock_in'].id
- valuation_line = picking.move_ids.mapped('account_move_ids.line_ids').filtered(lambda x: x.account_id.id == interim_account_id)
- self.assertTrue(valuation_line.full_reconcile_id, "The reconciliation should be total at that point.")
- def test_reconcile_cash_basis_bill(self):
- ''' Test the generation of the CABA move after bill payment
- '''
- self.env.company.tax_exigibility = True
- cash_basis_base_account = self.env['account.account'].create({
- 'code': 'cash.basis.base.account',
- 'name': 'cash_basis_base_account',
- 'account_type': 'income',
- 'company_id': self.company_data['company'].id,
- })
- self.company_data['company'].account_cash_basis_base_account_id = cash_basis_base_account
- cash_basis_transfer_account = self.env['account.account'].create({
- 'code': 'cash.basis.transfer.account',
- 'name': 'cash_basis_transfer_account',
- 'account_type': 'income',
- 'company_id': self.company_data['company'].id,
- })
- tax_account_1 = self.env['account.account'].create({
- 'code': 'tax.account.1',
- 'name': 'tax_account_1',
- 'account_type': 'income',
- 'company_id': self.company_data['company'].id,
- })
- tax_tags = self.env['account.account.tag'].create({
- 'name': 'tax_tag_%s' % str(i),
- 'applicability': 'taxes',
- } for i in range(8))
- cash_basis_tax_a_third_amount = self.env['account.tax'].create({
- 'name': 'tax_1',
- 'amount': 33.3333,
- 'company_id': self.company_data['company'].id,
- 'cash_basis_transition_account_id': cash_basis_transfer_account.id,
- 'tax_exigibility': 'on_payment',
- 'invoice_repartition_line_ids': [
- (0, 0, {
- 'repartition_type': 'base',
- 'tag_ids': [(6, 0, tax_tags[0].ids)],
- }),
- (0, 0, {
- 'repartition_type': 'tax',
- 'account_id': tax_account_1.id,
- 'tag_ids': [(6, 0, tax_tags[1].ids)],
- }),
- ],
- 'refund_repartition_line_ids': [
- (0, 0, {
- 'repartition_type': 'base',
- 'tag_ids': [(6, 0, tax_tags[2].ids)],
- }),
- (0, 0, {
- 'repartition_type': 'tax',
- 'account_id': tax_account_1.id,
- 'tag_ids': [(6, 0, tax_tags[3].ids)],
- }),
- ],
- })
- product_A = self.env["product.product"].create(
- {
- "name": "Product A",
- "type": "product",
- "default_code": "prda",
- "categ_id": self.stock_account_product_categ.id,
- "taxes_id": [(5, 0, 0)],
- "supplier_taxes_id": [(6, 0, cash_basis_tax_a_third_amount.ids)],
- "lst_price": 100.0,
- "standard_price": 10.0,
- "property_account_income_id": self.company_data["default_account_revenue"].id,
- "property_account_expense_id": self.company_data["default_account_expense"].id,
- }
- )
- product_A.categ_id.write(
- {
- "property_valuation": "real_time",
- "property_cost_method": "standard",
- }
- )
- date_po_and_delivery = '2018-01-01'
- purchase_order = self._create_purchase(product_A, date_po_and_delivery, set_tax=True, price_unit=300.0)
- self._process_pickings(purchase_order.picking_ids, date=date_po_and_delivery)
- bill = self._create_invoice_for_po(purchase_order, '2018-02-02')
- bill.action_post()
- # Register a payment creating the CABA journal entry on the fly and reconcile it with the tax line.
- self.env['account.payment.register']\
- .with_context(active_ids=bill.ids, active_model='account.move')\
- .create({})\
- ._create_payments()
- partial_rec = bill.mapped('line_ids.matched_debit_ids')
- caba_move = self.env['account.move'].search([('tax_cash_basis_rec_id', '=', partial_rec.id)])
- # Tax values based on payment
- # Invoice amount 300
- self.assertRecordValues(caba_move.line_ids, [
- # pylint: disable=C0326
- # Base amount:
- {'debit': 0.0, 'credit': 150.0, 'amount_currency': -300.0, 'account_id': cash_basis_base_account.id},
- {'debit': 150.0, 'credit': 0.0, 'amount_currency': 300.0, 'account_id': cash_basis_base_account.id},
- # tax:
- {'debit': 0.0, 'credit': 50.0, 'amount_currency': -100.0, 'account_id': cash_basis_transfer_account.id},
- {'debit': 50.0, 'credit': 0.0, 'amount_currency': 100.0, 'account_id': tax_account_1.id},
- ])
- def test_reconciliation_differed_billing(self):
- """
- Test whether products received billed at different time will be correctly reconciled
- valuation: automated
- - create a rfq
- - receive products
- - create bill - set quantity of product A = 0 - save
- - create bill - confirm
- -> the reconciliation should not take into account the lines of the first bill
- """
- date_po_and_delivery = '2022-03-02'
- self.product_a.write({
- 'categ_id': self.stock_account_product_categ,
- 'detailed_type': 'product',
- })
- self.product_b.write({
- 'categ_id': self.stock_account_product_categ,
- 'detailed_type': 'product',
- })
- purchase_order = self.env['purchase.order'].create({
- 'currency_id': self.currency_data['currency'].id,
- 'order_line': [
- Command.create({
- 'name': self.product_a.name,
- 'product_id': self.product_a.id,
- 'product_qty': 1,
- }),
- Command.create({
- 'name': self.product_b.name,
- 'product_id': self.product_b.id,
- 'product_qty': 1,
- }),
- ],
- 'partner_id': self.partner_a.id,
- })
- purchase_order.button_confirm()
- self._process_pickings(purchase_order.picking_ids, date=date_po_and_delivery)
- bill_1 = self._create_invoice_for_po(purchase_order, date_po_and_delivery)
- move_form = Form(bill_1)
- with move_form.invoice_line_ids.edit(0) as line_form:
- line_form.quantity = 0
- move_form.save()
- bill_2 = self._create_invoice_for_po(purchase_order, date=date_po_and_delivery)
- bill_2.action_post()
- aml = bill_2.line_ids.filtered(lambda line: line.display_type == "product")
- pol = purchase_order.order_line
- self.assertRecordValues(pol, [{'qty_invoiced': line.qty_received} for line in pol])
- self.assertRecordValues(aml, [{'reconciled': True} for line in aml])
|